datafusion-contrib / datafusion-contrib/datafusion-distributed
implement `replace_children` for all `ExecutinoPlan` implementations
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 139
- Forks
- 67
- Avg merge
- 3d 1h
- Merged PRs (30d)
- 35
Description
See https://github.com/datafusion-contrib/datafusion-distributed/pull/540#discussion_r3810978939
with_new_children is being deprecated in favor of replace_children. We need to implement that for all our custom plan nodes.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Read the linked pull request discussion first, then search the repository for custom plan-node implementations and their with_new_children methods. Replace the deprecated method with replace_children across those implementations and verify that all custom nodes compile and preserve their existing child-replacement behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- distributed-systems
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 58/100