[GitHub] [ratis] leo65535 commented on a diff in pull request #711: RATIS-1669. Combine shell lib folder and root jars folder

[GitBox <gi...@apache.org>]

leo65535 commented on code in PR #711:
URL: https://github.com/apache/ratis/pull/711#discussion_r943043823


##########
pom.xml:
##########
@@ -398,6 +398,11 @@
         <scope>test</scope>
         <version>1.7.29</version>
       </dependency>
+      <dependency>
+        <groupId>log4j</groupId>
+        <artifactId>log4j</artifactId>
+        <version>1.2.17</version>
+      </dependency>

Review Comment:
   Make sense, but when we run the command, always output
   ```
   [dcadmin@dcadmin-work apache-ratis-3.0.0-SNAPSHOT-bin]$ bin/ratis sh
   SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
   SLF4J: Defaulting to no-operation (NOP) logger implementation
   SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
   Usage: ratis sh [generic options]
   	 [election [transfer] [stepDown] [pause] [resume]]         
   	 [group [info] [list]]                                     
   	 [peer [add] [remove] [setPriority]]                       
   	 [snapshot [create]]       
   ```



-- 
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: issues-unsubscribe@ratis.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org