kevinrr888 commented on code in PR #4524: URL: https://github.com/apache/accumulo/pull/4524#discussion_r1702286669
########## core/src/main/java/org/apache/accumulo/core/fate/AbstractFateStore.java: ########## @@ -57,8 +57,8 @@ import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; // TODO 4131 should probably add support to AbstractFateStore, MetaFateStore, -// and UserFateStore to accept null lockID and zooCache (maybe make these fields -// Optional<>). This could replace the current createDummyLockID(). This support +// and UserFateStore to accept null lockID (maybe make this field Optional<>). Review Comment: Yes, I have all the TODOs marked `TODO 4131`. I was planning on creating new issues and deleting all the `TODO 4131` when this PR is ready to be merged. -- 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]
