cshannon commented on PR #4204: URL: https://github.com/apache/accumulo/pull/4204#issuecomment-1934259556
> Stubbed out possible interaction between fate and the fate store for keyed operations in #4178. Can see relevant changes in [FateStore](https://github.com/apache/accumulo/blob/8d5046f71624d14625101707ac8779f07b1cde01/core/src/main/java/org/apache/accumulo/core/fate/FateStore.java#L45-L66), [Fate](https://github.com/apache/accumulo/blob/8d5046f71624d14625101707ac8779f07b1cde01/core/src/main/java/org/apache/accumulo/core/fate/Fate.java#L339-L354), and [SplitTask](https://github.com/apache/accumulo/blob/8d5046f71624d14625101707ac8779f07b1cde01/server/manager/src/main/java/org/apache/accumulo/manager/split/SplitTask.java#L46-L56) I'll take a look in detail tomorrow, I see you added `Optional<FateTxStore<T>> createAndReserve(FateKey fateKey` and that is something I will likely add to this PR when I update it. I think it makes sense to add it now vs a follow on -- 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: notifications-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
