Skip to content

Commit

Permalink
feat: Add extended explain info to Comet plan
Browse files Browse the repository at this point in the history
  • Loading branch information
parthchandra committed Apr 17, 2024
1 parent 9321be6 commit 9456f6c
Show file tree
Hide file tree
Showing 13 changed files with 1,589 additions and 178 deletions.
2 changes: 2 additions & 0 deletions dev/ensure-jars-have-correct-contents.sh
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,8 @@ allowed_expr+="|^org/apache/spark/shuffle/sort/CometShuffleExternalSorter.*$"
allowed_expr+="|^org/apache/spark/shuffle/sort/RowPartition.class$"
allowed_expr+="|^org/apache/spark/shuffle/comet/.*$"
allowed_expr+="|^org/apache/spark/sql/$"
# allow ExplainPlanGenerator trait since it may not be available in older Spark versions
allowed_expr+="|^org/apache/spark/sql/ExtendedExplainGenerator.*$"
allowed_expr+="|^org/apache/spark/CometPlugin.class$"
allowed_expr+="|^org/apache/spark/CometDriverPlugin.*$"
allowed_expr+="|^org/apache/spark/CometTaskMemoryManager.class$"
Expand Down
3 changes: 3 additions & 0 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -927,6 +927,9 @@ under the License.
<ignoreClass>javax.annotation.meta.TypeQualifierValidator</ignoreClass>
<!-- this class is not properly excluded from comet-spark right now -->
<ignoreClass>org.apache.parquet.filter2.predicate.SparkFilterApi</ignoreClass>
<!-- we explicitly include a duplicate to allow older versions of Spark -->
<!-- this can be removed once we no longer support spark 3.x -->
<ignoreClass>org.apache.spark.sql.ExtendedExplainGenerator</ignoreClass>
</ignoreClasses>
</dependency>
<dependency>
Expand Down
Loading

0 comments on commit 9456f6c

Please sign in to comment.