Kwafoor commented on code in PR #931:
URL: https://github.com/apache/incubator-uniffle/pull/931#discussion_r1230332162
##########
cli/src/main/java/org/apache/uniffle/cli/UniffleCLI.java:
##########
@@ -25,28 +25,53 @@
import org.apache.uniffle.AbstractCustomCommandLine;
import org.apache.uniffle.UniffleCliArgsException;
+import org.apache.uniffle.api.AdminRestApi;
+import org.apache.uniffle.client.UniffleRestClient;
+
public class UniffleCLI extends AbstractCustomCommandLine {
private static final Logger LOG = LoggerFactory.getLogger(UniffleCLI.class);
private final Options allOptions;
private final Option uniffleClientCli;
private final Option uniffleAdminCli;
+ private final Option checkerClass;
+ private final Option coordServer;
+ private final Option coordPort;
+ private final Option refreshCheckerCli;
private final Option help;
+ protected UniffleRestClient client;
public UniffleCLI(String shortPrefix, String longPrefix) {
allOptions = new Options();
uniffleClientCli = new Option(shortPrefix + "c", longPrefix + "cli",
true, "This is an client cli command that will print args.");
uniffleAdminCli = new Option(shortPrefix + "a", longPrefix + "admin",
true, "This is an admin command that will print args.");
+ refreshCheckerCli = new Option(shortPrefix + "rc", longPrefix +
"refreshChecker",
+ false, "This is an admin command that will refresh access checker.");
+ checkerClass = new Option(shortPrefix + "cc", longPrefix + "checkerClass",
+ true, "This is the checker class that will be refreshed.");
+ coordServer = new Option(shortPrefix + "s", longPrefix + "host",
Review Comment:
We can fix this with another PR.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
