Manno15 commented on a change in pull request #2215:
URL: https://github.com/apache/accumulo/pull/2215#discussion_r678581217
##########
File path:
core/src/main/java/org/apache/accumulo/core/client/admin/InstanceOperations.java
##########
@@ -149,6 +152,59 @@ boolean testClassLoad(final String className, final String
asTypeName)
*/
void waitForBalance() throws AccumuloException;
+ /**
+ * Throws an exception if a tablet server can not be contacted.
+ *
+ * @param args
+ * Command line arguments passed in from user command.
+ * @param secretOption
+ * Specified instance secret to use for commands.
+ *
+ * @since 2.1.0
+ */
+ boolean fateFail(List<String> args, String secretOption) throws
AccumuloException;
Review comment:
> Not sure from this what values are appropriate for "args". Are these
just a list of FaTE transaction IDs to fail? The parameters to these methods
could be more specific.
Yes, Args will be the TiDs. Initially I was going to have it be more broad
and include other things inside it. I can see how it is confusing now. Since I
have parameters specifically for FilterTiDs for the print statement I could
possibly combine the two parameters.
--
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]
