mihaibudiu commented on code in PR #3626:
URL: https://github.com/apache/calcite/pull/3626#discussion_r1452671962
##########
core/src/test/java/org/apache/calcite/plan/RelWriterTest.java:
##########
@@ -442,6 +443,32 @@ class RelWriterTest {
+ " ]\n"
+ "}";
+ public static final String LIMIT_DYNAMIC_PARAM = "{\n"
Review Comment:
given that this is used only once my personal preference would be to make it
a local variable to the function that uses it. But that's not a big deal.
--
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: commits-unsubscr...@calcite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
