Checkstyle violations fixes

Author: pilardi

parquet-cli/src/main/java/org/apache/parquet/cli/commands/ConvertCommand.java

@@ -32,7 +32,6 @@
 import java.io.Closeable;
 import java.io.IOException;
 import java.util.List;
-
 import org.apache.avro.Schema;
 import org.apache.avro.generic.GenericData;
 import org.apache.hadoop.fs.FileSystem;

parquet-cli/src/main/java/org/apache/parquet/cli/commands/RewriteCommand.java

@@ -92,9 +92,7 @@ public RewriteCommand(Logger console) {
   }
 
   private RewriteOptions buildOptionsOrFail() throws IOException {
-    Preconditions.checkArgument(
-        inputs != null && !inputs.isEmpty(),
-        "Input parquet file paths are required.");
+    Preconditions.checkArgument(inputs != null && !inputs.isEmpty(), "Input parquet file paths are required.");
 
     // The builder below takes the job to validate all input parameters.
     RewriteOptions.Builder builder = createBuilder();

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

@@ -51,5 +51,4 @@ public void testConvertCommandToStdOut() throws IOException {
     Assert.assertEquals(0, command.run());
     Assert.assertTrue(baos.size() > 0);
   }
-
 }

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

@@ -111,5 +111,4 @@ public void testRewriteCommandToStdOut() throws IOException {
     Assert.assertEquals(0, command.run());
     Assert.assertTrue(baos.size() > 0);
   }
-
 }

parquet-common/src/main/java/org/apache/parquet/io/StandardOutputFile.java

@@ -26,61 +26,59 @@
  */
 public class StandardOutputFile implements OutputFile {
 
-    /**
-     * @implNote we don't want to close the standard output (it's up to the process lifecycle to handle that)
-     */
-    public class RawPositionOutputStream extends PositionOutputStream {
+  /**
+   * @implNote we don't want to close the standard output (it's up to the process lifecycle to handle that)
+   */
+  public class RawPositionOutputStream extends PositionOutputStream {
 
-        private long pos = 0;
-
-        @Override
-        public long getPos() throws IOException {
-            return this.pos;
-        }
-
-        @Override
-        public void write(int b) throws IOException {
-            this.pos++;
-            System.out.write(b);
-        }
-
-        @Override
-        public void write(byte[] b) throws IOException {
-            this.pos += b.length;
-            System.out.write(b);
-        }
-
-        @Override
-        public void write(byte[] b, int off, int len) throws IOException {
-            this.pos += len;
-            System.out.write(b, off, len);
-        }
-
-        @Override
-        public void flush() throws IOException {
-            System.out.flush();
-        }
+    private long pos = 0;
 
+    @Override
+    public long getPos() throws IOException {
+      return this.pos;
     }
 
     @Override
-    public PositionOutputStream create(long blockSizeHint) throws IOException {
-        return new RawPositionOutputStream();
+    public void write(int b) throws IOException {
+      this.pos++;
+      System.out.write(b);
     }
 
     @Override
-    public PositionOutputStream createOrOverwrite(long blockSizeHint) throws IOException {
-        return new RawPositionOutputStream();
+    public void write(byte[] b) throws IOException {
+      this.pos += b.length;
+      System.out.write(b);
     }
 
     @Override
-    public boolean supportsBlockSize() {
-        return false;
+    public void write(byte[] b, int off, int len) throws IOException {
+      this.pos += len;
+      System.out.write(b, off, len);
     }
 
     @Override
-    public long defaultBlockSize() {
-        return -1;
+    public void flush() throws IOException {
+      System.out.flush();
     }
-
+  }
+
+  @Override
+  public PositionOutputStream create(long blockSizeHint) throws IOException {
+    return new RawPositionOutputStream();
+  }
+
+  @Override
+  public PositionOutputStream createOrOverwrite(long blockSizeHint) throws IOException {
+    return new RawPositionOutputStream();
+  }
+
+  @Override
+  public boolean supportsBlockSize() {
+    return false;
+  }
+
+  @Override
+  public long defaultBlockSize() {
+    return -1;
+  }
 }

parquet-common/src/test/java/org/apache/parquet/io/TestStandardOutputFile.java

@@ -20,11 +20,10 @@
 
 import static org.junit.Assert.assertEquals;
 
-import org.junit.Test;
-
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.io.PrintStream;
+import org.junit.Test;
 
 public class TestStandardOutputFile {
 
@@ -65,5 +64,4 @@ public void outputFileWriteArrayOffsetToStdOut() throws IOException, Interrupted
     assertEquals(4, baos.toByteArray().length);
     assertEquals(TEST.substring(1, 5), new String(baos.toByteArray()));
   }
-
 }