Skip to content

Remove the "Exec" suffix for physical operators#1349

Merged
richox merged 1 commit intoapache:masterfrom
XorSum:nodename-remove-exec
Sep 27, 2025
Merged

Remove the "Exec" suffix for physical operators#1349
richox merged 1 commit intoapache:masterfrom
XorSum:nodename-remove-exec

Conversation

@XorSum
Copy link
Contributor

@XorSum XorSum commented Sep 25, 2025

Which issue does this PR close?

Closes #.

Rationale for this change

https://github.com/apache/spark/blob/v4.0.1/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala#L837

  /**
   * Returns the name of this type of TreeNode.  Defaults to the class name.
   * Note that we remove the "Exec" suffix for physical operators here.
   */
  def nodeName: String = simpleClassName.replaceAll("Exec$", "")

What changes are included in this PR?

Are there any user-facing changes?

How was this patch tested?

@github-actions github-actions bot added the spark label Sep 25, 2025
@richox richox merged commit 268e852 into apache:master Sep 27, 2025
226 of 228 checks passed
Tartarus0zm pushed a commit to Tartarus0zm/auron that referenced this pull request Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants