address review comments

Author: pan3793

parquet-cli/src/main/java/org/apache/parquet/cli/Main.java

@@ -34,27 +34,7 @@
 import org.apache.hadoop.util.ToolRunner;
 import org.apache.log4j.Level;
 import org.apache.log4j.PropertyConfigurator;
-import org.apache.parquet.cli.commands.CSVSchemaCommand;
-import org.apache.parquet.cli.commands.CatCommand;
-import org.apache.parquet.cli.commands.CheckParquet251Command;
-import org.apache.parquet.cli.commands.ColumnMaskingCommand;
-import org.apache.parquet.cli.commands.ColumnSizeCommand;
-import org.apache.parquet.cli.commands.ConvertCSVCommand;
-import org.apache.parquet.cli.commands.ConvertCommand;
-import org.apache.parquet.cli.commands.ParquetMetadataCommand;
-import org.apache.parquet.cli.commands.PruneColumnsCommand;
-import org.apache.parquet.cli.commands.RewriteCommand;
-import org.apache.parquet.cli.commands.ScanCommand;
-import org.apache.parquet.cli.commands.SchemaCommand;
-import org.apache.parquet.cli.commands.ShowBloomFilterCommand;
-import org.apache.parquet.cli.commands.ShowColumnIndexCommand;
-import org.apache.parquet.cli.commands.ShowDictionaryCommand;
-import org.apache.parquet.cli.commands.ShowFooterCommand;
-import org.apache.parquet.cli.commands.ShowGeospatialStatisticsCommand;
-import org.apache.parquet.cli.commands.ShowPagesCommand;
-import org.apache.parquet.cli.commands.ShowSizeStatisticsCommand;
-import org.apache.parquet.cli.commands.ToAvroCommand;
-import org.apache.parquet.cli.commands.TransCompressionCommand;
+import org.apache.parquet.cli.commands.*;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 

…ache/parquet/cli/ShowVersionCommand.java …uet/cli/commands/ShowVersionCommand.javaparquet-cli/src/main/java/org/apache/parquet/cli/ShowVersionCommand.java renamed to parquet-cli/src/main/java/org/apache/parquet/cli/commands/ShowVersionCommand.java parquet-cli/src/main/java/org/apache/parquet/cli/ShowVersionCommand.java renamed to parquet-cli/src/main/java/org/apache/parquet/cli/commands/ShowVersionCommand.java

@@ -16,21 +16,20 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.parquet.cli;
+package org.apache.parquet.cli.commands;
 
-import com.beust.jcommander.*;
+import com.beust.jcommander.Parameters;
 import java.util.Collections;
 import java.util.List;
 import org.apache.parquet.Version;
+import org.apache.parquet.cli.BaseCommand;
 import org.slf4j.Logger;
 
 @Parameters(commandDescription = "Print version of the Parquet CLI tool")
-public class ShowVersionCommand implements Command {
-
-  private final Logger console;
+public class ShowVersionCommand extends BaseCommand {
 
   public ShowVersionCommand(Logger console) {
-    this.console = console;
+    super(console);
   }
 
   @Override

parquet-cli/src/test/java/org/apache/parquet/cli/commands/ShowVersionCommandTest.java

@@ -20,7 +20,6 @@
 
 import java.util.Queue;
 import org.apache.parquet.Version;
-import org.apache.parquet.cli.ShowVersionCommand;
 import org.junit.Assert;
 import org.junit.Test;
 import org.slf4j.Logger;