kkhatua commented on issue #1608: DRILL-6960: Auto Limit Wrapping should not apply to non-select query URL: https://github.com/apache/drill/pull/1608#issuecomment-456216764 @ihuzenko / @arina-ielchiieva / @vdiravka I've made changes within this PR to capture the requirements. The automatic limitting of the resultset is now handled by the server, and involved making changes to the Protobuf as well. Here are the changes done for the latest commit to this PR. 1. Submit autoLimit value to server 2. Server is able to detect the autoLimit and apply the wrapper around the input SQL. 3. If autoLimit is not applicable, remove the value to avoid incorrectly marking the profile during execution. 4. Optional AutoLimit value is set in the profile if autoLimit is being applied 5. WebUI will render a pinned warning indicating that the result set size was artificially limited (NOTE: this is a workaround since QueryManager cannot modify the actually query text with header comments, etc.)
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
