kkhatua edited a comment on issue #1608: DRILL-6960: AutoLimit the size of ResultSet for a WebUI (or REST) client URL: https://github.com/apache/drill/pull/1608#issuecomment-461208356 @vvysotskyi (cc @ihuzenko ) I took your advise and enforced the limit in the WebUserConnection. That way, I don't modify the query and no changes to the protobufs either. Thanks for pointing me to the approach as it is much cleaner. This way, we mimic exactly what a JDBC driver would do after fetching a fixed number of records before closing the connection. Still testing some corner cases. Will update here when ready.
---------------------------------------------------------------- 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
