[VL] RAS: Minor code cleanup for offloading project (#6452)

apache · Jul 16, 2024 · 90e0087 · 90e0087
1 parent 7bf1be2
commit 90e0087
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git a/.../src/main/scala/org/apache/gluten/extension/columnar/enumerated/EnumeratedTransform.scala b/.../src/main/scala/org/apache/gluten/extension/columnar/enumerated/EnumeratedTransform.scala
@@ -63,7 +63,6 @@ case class EnumeratedTransform(session: SparkSession, outputsColumnar: Boolean)
         OffloadOthers())
       .toRule,
     RasOffload.from[CoalesceExec](OffloadOthers()).toRule,
-    RasOffload.from[ProjectExec](OffloadOthers()).toRule,
     RasOffload.from[SortAggregateExec](OffloadOthers()).toRule,
     RasOffload.from[ObjectHashAggregateExec](OffloadOthers()).toRule,
     RasOffload.from[UnionExec](OffloadOthers()).toRule,

diff --git a/...re/src/main/scala/org/apache/gluten/extension/columnar/enumerated/RasOffloadProject.scala b/...re/src/main/scala/org/apache/gluten/extension/columnar/enumerated/RasOffloadProject.scala
@@ -20,6 +20,7 @@ import org.apache.gluten.execution.ProjectExecTransformer
 
 import org.apache.spark.sql.execution.{ProjectExec, SparkPlan}
 
+/** TODO: Map [[org.apache.gluten.extension.columnar.OffloadProject]] to RAS. */
 object RasOffloadProject extends RasOffload {
   override def offload(node: SparkPlan): SparkPlan = node match {
     case ProjectExec(projectList, child) =>