dlmarion commented on PR #6121: URL: https://github.com/apache/accumulo/pull/6121#issuecomment-3892269428
FYI that with the CommandGroup that was added in https://github.com/apache/accumulo/commit/151a7dad03df0930df3eaa867d3aadd59e0ab680 to restore the backward compatibility of KeywordExecutable, the classloader commands would continue to use their existing keyword() values in version 2.1 (`accumulo create-classloader-manifest` and `accumulo init-classloader-cache-dir`). Currently in this PR there is no default CommandGroup on the KeywordExecutable interface. This makes it easier to track that all implementations have a CommandGroup specified until we have them where we want them. If we made the default `OTHER`, then in 4.0 the classloader commands would be: ``` accumulo other create-classloader-manifest accumulo other init-classloader-cache-dir ``` -- 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]
