gianm commented on a change in pull request #10040:
URL: https://github.com/apache/druid/pull/10040#discussion_r441786602
##########
File path: sql/src/main/java/org/apache/druid/sql/avatica/DruidMeta.java
##########
@@ -446,15 +446,16 @@ public MetaResultSet getColumns(
}
if (schemaPattern.s != null) {
- whereBuilder.add("COLUMNS.TABLE_SCHEMA LIKE " +
Calcites.escapeStringLiteral(schemaPattern.s));
Review comment:
This looks like the right fix. Could you please make it to all of
`getSchemas`, `getTables`, and `getColumns`? They have the same problem. It
might be nice to create a private static helper function in this file to avoid
all the boilerplate, too.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
