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

[GitBox]

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-unsubscr...@drill.apache.org.apache.org

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