add a comment

apache · Jul 1, 2024 · c6eaae5 · c6eaae5
1 parent c55b7cc
commit c6eaae5
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/spark/src/main/scala/org/apache/comet/CometSparkSessionExtensions.scala b/spark/src/main/scala/org/apache/comet/CometSparkSessionExtensions.scala
@@ -207,6 +207,7 @@ class CometSparkSessionExtensions
       if (CometConf.COMET_CBO_ENABLED.get()) {
         // simple heuristic to avoid moving from Spark execution to Comet execution just
         // for the final sort
+        // in the future, we can make this check more generic and base it on actual costs
         cometPlan match {
           case CometSortExec(_, _, _, e: CometShuffleExchangeExec, _)
               if !e.child.supportsColumnar =>