[GitHub] arina-ielchiieva commented on issue #1608: DRILL-6960: Auto Limit Wrapping should not apply to non-select query

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org


With regards,
Apache Git Services