This is an automated email from the ASF dual-hosted git repository. fokko pushed a commit to branch fd-minor-nits in repository https://gitbox.apache.org/repos/asf/parquet-java.git
commit 03edbf9e99e07c5870979ddcac0daad7f054d693 Author: Fokko Driesprong <[email protected]> AuthorDate: Fri Aug 8 14:53:04 2025 +0200 minor: Cleanup some tests --- .../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());
