[ 
https://issues.apache.org/jira/browse/DRILL-7603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17045112#comment-17045112
 ] 

ASF GitHub Bot commented on DRILL-7603:
---------------------------------------

paul-rogers commented on pull request #1996: DRILL-7603 Allow default schema to 
be set for HTTP queries
URL: https://github.com/apache/drill/pull/1996#discussion_r384257662
 
 

 ##########
 File path: 
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/QueryResources.java
 ##########
 @@ -96,10 +95,12 @@ public QueryResult submitQueryJSON(QueryWrapper query) 
throws Exception {
   @Produces(MediaType.TEXT_HTML)
   public Viewable submitQuery(@FormParam("query") String query,
                               @FormParam("queryType") String queryType,
-                              @FormParam("autoLimit") String autoLimit) throws 
Exception {
+                              @FormParam("autoLimit") String autoLimit,
+                              @FormParam("defaultSchema") String 
defaultSchema) throws Exception {
 
 Review comment:
   Nice improvement! I wonder, would it make sense to accept a list of 
name/value pairs so we can also pass along session options? We could identify 
the schema as a REST-specific property in that list.
 
----------------------------------------------------------------
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:
[email protected]


> Cannot set default schema using REST API / Web UI
> -------------------------------------------------
>
>                 Key: DRILL-7603
>                 URL: https://issues.apache.org/jira/browse/DRILL-7603
>             Project: Apache Drill
>          Issue Type: Improvement
>          Components: Web Server
>            Reporter: Dobes Vandermeer
>            Assignee: Dobes Vandermeer
>            Priority: Minor
>
> Currently, to set the default schema you must run the `USE <schema>` command. 
>  However, the Web UI and REST API do not keep a session open so the `USE` 
> command does not affect the next query sent to the API.
> To support a default schema for REST API & Web UI I propose a parameter to 
> the API "defaultSchema" which sets the default schema for that query.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to