vogievetsky opened a new pull request, #16868:
URL: https://github.com/apache/druid/pull/16868

   This PR is inspired by my comment in another PR 
https://github.com/apache/druid/pull/16819#pullrequestreview-2213300179 . 
Basically I made that comment and I realized it is not 100% true that 
`mandatoryQueryContext` does that @lorem--ipsum tried to do. I instead added 
`baseQueryContext` that I believe works as expected and also added a 
configurable `serverQueryContext` which is my implementation of the 
`clusterDefaultContext` I suggested in the comment.
   
   I believe this PR fully obviates the need for 16819
   
   Also fixes a bug where SQL data loader crashes on 
https://github.com/apache/druid/blob/master/processing/src/test/resources/nested-array-test-data.json


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to