tkalkirill commented on code in PR #5004: URL: https://github.com/apache/calcite/pull/5004#discussion_r3577954426
########## core/src/main/java/org/apache/calcite/interpreter/SortNode.java: ########## Review Comment: Yes, FETCH and OFFSET should share the same runtime numeric conversion and validation. I will update SortNode to handle the currently supported OFFSET literals and dynamic parameters consistently with FETCH. Support for parenthesized OFFSET expressions and LIMIT expressions will remain out of scope. -- 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]
