keith-turner commented on issue #4710: URL: https://github.com/apache/accumulo/issues/4710#issuecomment-2251008016
While researching this noticed the volume replace condition update is checking the files it added are present on failure. Because of race conditions they may have been added but not be present. It should probably check the files it removed are not there to see if the update went through. I plan to make this change when adding test for volume replacement, this is comment is a reminder to do that. https://github.com/apache/accumulo/blob/3ad9198a86ae9f8b2b24831f6593d33362f8088a/server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java#L1051-L1053 -- 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]
