Skip to content

Commit

Permalink
Improve & unify validation in LogicalPlan::with_new_exprs
Browse files Browse the repository at this point in the history
When adding new plan node type, `LogicalPlan::with_new_exprs` needs to
be updated. Different code branches apply different inputs validation
style (no validation, just assert, or assert with messages), so it's
unclear which code pattern to follow.  This commit unifies the
validation and adds it to the branches where there was none.
  • Loading branch information
findepi committed Aug 30, 2024
1 parent e603185 commit e4c5a92
Showing 1 changed file with 185 additions and 87 deletions.
Loading

0 comments on commit e4c5a92

Please sign in to comment.