snuyanzin commented on code in PR #22179: URL: https://github.com/apache/flink/pull/22179#discussion_r1796853268
########## flink-table/flink-sql-parser/src/test/java/org/apache/flink/sql/parser/FlinkSqlParserImplTest.java: ########## @@ -52,6 +52,24 @@ public SqlParserFixture fixture() { @Test void testShowCatalogs() { sql("show catalogs").ok("SHOW CATALOGS"); + + // ----------- The following cases are used to detect LIKE ----------- + + sql("show catalogs like '%'").ok("SHOW CATALOGS LIKE '%'"); + sql("show catalogs not like '%'").ok("SHOW CATALOGS NOT LIKE '%'"); + + // ----------- The following cases are used to detect ILIKE ----------- + Review Comment: it's better to remove this empty line otherwise one might think (including me) that this comment relates to all the rest lines in this test -- 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...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org