DanielLeens commented on PR #11215:
URL: https://github.com/apache/seatunnel/pull/11215#issuecomment-4885777073

   Thanks for flagging this. I rechecked 
`ClusterMetricExports.resolveClusterSafe()` on the current head, and I agree 
this is still a blocker as written: when the `PartitionServiceMBean` path is 
unavailable, the fallback currently reduces a cluster-level metric to 
local-member evidence only.
   
   Please switch that fallback to a cluster-scoped safety signal and add one 
regression test for the MBean-missing path. After that, and after syncing the 
latest `dev`, I am happy to take another look.
   


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

Reply via email to