(parquet-java) branch master updated: Minor: Cleanup some tests (#3265)

[gangwu]

This is an automated email from the ASF dual-hosted git repository.

gangwu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/parquet-java.git


The following commit(s) were added to refs/heads/master by this push:
     new 6d2635b2a Minor: Cleanup some tests (#3265)
6d2635b2a is described below

commit 6d2635b2a4d4316816c29602ce9e3f770157b047
Author: Fokko Driesprong <fo...@apache.org>
AuthorDate: Thu Aug 21 07:26:14 2025 +0200

    Minor: Cleanup some tests (#3265)
---
 .../java/org/apache/parquet/hadoop/TestParquetFileWriter.java     | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git 
a/parquet-hadoop/src/test/java/org/apache/parquet/hadoop/TestParquetFileWriter.java
 
b/parquet-hadoop/src/test/java/org/apache/parquet/hadoop/TestParquetFileWriter.java
index 3126e1746..4269cf516 100644
--- 
a/parquet-hadoop/src/test/java/org/apache/parquet/hadoop/TestParquetFileWriter.java
+++ 
b/parquet-hadoop/src/test/java/org/apache/parquet/hadoop/TestParquetFileWriter.java
@@ -132,7 +132,7 @@ public class TestParquetFileWriter {
 
   private static final Logger LOG = 
LoggerFactory.getLogger(TestParquetFileWriter.class);
 
-  private static final MessageType SCHEMA = 
MessageTypeParser.parseMessageType("" + "message m {"
+  private static final MessageType SCHEMA = 
MessageTypeParser.parseMessageType("message m {"
       + "  required group a {"
       + "    required binary b;"
       + "  }"
@@ -156,8 +156,6 @@ public class TestParquetFileWriter {
               Types.required(PrimitiveTypeName.BINARY).named("test_binary"))
           .build();
 
-  private String writeSchema;
-
   @Rule
   public final TemporaryFolder temp = new TemporaryFolder();
 
@@ -1032,7 +1030,7 @@ public class TestParquetFileWriter {
     File testFile = temp.newFile();
     testFile.delete();
 
-    writeSchema = "message example {\n" + "required binary content (UTF8);\n" 
+ "}";
+    String writeSchema = "message example {\n" + "required binary content 
(UTF8);\n" + "}";
 
     Path path = new Path(testFile.toURI());
 
@@ -1041,7 +1039,7 @@ public class TestParquetFileWriter {
     configuration.setBoolean("parquet.strings.signed-min-max.enabled", true);
     GroupWriteSupport.setSchema(schema, configuration);
 
-    // close any filesystems to ensure that the the FS used by the writer 
picks up the configuration
+    // close any filesystems to ensure that the FS used by the writer picks up 
the configuration
     FileSystem.closeAll();
     ParquetWriter<Group> writer = new ParquetWriter<Group>(path, 
configuration, new GroupWriteSupport());