You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2022/11/05 13:19:09 UTC

[GitHub] [drill] xqgzh opened a new issue, #2701: Http Storage Plugins need reorder "tail" and paginator parameters

xqgzh opened a new issue, #2701:
URL: https://github.com/apache/drill/issues/2701

   **Describe the bug**
   append the paginator parameters after tail (when requireTail = true)
   
   **To Reproduce**
   1. set the HTTP Storage Plugins configuration as follows:
   {
     "type": "http",
     "connections": {
       "sunrise": {
         "url": "https://api.sunrise-sunset.org/json",
         "requireTail": true,
         .....
         "paginator": {
          "limitParam": "limit",
          "offsetParam": "offset",
          "pageSize": 100,
          "method": "OFFSET"
         }
       }
     }
   }
   
   2. when i exec sql like this:
    select * from http.sunrise.`/test/123.json` limit 10 offset 20;
   
   3. I get the error request url from drill:
   "GET https://api.sunrise-sunset.org/json?offset=20&limit=10/test/123.json"
   
   4. but I expect the request is: 
   "GET https://api.sunrise-sunset.org/json/test/123.json?offset=20&limit=10"
   
   


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

To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [drill] cgivre closed issue #2701: Http Storage Plugins need reorder "tail" and paginator parameters

Posted by GitBox <gi...@apache.org>.
cgivre closed issue #2701: Http Storage Plugins need reorder "tail" and paginator parameters
URL: https://github.com/apache/drill/issues/2701


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

To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [drill] cgivre commented on issue #2701: Http Storage Plugins need reorder "tail" and paginator parameters

Posted by GitBox <gi...@apache.org>.
cgivre commented on issue #2701:
URL: https://github.com/apache/drill/issues/2701#issuecomment-1336515492

   Hi @xqgzh 
   I just submitted a bug fix for this.  If you could take a look and review, I'd really appreciate it. 


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

To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org