Skip to content
Snippets Groups Projects
Commit 4e22221a authored by Sonia Zorba's avatar Sonia Zorba
Browse files

Removed quotes from group names

parent acb4e2b9
Branches
No related tags found
No related merge requests found
...@@ -16,7 +16,7 @@ public class PrivateRowsQueryExecutor extends QueryExecutor { ...@@ -16,7 +16,7 @@ public class PrivateRowsQueryExecutor extends QueryExecutor {
CustomJobOwner user = (CustomJobOwner) jobOwner; CustomJobOwner user = (CustomJobOwner) jobOwner;
statement.execute("SET my.tap_groups = \"{" statement.execute("SET my.tap_groups = \"{"
+ String.join(",", user.getGroups().stream() + String.join(",", user.getGroups().stream()
.map(g -> "'" + g.replace("'", "''") + "'") .map(g -> g.replace("'", "''"))
.collect(Collectors.toList())) .collect(Collectors.toList()))
+ "}\""); + "}\"");
} }
......
...@@ -19,7 +19,7 @@ public class PrivateRowsQueryExecutorTest { ...@@ -19,7 +19,7 @@ public class PrivateRowsQueryExecutorTest {
Statement mockedStatement = mock(Statement.class); Statement mockedStatement = mock(Statement.class);
queryExecutor.executeQuery(mockedStatement, "", user); 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()); user = new CustomJobOwner("anonymous", Arrays.asList());
queryExecutor.executeQuery(mockedStatement, "", user); queryExecutor.executeQuery(mockedStatement, "", user);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment