paul-rogers commented on issue #2052: DRILL-7603 and DRILL-7604: Add schema, 
options to REST query
URL: https://github.com/apache/drill/pull/2052#issuecomment-612576587
 
 
   @arina-ielchiieva, thanks much for the review. As far as I could tell, the 
only UI changes in the original PR were for the default schema. It would take a 
bit of effort to create a UI for a list of options. In fact, it might be better 
to extend the back-end to allow the user to post a series of queries in one go, 
with all but the last being trivial. This approach would be easier for humans 
because there is no need to figure out how a UI maps to a command:
   
   ```
   USE foo;
   ALTER SESSION SET `bar` = 'mumble';
   SELECT ... FROM something ...
   ```
   
   For the REST API, it is probably easier to include these extras in the REST 
request. A clever client can accumulate changes and include them with each 
query request.
   
   The REST API can allow additional features that the UI ignores. In the case 
of options, the UI won't post any and the back-end will skip the options step.
   
   What do you think?
   
   Finally, thanks for pointing out DRILL-7655. How did that get filed if the 
original PR never made it into master? Anyway, I agree it should be fixed and 
I'll look into that for this PR.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to