GWphua opened a new pull request, #18744:
URL: https://github.com/apache/druid/pull/18744
### Description
#### Log query metrics to QueryStats HashMap
Allow QueryMetrics to participate in populating the map of `QueryStats`.
This allows for polymorphic behaviour, in the case that we want to add
per-query logging based on query type (e.g. GroupByQueryMetrics can now log
additional metrics such as number of merge buffer used).
##### Key changed/added classes in this PR
* `DefaultQueryMetrics`
* `DefaultSearchQueryMetrics`
* `QueryMetrics`
* `QueryLifecycle`
<hr>
This PR has:
- [x] been self-reviewed.
- [ ] 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.
- [ ] been tested in a test Druid cluster. (Pending)
--
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]
