nielsbasjes commented on pull request #2112: URL: https://github.com/apache/drill/pull/2112#issuecomment-736367332
I'm cool with what makes it work for Drill. Do note that if you have a LOT of query params and you only need a single won then putting them all into a map will have a impact on performance as all parameters are extracted, urldecoded and put into a map. Yet when you only want a single one the system is optimized to only do those steps for the requested field. In general I expect the performance impact of this to be tiny. ---------------------------------------------------------------- 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]
