dimas-b commented on code in PR #1802: URL: https://github.com/apache/polaris/pull/1802#discussion_r2126899050
########## persistence/relational-jdbc/src/main/java/org/apache/polaris/persistence/relational/jdbc/QueryGenerator.java: ########## @@ -37,35 +40,51 @@ public class QueryGenerator { - public static <T> String generateSelectQuery( + public static class PreparedQuery { + private final String sql; + private final List<Object> parameters; + + public PreparedQuery(String sql, List<Object> parameters) { + this.sql = sql; + this.parameters = parameters; + } + + public String getSql() { + return sql; + } + + public List<Object> getParameters() { + return parameters; + } + } + + public static <T> PreparedQuery generateSelectQuery( @Nonnull Converter<T> entity, @Nonnull Map<String, Object> whereClause) { Review Comment: `PreparedStatement` by itself is not very meaningful unless the SQL that goes into it is detached from the runtime parameters. The `PreparedStatement` ensures that parameters are separated at execution time. I believe it is also important to isolate runtime parameters at SQL text generation time (I do not think all SQL in this Persistence can be constant, but we should make the SQL generation work without using runtime entities as input). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org