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

   <!-- Thanks for trying to help us make Apache Druid be the best it can be! 
Please fill out as much of the following information as is possible (where 
relevant, and remove it when irrelevant) to help make the intention and scope 
of this PR clear in order to ease review. -->
   
   <!-- Please read the doc for contribution 
(https://github.com/apache/druid/blob/master/CONTRIBUTING.md) before making 
this PR. Also, once you open a PR, please _avoid using force pushes and 
rebasing_ since these make it difficult for reviewers to see what you've 
changed in response to their reviews. See [the 'If your pull request shows 
conflicts with master' 
section](https://github.com/apache/druid/blob/master/CONTRIBUTING.md#if-your-pull-request-shows-conflicts-with-master)
 for more details. -->
   
   Fixes #17746 , related to #17360.
   
   ### Description
   
   Previously, #17746 added a table of 3 columns, and is missing `Dimensions`. 
This causes 'Normal value' to not make sense.
   
   
![image](https://github.com/user-attachments/assets/58995dfe-d5cc-40c0-b571-28f61f14b193)
   
   Change now should be ok
   
   
![image](https://github.com/user-attachments/assets/78839056-2186-4e5d-a740-e0ee35c04025)
   
   Also, I followed the example of `SubqueryCountStatsMonitor` by attaching the 
table values under `Broker`, `Historical` and `Realtime` instead. I feel that 
the intended `Dimensions` column only make sense by doing this. 
   
   This PR has:
   
   - [x] been self-reviewed.


-- 
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