[GitHub] [geode] DonalEvans commented on pull request #5590: GEODE-8546: added retry create buckets in case colocation is not comp…

[GitBox <gi...@apache.org>]

DonalEvans commented on pull request #5590:
URL: https://github.com/apache/geode/pull/5590#issuecomment-721882594


   > @DonalEvans Do you have any capacity to assist with this?
   
   I emailed back and forth with Mario a bit trying to figure out a way to reproduce the issue in a DUnit test, but it's not an area of the code I'm particularly familiar with so my ability to help was fairly limited. I thought that if the assumption that the problem was caused by colocation taking too long was correct, then artificially slowing down colocation by using a listener to wait a little after each bucket is created could reproduce it.
   
   Since part of the proposed fix seems to be to wait a total of 9000ms for colocation to complete in `CreateMIssingBucketTask.java`, it seems like forcing colocation to take close to that long should be a sure-fire way to reproduce the issue, if the fix is effective. It seems like this wasn't the case though, so maybe my idea was off the mark.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org