[cleanup][cli] Cleanup jcommander

Signed-off-by: Zixuan Liu <nodeces@gmail.com>

distribution/server/src/assemble/LICENSE.bin.txt

@@ -243,7 +243,6 @@ support library is itself covered by the above license.
 This projects includes binary packages with the following licenses:
 
 The Apache Software License, Version 2.0
- * JCommander -- com.beust-jcommander-1.82.jar
  * Picocli
      - info.picocli-picocli-4.7.5.jar
      - info.picocli-picocli-shell-jline3-4.7.5.jar

distribution/shell/src/assemble/LICENSE.bin.txt

@@ -309,7 +309,6 @@ pulsar-client-cpp/lib/checksum/crc32c_sw.cc
 This projects includes binary packages with the following licenses:
 
 The Apache Software License, Version 2.0
- * JCommander -- jcommander-1.82.jar
  * Picocli
      - picocli-4.7.5.jar
      - picocli-shell-jline3-4.7.5.jar

pom.xml

@@ -213,7 +213,6 @@ flexible messaging model and an intuitive client API.</description>
     <confluent.version>6.2.8</confluent.version>
     <aircompressor.version>0.20</aircompressor.version>
     <asynchttpclient.version>2.12.1</asynchttpclient.version>
-    <jcommander.version>1.82</jcommander.version>
     <commons-lang3.version>3.11</commons-lang3.version>
     <commons-configuration.version>1.10</commons-configuration.version>
     <commons-io.version>2.8.0</commons-io.version>
@@ -693,12 +692,6 @@ flexible messaging model and an intuitive client API.</description>
         <classifier>linux-aarch_64</classifier>
       </dependency>
 
-      <dependency>
-        <groupId>com.beust</groupId>
-        <artifactId>jcommander</artifactId>
-        <version>${jcommander.version}</version>
-      </dependency>
-
       <dependency>
         <groupId>info.picocli</groupId>
         <artifactId>picocli</artifactId>

pulsar-cli-utils/pom.xml

@@ -35,11 +35,6 @@
   <description>Isolated CLI utility module</description>
 
   <dependencies>
-    <dependency>
-      <groupId>com.beust</groupId>
-      <artifactId>jcommander</artifactId>
-      <scope>compile</scope>
-    </dependency>
     <dependency>
       <groupId>info.picocli</groupId>
       <artifactId>picocli</artifactId>

pulsar-cli-utils/src/test/java/org/apache/pulsar/cli/converters/TimeConversionTest.java

@@ -22,6 +22,7 @@
 import static org.testng.Assert.assertThrows;
 import java.util.concurrent.TimeUnit;
 import org.apache.pulsar.cli.converters.picocli.TimeUnitToMillisConverter;
+import org.apache.pulsar.cli.converters.picocli.TimeUnitToSecondsConverter;
 import org.testng.annotations.DataProvider;
 import org.testng.annotations.Test;
 
@@ -57,8 +58,8 @@ public void testSuccessfulRelativeTimeUtilParsing(String input, long expected) {
     }
 
     @Test(dataProvider = "successfulRelativeTimeUtilTestCases")
-    public void testSuccessfulTimeUnitToSecondsConverter(String input, long expected) {
-        TimeUnitToSecondsConverter secondsConverter = new TimeUnitToSecondsConverter("optionName");
+    public void testSuccessfulTimeUnitToSecondsConverter(String input, long expected) throws Exception {
+        TimeUnitToSecondsConverter secondsConverter = new TimeUnitToSecondsConverter();
         assertEquals(secondsConverter.convert(input), Long.valueOf(expected));
     }