kevinrr888 commented on code in PR #4787:
URL: https://github.com/apache/accumulo/pull/4787#discussion_r1704591542
##########
core/src/main/java/org/apache/accumulo/core/fate/user/UserFateStore.java:
##########
@@ -105,14 +106,17 @@ public FateId create() {
case ACCEPTED:
return fateId;
case UNKNOWN:
+ log.error("Fate creation returned UNKNOWN status, raising
exception");
+ throw new IllegalStateException("Fate creation returned UNKNOWN
status");
Review Comment:
I don't think we want to throw an exception in this case. This just means
it's unknown if the mutation was successfully written or not, in which case we
would want to just retry. The UNKNOWN status isn't quite equivalent to the
KeeperException thrown in MFS
--
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]
