sanjeet006py commented on PR #6868: URL: https://github.com/apache/hbase/pull/6868#issuecomment-2827171706
> One high-level question though: why are you adding this as a bolt-on" instead of extending the existing ScanMetrics object with the per-region collection? Actually I didn't consider extending existing ScanMetrics. And, I agree by extending existing ScanMetrics many of above comments will also get resolved along with making implementation more cleaner. Will rework on this. Thanks a lot. -- 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: issues-unsubscr...@hbase.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
