[GitHub] [druid] paul-rogers commented on pull request #13049: Synchronize query context getMergedParams()

[GitBox <gi...@apache.org>]

paul-rogers commented on PR #13049:
URL: https://github.com/apache/druid/pull/13049#issuecomment-1242112308

   All good points. More fundamentally, context variables only make sense to set in the planning thread, then only read in the execution threads. It is inherently unstable to change a value in one thread concurrently with reading the context in another. 
   
   I’ll do more debug code to track down the issue and enforce proper semantics.


-- 
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: commits-unsubscribe@druid.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org