dlmarion commented on code in PR #3192:
URL: https://github.com/apache/accumulo/pull/3192#discussion_r1103203398
##########
server/base/src/main/java/org/apache/accumulo/server/ServerOpts.java:
##########
@@ -20,17 +20,11 @@
import org.apache.accumulo.core.cli.ConfigOpts;
-import com.beust.jcommander.Parameter;
+public final class ServerOpts extends ConfigOpts {
-public class ServerOpts extends ConfigOpts {
+ // This class is empty on purpose. The intent here is that
+ // the Accumulo server processes will only ConfigOpts. Can't
+ // make ConfigOpts final as it's used by utility classes
+ // that subclass it.
Review Comment:
removed in fae4135
--
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]
