cgivre commented on code in PR #2747:
URL: https://github.com/apache/drill/pull/2747#discussion_r1169428023
##########
contrib/storage-http/README.md:
##########
@@ -138,11 +140,26 @@ key2=value2"
```
`postBodyLocation`: If the API uses the `POST` method, you can send
parameters in several different ways:
-* `query_string`: Parameters from the query are pushed down to the query
string. Static parameters are pushed to the post body.
Review Comment:
If the user enables this feature, are they required to specify the
destination of the supplied parameters? What happens if they don't?
##########
contrib/storage-http/README.md:
##########
@@ -15,6 +15,7 @@ To configure the plugin, create a new storage plugin, and add
the following conf
{
"type": "http",
"cacheResults": true,
+ "useLegacyRequestParamSyntax": false,
Review Comment:
See comment. I'd like to make the default behavior what Drill currently
does and have the user explicitly enable this functionality. Could we rename
this param `advanced` or `enhanced` and thus change the boolean to
`enableEnhancedParamSyntax`?
--
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
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
