zhztheplayer commented on code in PR #10763:
URL:
https://github.com/apache/incubator-gluten/pull/10763#discussion_r2371787700
##########
gluten-core/src/main/scala/org/apache/gluten/extension/columnar/transition/package.scala:
##########
@@ -65,11 +65,11 @@ package object transition {
}
}
- // Extractor for Gluten's C2C
+ // Extractor for Gluten's C2C with different convention
object ColumnarToColumnarLike {
def unapply(plan: SparkPlan): Option[SparkPlan] = {
plan match {
- case c2c: ColumnarToColumnarTransition =>
+ case c2c: ColumnarToColumnarTransition if !c2c.isSameConvention =>
Review Comment:
Currently `VeloxResizeBatchesExec` is added by rule
https://github.com/apache/incubator-gluten/blob/cd2c0cca9b9478a050bfbb90f15e75f99e7adcd2/backends-velox/src/main/scala/org/apache/gluten/extension/AppendBatchResizeForShuffleInputAndOutput.scala#L31-L59
without costers involved. Am I missing something? How is it eliminated after
being added by the rule?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
