LYCJeff commented on PR #2747:
URL: https://github.com/apache/drill/pull/2747#issuecomment-1451555417

   > Two ideas
   > 
   > 1. Since we won't backport this PR and it will only go out in the next 
major release, some breakage inside a plugin is probably something that can be 
swallowed.
   > 2. If it is still desired to preserve the ability to use the existing 
syntax in Drill 1.22 and beyond then a storage config option like 
`"useLegacyRequestParmSyntax": true` could be added for users who want it.
   
   @jnturton @cgivre That's a good idea without confusing old and new syntax, 
although it requires existing users to make small additions to the 
configuration. If it is acceptable to you, I will take some time to add a 
configuration item in the near future.


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

Reply via email to