sanjeet006py commented on PR #6868: URL: https://github.com/apache/hbase/pull/6868#issuecomment-2948002341
> What if, for example, instead of altering the existing fields, you collect the per-region values in their own accumulators, and update the existing scan-level field values on each transition. That way the existing references stay as final and existing access patterns remain valid. @ndimiduk I have modified the implementation to incorporate above suggestion. I have implemented a slight variation of above approach: I now collect per-region values in their own accumulators but while updating per-region values I also update existing scan-level field values. This way existing access patterns remain valid. I have also addressed all the rest of open comments. Please take a look. Thanks -- 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]
