update

Author: ArnavBalyan

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

@@ -82,8 +82,7 @@ public class Main extends Configured implements Tool {
   @VisibleForTesting
   final JCommander jc;
 
-  @VisibleForTesting
-  public Main(Logger console) {
+  Main(Logger console) {
     this.console = console;
     this.jc = new JCommander(this);
     this.help = new Help(jc, console);

…parquet/cli/testing/CapturingLogger.java …/apache/parquet/cli/CapturingLogger.javaparquet-cli/src/test/java/org/apache/parquet/cli/testing/CapturingLogger.java renamed to parquet-cli/src/test/java/org/apache/parquet/cli/CapturingLogger.java parquet-cli/src/test/java/org/apache/parquet/cli/testing/CapturingLogger.java renamed to parquet-cli/src/test/java/org/apache/parquet/cli/CapturingLogger.java

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.parquet.cli.testing;
+package org.apache.parquet.cli;
 
 import org.slf4j.helpers.MarkerIgnoringBase;
 import org.slf4j.helpers.MessageFormatter;

…ache/parquet/cli/testing/CliHarness.java …a/org/apache/parquet/cli/CliHarness.javaparquet-cli/src/test/java/org/apache/parquet/cli/testing/CliHarness.java renamed to parquet-cli/src/test/java/org/apache/parquet/cli/CliHarness.java parquet-cli/src/test/java/org/apache/parquet/cli/testing/CliHarness.java renamed to parquet-cli/src/test/java/org/apache/parquet/cli/CliHarness.java

@@ -16,10 +16,9 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.parquet.cli.testing;
+package org.apache.parquet.cli;
 
 import org.apache.hadoop.conf.Configuration;
-import org.apache.parquet.cli.Main;
 import org.slf4j.Logger;
 
 public final class CliHarness {

…pache/parquet/cli/testing/CliResult.java …va/org/apache/parquet/cli/CliResult.javaparquet-cli/src/test/java/org/apache/parquet/cli/testing/CliResult.java renamed to parquet-cli/src/test/java/org/apache/parquet/cli/CliResult.java parquet-cli/src/test/java/org/apache/parquet/cli/testing/CliResult.java renamed to parquet-cli/src/test/java/org/apache/parquet/cli/CliResult.java

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.parquet.cli.testing;
+package org.apache.parquet.cli;
 
 import static org.junit.Assert.*;
 

…che/parquet/cli/testing/CliTestBase.java …/org/apache/parquet/cli/CliTestBase.javaparquet-cli/src/test/java/org/apache/parquet/cli/testing/CliTestBase.java renamed to parquet-cli/src/test/java/org/apache/parquet/cli/CliTestBase.java parquet-cli/src/test/java/org/apache/parquet/cli/testing/CliTestBase.java renamed to parquet-cli/src/test/java/org/apache/parquet/cli/CliTestBase.java

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.parquet.cli.testing;
+package org.apache.parquet.cli;
 
 import org.apache.parquet.cli.commands.ParquetFileTest;
 

parquet-cli/src/test/java/org/apache/parquet/cli/SchemaCliTest.java

@@ -19,7 +19,6 @@
 package org.apache.parquet.cli;
 
 import java.io.File;
-import org.apache.parquet.cli.testing.CliTestBase;
 import org.junit.Test;
 
 public class SchemaCliTest extends CliTestBase {

parquet-cli/src/test/java/org/apache/parquet/cli/ShowSizeStatisticsCliTest.java

@@ -33,7 +33,6 @@
 import java.util.Random;
 import org.apache.hadoop.conf.Configuration;
 import org.apache.hadoop.fs.Path;
-import org.apache.parquet.cli.testing.CliTestBase;
 import org.apache.parquet.example.data.simple.SimpleGroup;
 import org.apache.parquet.hadoop.ParquetWriter;
 import org.apache.parquet.hadoop.example.ExampleParquetWriter;

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

@@ -33,13 +33,13 @@
 
 public abstract class FileTest {
 
-  public static final String INT32_FIELD = "int32_field";
-  public static final String INT64_FIELD = "int64_field";
-  public static final String FLOAT_FIELD = "float_field";
-  public static final String DOUBLE_FIELD = "double_field";
-  public static final String BINARY_FIELD = "binary_field";
-  public static final String FIXED_LEN_BYTE_ARRAY_FIELD = "flba_field";
-  public static final String DATE_FIELD = "date_field";
+  static final String INT32_FIELD = "int32_field";
+  static final String INT64_FIELD = "int64_field";
+  static final String FLOAT_FIELD = "float_field";
+  static final String DOUBLE_FIELD = "double_field";
+  static final String BINARY_FIELD = "binary_field";
+  static final String FIXED_LEN_BYTE_ARRAY_FIELD = "flba_field";
+  static final String DATE_FIELD = "date_field";
 
   static final String[] COLORS = {"RED", "BLUE", "YELLOW", "GREEN", "WHITE"};
 

parquet-cli/src/test/resources/cli-outputs/size-stats.txt

@@ -1,12 +1,13 @@
 
+
 Row group 0
 --------------------------------------------------------------------------------
-column        unencoded bytes rep level histogram                      def level histogram
-[DocId]       -               -                                        -
-[CategoryId]  -               -                                        -
-[IsActive]    -               -                                        -
-[Prices]      -               [10000, 7425]                            -
-[Tags]        152.405 kB      [10000, 11931]                           -
-[ProductName] 156.250 kB      -                                        -
-[Description] 1.170 MB        -                                        -
-[UUID]        -               -                                        -
+column        unencoded bytes rep level histogram                      def level histogram                     
+[DocId]       -               -                                        -                                       
+[CategoryId]  -               -                                        -                                       
+[IsActive]    -               -                                        -                                       
+[Prices]      -               [10000, 7425]                            -                                       
+[Tags]        152.405 kB      [10000, 11931]                           -                                       
+[ProductName] 156.250 kB      -                                        -                                       
+[Description] 1.170 MB        -                                        -                                       
+[UUID]        -               -                                        -