arina-ielchiieva 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-453924531
 
 
   @kkhatua I also agree with Igor. Parsing query on the client side is error 
prone (I already can think of two cases that was missed: values queries and 
multiline comments) and looks like overhead when on server side we have all 
means to add query limit without line by line parsing. You don't need to change 
existing REST API, just add new call for Web UI that would pass limit along 
with the query. You can also take a look at JDBC calls where limit 0 is added 
for the queries to find out data types.

----------------------------------------------------------------
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

Reply via email to