upthewaterspout commented on PR #7596: URL: https://github.com/apache/geode/pull/7596#issuecomment-1110288344
@pivotal-eshu and I discussed an alternative to this fix that involved making all colocated BucketAdvisors use the same read/write lock (instead of many, as we do currently). This would ensure that we could not still have operations in progress on the old primary for some regions as well as in progress on the new primary for other regions. I feel like something along those lines is a cleaner fix - something that doesn't involve introducing additional state with additional locking. -- 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: notifications-unsubscr...@geode.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
