rubenada commented on code in PR #5370:
URL: https://github.com/apache/calcite/pull/5370#discussion_r4457939513


##########
core/src/test/java/org/apache/calcite/test/RecursiveCteTest.java:
##########
@@ -105,6 +116,232 @@ class RecursiveCteTest {
     assertThat(((SqlWithItem) with.withList.get(1)).recursive.booleanValue(), 
is(true));
   }
 
+  /** SEARCH follows AS (query); parsing and unparsing preserve its mode and 
keys. */

Review Comment:
   Apart from these tests in RecursiveCteTest, should we add also new tests 
parser-wise on  SqlParserTest?



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to