PARQUET-686: Add getConfiguration to HadoopInputFile.

This is used to avoiding extra public methods in ParquetFileReader.

Author: rdblue

parquet-hadoop/src/main/java/org/apache/parquet/hadoop/ParquetFileReader.java

@@ -52,6 +52,7 @@
 import java.util.concurrent.Executors;
 import java.util.concurrent.Future;
 
+import org.apache.hadoop.conf.Configurable;
 import org.apache.hadoop.conf.Configuration;
 import org.apache.hadoop.fs.FileStatus;
 import org.apache.hadoop.fs.FileSystem;
@@ -444,9 +445,16 @@ public static final ParquetMetadata readFooter(Configuration configuration, File
    */
   public static final ParquetMetadata readFooter(
       InputFile file, MetadataFilter filter) throws IOException {
+    ParquetMetadataConverter converter;
+    if (file instanceof HadoopInputFile) {
+      converter = new ParquetMetadataConverter(
+          ((HadoopInputFile) file).getConfiguration());
+    } else {
+      converter = new ParquetMetadataConverter();
+    }
     try (SeekableInputStream in = file.newStream()) {
-      return readFooter(new ParquetMetadataConverter(), file.getLength(),
-          file.toString(), in, filter);
+
+      return readFooter(converter, file.getLength(), file.toString(), in, filter);
     }
   }
 

parquet-hadoop/src/main/java/org/apache/parquet/hadoop/util/HadoopInputFile.java

@@ -31,22 +31,28 @@ public class HadoopInputFile implements InputFile {
 
   private final FileSystem fs;
   private final FileStatus stat;
+  private final Configuration conf;
 
   public static HadoopInputFile fromPath(Path path, Configuration conf)
       throws IOException {
     FileSystem fs = path.getFileSystem(conf);
-    return new HadoopInputFile(fs, fs.getFileStatus(path));
+    return new HadoopInputFile(fs, fs.getFileStatus(path), conf);
   }
 
   public static HadoopInputFile fromStatus(FileStatus stat, Configuration conf)
       throws IOException {
     FileSystem fs = stat.getPath().getFileSystem(conf);
-    return new HadoopInputFile(fs, stat);
+    return new HadoopInputFile(fs, stat, conf);
   }
 
-  private HadoopInputFile(FileSystem fs, FileStatus stat) {
+  private HadoopInputFile(FileSystem fs, FileStatus stat, Configuration conf) {
     this.fs = fs;
     this.stat = stat;
+    this.conf = conf;
+  }
+
+  public Configuration getConfiguration() {
+    return conf;
   }
 
   @Override