paul-rogers opened a new pull request, #12633: URL: https://github.com/apache/druid/pull/12633
Query context cleanup in support of the [planner test framework PR](https://github.com/apache/druid/pull/12545). This PR splits up the files from the planner test PR to make review a bit easier. * Uses `QueryContext` constants in place of literal string keys for several commonly-used keys. * Moves some context-related implementations from `QueryContext` to `QueryContexts` so that they can be reused, such as in the planner tests. No functional changes; only refactoring. <hr> This PR has: - [X] been self-reviewed. - [X] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader. - [X] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for [code coverage](https://github.com/apache/druid/blob/master/dev/code-review/code-coverage.md) is met. -- 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]
