amousavigourabi commented on code in PR #1141: URL: https://github.com/apache/parquet-mr/pull/1141#discussion_r1335214709
########## parquet-hadoop/src/main/java/org/apache/parquet/hadoop/InternalParquetRecordReader.java: ########## @@ -167,13 +169,13 @@ public float getProgress() throws IOException, InterruptedException { public void initialize(ParquetFileReader reader, ParquetReadOptions options) { // copy custom configuration to the Configuration passed to the ReadSupport - Configuration conf = new Configuration(); - if (options instanceof HadoopReadOptions) { - conf = ((HadoopReadOptions) options).getConf(); - } + ParquetConfiguration conf = Objects.requireNonNull(options).getConfiguration(); for (String property : options.getPropertyNames()) { conf.set(property, options.getProperty(property)); } + for (Map.Entry<String, String> property : new Configuration()) { Review Comment: The Hadoop specific stuff I agree that it would be a bit silly. However, this class `InternalParquetRecordReader` is part of the read/write API for which we're trying to address it. That the read/write API is part of parquet-hadoop is somewhat unfortunate, but changing it at this point would break stuff. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@parquet.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org