milleruntime commented on PR #2780:
URL: https://github.com/apache/accumulo/pull/2780#issuecomment-1167758273
@ctubbsii @EdColeman In my latest changes I added another method to
instanceOps and a new enum type to assist with the multiple TXs use case:
```java
void executeFateAction(FateAction action, List<FateTxId> transactions)
throws AccumuloException, AccumuloSecurityException;
```
I started writing the javadoc for the API and realized we need to provide
some background about how to use some of the methods, such as `cancel` and
`fail`, that can only be used on `FateTransactions` with a certain status. I
thought the name `FateAction` is more appropriate for the API, instead of
"operation", which refers to the FATE REPO. Then changed the Thrift types to
reflect this naming. I also created an enum in FateTransaction for `Status`
that directly mirrors the internal enum `TStatus` in `ReadOnlyTStore`. Let me
know what you think.
--
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]
