Improve `CommonSubexprEliminate` rule
Open
enhancement
- Dominant language
- Rust
- Stars
- 9.3k
- Forks
- 2.4k
- Avg merge
- 3d 7h
- Merged PRs (30d)
- 344
Description
### Is your feature request related to a problem or challenge?
https://github.com/apache/datafusion/pull/10473 improved the identifiers used in CSE but there are a few TODOs left in the code. Let's fix them.
### Describe the solution you'd like
_No response_
### Describe alternatives you've considered
_No response_
### Additional context
_No response_
Contributor guide
Research direction
Start by reviewing the CommonSubexprEliminate rule and Apache DataFusion pull request 10473, then locate the remaining TODOs referenced in the issue. Resolve those TODOs and verify that the rule's identifier improvements remain correct; the issue does not name specific files or tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- data-engineering
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100