Skip to content

fix: implement replace_children on distributed plan nodes - #658

Closed
shinzoxD wants to merge 13 commits into
datafusion-contrib:branch-55from
shinzoxD:fix/replace-children-657
Closed

fix: implement replace_children on distributed plan nodes#658
shinzoxD wants to merge 13 commits into
datafusion-contrib:branch-55from
shinzoxD:fix/replace-children-657

Conversation

@shinzoxD

@shinzoxD shinzoxD commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Closes #657

Depends on #540 (DataFusion 55). with_new_children is deprecated in favor of replace_children.

Implement replace_children on our custom ExecutionPlan nodes. with_new_children still works and forwards ChildrenPropertiesMode::Recompute.

Nodes:

  • BroadcastExec, SamplerExec, DistributedLeafExec, MetricsWrapperExec, ChildrenIsolatorUnionExec
  • NetworkBroadcastExec, NetworkCoalesceExec, NetworkShuffleExec
  • DistributedExec, DistributedAnalyzeExec
  • test helpers: MockExec, URLEmitterExec, RowGeneratorExec

@shinzoxD shinzoxD changed the title tmp-permission-check fix: implement replace_children on distributed plan nodes Aug 19, 2026
@shinzoxD
shinzoxD changed the base branch from main to branch-55 August 19, 2026 19:38
@shinzoxD

Copy link
Copy Markdown
Contributor Author

Recreating as a non-draft PR targeting branch-55.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants