print file path in the warning

apache · Jan 11, 2023 · f3ba6e8 · f3ba6e8
1 parent a496c31
commit f3ba6e8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/parquet-hadoop/src/main/java/org/apache/parquet/hadoop/ParquetFileReader.java b/parquet-hadoop/src/main/java/org/apache/parquet/hadoop/ParquetFileReader.java
@@ -929,7 +929,7 @@ public PageReadStore readNextRowGroup() throws IOException {
     try {
       rowGroup = internalReadRowGroup(currentBlock);
     } catch (ParquetEmptyBlockException e) {
-      LOG.warn("Read empty block at index {}", currentBlock);
+      LOG.warn("Read empty block at index {} from {}", currentBlock, getFile());
       advanceToNextBlock();
       return readNextRowGroup();
     }
@@ -1044,7 +1044,7 @@ public PageReadStore readNextFilteredRowGroup() throws IOException {
     }
     BlockMetaData block = blocks.get(currentBlock);
     if (block.getRowCount() == 0L) {
-      LOG.warn("Read empty block at index {}", currentBlock);
+      LOG.warn("Read empty block at index {} from {}", currentBlock, getFile());
       // Skip the empty block
       advanceToNextBlock();
       return readNextFilteredRowGroup();

diff --git a/pom.xml b/pom.xml
@@ -89,7 +89,7 @@
     <pig.version>0.16.0</pig.version>
     <pig.classifier>h2</pig.classifier>
     <thrift-maven-plugin.version>0.10.0</thrift-maven-plugin.version>
-    <thrift.version>0.16.0</thrift.version>
+    <thrift.version>0.17.0</thrift.version>
     <format.thrift.version>${thrift.version}</format.thrift.version>
     <fastutil.version>8.4.2</fastutil.version>
     <semver.api.version>0.9.33</semver.api.version>
@@ -424,7 +424,7 @@
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-surefire-plugin</artifactId>
         <configuration>
-          <argLine>${surefire.argLine} ${extraJavaTestArgs}</argLine>
+          <argLine>${extraJavaTestArgs}</argLine>
           <systemPropertyVariables>
             <!-- Configure Parquet logging during tests
                  See http://www.slf4j.org/api/org/slf4j/impl/SimpleLogger.html