Zouxxyy commented on code in PR #10763:
URL:
https://github.com/apache/incubator-gluten/pull/10763#discussion_r2374572634
##########
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:
It seems that the current c2c is automatically added, and resize is manually
added, so we should not remove it. and whether velox resize should also be
included in the cost
--
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]
