Address spotless errors

Author: dossett

parquet-thrift/src/test/java/org/apache/parquet/hadoop/thrift/TestParquetToThriftReadWriteAndProjection.java

@@ -44,9 +44,6 @@
 import org.apache.parquet.thrift.test.RequiredPrimitiveFixture;
 import org.apache.parquet.thrift.test.RequiredSetFixture;
 import org.apache.parquet.thrift.test.StructWithReorderedOptionalFields;
-import org.apache.parquet.thrift.test.compat.MapWithPrimMapValue;
-import org.apache.parquet.thrift.test.compat.MapWithStructMapValue;
-import org.apache.parquet.thrift.test.compat.MapWithStructValue;
 import org.apache.parquet.thrift.test.compat.StructV3;
 import org.apache.parquet.thrift.test.compat.StructV4WithExtracStructField;
 import org.apache.thrift.TBase;

parquet-thrift/src/test/java/org/apache/parquet/thrift/TestParquetReadProtocol.java

@@ -45,7 +45,6 @@
 import org.apache.parquet.column.ParquetProperties;
 import org.apache.parquet.column.impl.ColumnWriteStoreV1;
 import org.apache.parquet.column.page.mem.MemPageStore;
-import org.apache.parquet.conf.ParquetConfiguration;
 import org.apache.parquet.io.ColumnIOFactory;
 import org.apache.parquet.io.MessageColumnIO;
 import org.apache.parquet.io.RecordReader;
@@ -166,7 +165,8 @@ public void testStructInMap() throws Exception {
     recordWriter.flush();
     columns.flush();
 
-    ThriftRecordConverter<T> converter = new TBaseRecordConverter<T>(thriftClass, schema, thriftType, new Configuration());
+    ThriftRecordConverter<T> converter =
+        new TBaseRecordConverter<T>(thriftClass, schema, thriftType, new Configuration());
     final RecordReader<T> recordReader = columnIO.getRecordReader(memPageStore, converter);
 
     final T result = recordReader.read();

parquet-thrift/src/test/java/org/apache/parquet/thrift/TestParquetWriteProtocol.java

@@ -177,7 +177,7 @@ public void testNameList() throws TException {
     names.add("John");
     names.add("Jack");
     final TestNameList o = new TestNameList("name", names);
-    
+
     String[] expectations = {
       "startMessage()",
       "startField(name, 0)",

parquet-thrift/src/test/java/org/apache/parquet/thrift/TestThriftParquetReaderWriter.java

@@ -28,7 +28,6 @@
 import org.apache.hadoop.fs.FileSystem;
 import org.apache.hadoop.fs.Path;
 import org.apache.parquet.hadoop.metadata.CompressionCodecName;
-import org.junit.Assert;
 import org.junit.Test;
 
 public class TestThriftParquetReaderWriter {

parquet-thrift/src/test/java/org/apache/parquet/thrift/TestThriftSchemaConverter.java

@@ -244,8 +244,7 @@ private void shouldThrowWhenNoColumnsAreSelected(String filters, Class<? extends
       getStrictFilteredSchema(filters, thriftClass);
       fail("this should throw");
     } catch (ThriftProjectionException e) {
-      assertEquals(
-          "No columns have been selected", e.getMessage());
+      assertEquals("No columns have been selected", e.getMessage());
     }
   }
 
@@ -256,13 +255,21 @@ public void testThrowWhenNoColumnsAreSelected() {
 
   @Test
   public void testThrowWhenProjectionFilterMatchesNothing() {
-    shouldThrowWhenProjectionFilterMatchesNothing("name;non_existing", "Pattern: 'non_existing' (when expanded to 'non_existing')", TestStructInMap.class);
-    shouldThrowWhenProjectionFilterMatchesNothing("**;non_existing", "Pattern: 'non_existing' (when expanded to 'non_existing')", TestStructInMap.class);
     shouldThrowWhenProjectionFilterMatchesNothing(
-        "**;names/non_existing", "Pattern: 'names/non_existing' (when expanded to 'names/non_existing')", TestStructInMap.class);
+        "name;non_existing",
+        "Pattern: 'non_existing' (when expanded to 'non_existing')",
+        TestStructInMap.class);
+    shouldThrowWhenProjectionFilterMatchesNothing(
+        "**;non_existing", "Pattern: 'non_existing' (when expanded to 'non_existing')", TestStructInMap.class);
     shouldThrowWhenProjectionFilterMatchesNothing(
-        "**;names/non_existing;non_existing", "Pattern: 'names/non_existing' (when expanded to 'names/non_existing')\n" +
-            "Pattern: 'non_existing' (when expanded to 'non_existing')", TestStructInMap.class);
+        "**;names/non_existing",
+        "Pattern: 'names/non_existing' (when expanded to 'names/non_existing')",
+        TestStructInMap.class);
+    shouldThrowWhenProjectionFilterMatchesNothing(
+        "**;names/non_existing;non_existing",
+        "Pattern: 'names/non_existing' (when expanded to 'names/non_existing')\n"
+            + "Pattern: 'non_existing' (when expanded to 'non_existing')",
+        TestStructInMap.class);
   }
 
   @Test