diff --git a/private-rows-extension/src/main/java/it/inaf/ia2/vollt/PrivateRowsQueryExecutor.java b/private-rows-extension/src/main/java/it/inaf/ia2/vollt/PrivateRowsQueryExecutor.java index b660db9cc3818547aa93f54923695aa85624bc2c..de28e109742d4fb10e3da6d341205123cb3efda9 100644 --- a/private-rows-extension/src/main/java/it/inaf/ia2/vollt/PrivateRowsQueryExecutor.java +++ b/private-rows-extension/src/main/java/it/inaf/ia2/vollt/PrivateRowsQueryExecutor.java @@ -14,11 +14,11 @@ public class PrivateRowsQueryExecutor extends QueryExecutor { if (jobOwner instanceof CustomJobOwner) { CustomJobOwner user = (CustomJobOwner) jobOwner; - statement.execute("SET my.tap_groups = \"{" + statement.execute("SET my.tap_groups = '{" + String.join(",", user.getGroups().stream() .map(g -> g.replace("'", "''")) .collect(Collectors.toList())) - + "}\""); + + "}'"); } return super.executeQuery(statement, sql, jobOwner); diff --git a/private-rows-extension/src/test/java/it/inaf/ia2/vollt/PrivateRowsQueryExecutorTest.java b/private-rows-extension/src/test/java/it/inaf/ia2/vollt/PrivateRowsQueryExecutorTest.java index 58443f78c4ae3c7f1cffc0989bbd0eebb7ab1baa..7d5a9e5d81596851c93ac7179ec2fc8e13e61850 100644 --- a/private-rows-extension/src/test/java/it/inaf/ia2/vollt/PrivateRowsQueryExecutorTest.java +++ b/private-rows-extension/src/test/java/it/inaf/ia2/vollt/PrivateRowsQueryExecutorTest.java @@ -19,11 +19,11 @@ public class PrivateRowsQueryExecutorTest { Statement mockedStatement = mock(Statement.class); queryExecutor.executeQuery(mockedStatement, "", user); - verify(mockedStatement).execute(eq("SET my.tap_groups = \"{group1,group2}\"")); + verify(mockedStatement).execute(eq("SET my.tap_groups = '{group1,group2}'")); user = new CustomJobOwner("anonymous", Arrays.asList()); queryExecutor.executeQuery(mockedStatement, "", user); - verify(mockedStatement).execute(eq("SET my.tap_groups = \"{}\"")); + verify(mockedStatement).execute(eq("SET my.tap_groups = '{}'")); } }