FrankChen021 commented on code in PR #13071: URL: https://github.com/apache/druid/pull/13071#discussion_r990609600
########## sql/src/main/java/org/apache/druid/sql/AbstractStatement.java: ########## @@ -49,6 +57,14 @@ public abstract class AbstractStatement implements Closeable protected final SqlToolbox sqlToolbox; protected final SqlQueryPlus queryPlus; protected final SqlExecutionReporter reporter; + + /** + * Copy of the query context modified during planning. Modifications are + * valid in tasks that run in the request thread. Once the query forks + * child threads, then concurrent modifications to the query context will + * result in an undefined muddle of race conditions. + */ + protected final Map<String, Object> queryContext; Review Comment: So this is mutable? -- 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]
