cshannon commented on PR #2799: URL: https://github.com/apache/accumulo/pull/2799#issuecomment-1203035268
@ctubbsii, @EdColeman, @dlmarion - I just pushed an updated draft PR that has changes to handle detect and handle version changes when modifying properties and is ready for review again. I added a new commit for now just to make it easier to see what my latest changes were. I will rebase/squash the commits and update the commit message when it's ready for merging. I took @keith-turner's suggestion and went ahead and updated the `modifyProperties() ` methods to retry on concurrent modification exceptions up to 5 times before giving up and propagating the exception back to the caller of the method. We could also make this fail immediately and not retry if desired. Let me know what you think of the latest updates. -- 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]
