Skip to content
Commits on Source (2)
......@@ -44,7 +44,7 @@ build_vollt_base_war:
stage: build_base_war
tags:
- docker
image: gradle:jdk8
image: gradle:6-jdk8
only:
refs:
- master
......
FROM gradle:jdk8
FROM gradle:6-jdk8
RUN apt-get update && apt install -y openjdk-14-jdk maven
......@@ -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);
......
......@@ -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 = '{}'"));
}
}