Addressed all comments

apache · Oct 28, 2021 · d85d4ba · d85d4ba
1 parent 8c7c617
commit d85d4ba
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/...ore/src/main/java/org/apache/spark/sql/execution/datasources/orc/OrcColumnStatistics.java b/...ore/src/main/java/org/apache/spark/sql/execution/datasources/orc/OrcColumnStatistics.java
@@ -31,7 +31,7 @@
  * tree pre-ordering. This is used for aggregate push down in ORC.
  *
  * For nested data types (array, map and struct), the sub-field statistics are stored recursively
- * inside parent column's `children` field. Here is an example of `OrcColumnStatistics`:
+ * inside parent column's children field. Here is an example of {@link OrcColumnStatistics}:
  *
  * Data schema:
  * c1: int

diff --git a/sql/core/src/main/java/org/apache/spark/sql/execution/datasources/orc/OrcFooterReader.java b/sql/core/src/main/java/org/apache/spark/sql/execution/datasources/orc/OrcFooterReader.java
@@ -27,7 +27,7 @@
 import java.util.Queue;
 
 /**
- * `OrcFooterReader` is a util class which encapsulates the helper
+ * {@link OrcFooterReader} is a util class which encapsulates the helper
  * methods of reading ORC file footer.
  */
 public class OrcFooterReader {

diff --git a/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcUtils.scala b/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcUtils.scala
@@ -402,7 +402,7 @@ object OrcUtils extends Logging {
     var columnsStatistics: OrcColumnStatistics = null
     try {
       columnsStatistics = OrcFooterReader.readStatistics(reader)
-    } catch { case e: RuntimeException =>
+    } catch { case e: Exception =>
       throw new SparkException(
         s"Cannot read columns statistics in file: $filePath. Please consider disabling " +
         s"ORC aggregate push down by setting 'spark.sql.orc.aggregatePushdown' to false.", e)