cloud-fan commented on code in PR #58419:
URL: https://github.com/apache/spark/pull/58419#discussion_r4000514085
##########
sql/core/src/main/scala/org/apache/spark/sql/execution/basicPhysicalOperators.scala:
##########
@@ -1284,6 +1351,26 @@ case class UnionExec(children: Seq[SparkPlan]) extends
SparkPlan with CodegenSup
}
object UnionExec {
+ /**
+ * The latched "is this a plain concatenation" decision. See `isPlainUnion`.
+ *
+ * Rebuilds carry it: `withNewChildren` and a transform rule's replacement
both go through
Review Comment:
Confirmed: the DECISIONS comment now qualifies tag inheritance to fresh
targets with no existing tags and notes that already-tagged replacements are
not overwritten. Resolved.
<!-- SPARK_DEV_REVIEW_REPLY
{"feedback_id":"inline:3993643938","thread_id":"inline:3993643938","verdict_sha256":"3f145606514841e98aa4f828110c8e9a2ef2a823fe82b8c1858abbafd71ce3c8"}
-->
--
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]
