keith-turner commented on PR #2967: URL: https://github.com/apache/accumulo/pull/2967#issuecomment-1269605242
> LGTM, I think the log messages should be modified to note that we are going to retry. The Retry code will append the following to the log message to indicate that its going to retry. https://github.com/apache/accumulo/blob/6976933640be998c01c2a47c4a1bf8d0dca84d86/core/src/main/java/org/apache/accumulo/fate/util/Retry.java#L239-L242 > To that end, it may be useful to log that it was successful when a retry was performed. That would be really nice to do. Looking at the Retry class it does not support this. I am going to look into doing this and see if it should be done in this PR or as a follow on PR that makes a more general change to Retry. -- 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]
