ctubbsii commented on code in PR #3192:
URL: https://github.com/apache/accumulo/pull/3192#discussion_r1103174553
##########
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:
If we need to re-add it in future, that's fine. If not, it stays deleted.
It's internal code organization only, not public API, so it's okay to delete if
we don't need it.
--
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]
