Avro: Fix test configuration (#3244)

zheguang · web-flow · commit 46595e286d4e · 2025-06-25T19:36:09.000+02:00
diff --git a/parquet-avro/src/test/java/org/apache/parquet/avro/TestReadWrite.java b/parquet-avro/src/test/java/org/apache/parquet/avro/TestReadWrite.java
@@ -962,9 +962,9 @@ private ParquetWriter<GenericRecord> writer(String file, Schema schema) throws I
           writerBuilder.withCodecFactory(HadoopCodecs.newFactory(ParquetProperties.DEFAULT_PAGE_SIZE));
     }
     if (conf == ConfigurationType.PLAIN_PARQUET_INTERFACE) {
-      return writerBuilder.withConf(hadoopConfWithInterface).build();
-    } else if (conf == ConfigurationType.HADOOP_PARQUET_INTERFACE) {
       return writerBuilder.withConf(plainParquetConf).build();
+    } else if (conf == ConfigurationType.HADOOP_PARQUET_INTERFACE) {
+      return writerBuilder.withConf(hadoopConfWithInterface).build();
     } else {
       return writerBuilder.withConf(testConf).build();
     }
@@ -983,9 +983,9 @@ private ParquetReader<GenericRecord> reader(String file) throws IOException {
           readerBuilder.withCodecFactory(HadoopCodecs.newFactory(ParquetProperties.DEFAULT_PAGE_SIZE));
     }
     if (conf == ConfigurationType.PLAIN_PARQUET_INTERFACE) {
-      return readerBuilder.withConf(hadoopConfWithInterface).build();
-    } else if (conf == ConfigurationType.HADOOP_PARQUET_INTERFACE) {
       return readerBuilder.withConf(plainParquetConf).build();
+    } else if (conf == ConfigurationType.HADOOP_PARQUET_INTERFACE) {
+      return readerBuilder.withConf(hadoopConfWithInterface).build();
     } else {
       return readerBuilder.withConf(testConf).build();
     }

Original file line number	Diff line number	Diff line change
`@@ -962,9 +962,9 @@ private ParquetWriter<GenericRecord> writer(String file, Schema schema) throws I`
`962`	`962`	`writerBuilder.withCodecFactory(HadoopCodecs.newFactory(ParquetProperties.DEFAULT_PAGE_SIZE));`
`963`	`963`	`}`
`964`	`964`	`if (conf == ConfigurationType.PLAIN_PARQUET_INTERFACE) {`
`965`		`- return writerBuilder.withConf(hadoopConfWithInterface).build();`
`966`		`- } else if (conf == ConfigurationType.HADOOP_PARQUET_INTERFACE) {`
`967`	`965`	`return writerBuilder.withConf(plainParquetConf).build();`
	`966`	`+ } else if (conf == ConfigurationType.HADOOP_PARQUET_INTERFACE) {`
	`967`	`+ return writerBuilder.withConf(hadoopConfWithInterface).build();`
`968`	`968`	`} else {`
`969`	`969`	`return writerBuilder.withConf(testConf).build();`
`970`	`970`	`}`
`@@ -983,9 +983,9 @@ private ParquetReader<GenericRecord> reader(String file) throws IOException {`
`983`	`983`	`readerBuilder.withCodecFactory(HadoopCodecs.newFactory(ParquetProperties.DEFAULT_PAGE_SIZE));`
`984`	`984`	`}`
`985`	`985`	`if (conf == ConfigurationType.PLAIN_PARQUET_INTERFACE) {`
`986`		`- return readerBuilder.withConf(hadoopConfWithInterface).build();`
`987`		`- } else if (conf == ConfigurationType.HADOOP_PARQUET_INTERFACE) {`
`988`	`986`	`return readerBuilder.withConf(plainParquetConf).build();`
	`987`	`+ } else if (conf == ConfigurationType.HADOOP_PARQUET_INTERFACE) {`
	`988`	`+ return readerBuilder.withConf(hadoopConfWithInterface).build();`
`989`	`989`	`} else {`
`990`	`990`	`return readerBuilder.withConf(testConf).build();`
`991`	`991`	`}`