You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@parquet.apache.org by fo...@apache.org on 2025/08/08 12:53:12 UTC
(parquet-java) 01/01: minor: Cleanup some tests
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 <fo...@apache.org>
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());