Skip to content

Commit

Permalink
Use shorter aliases in CSE (apache#10939)
Browse files Browse the repository at this point in the history
* initial change

* test renaming

* use counter instead of indexmap

* order slt tests

* change cse tests

* restore slt tests

* fix slt test

* formatting

* ensure no alias collision

* keep original alias numbers for collision

* ensure no collision in aggregate cse

* use `AliasGenerator` to generate aliases, use `__cse` prefix in common expression aliases, remove `DataType` from `ExprStats` as not needed, store aliases in `CommonExprs`, revert unnecessary changes

* use `into_values()` instead of `into_iter()` where possible

* fix docstring of `ExprStats` and `CommonExprs`

* use `__common_expr` prefix

---------

Co-authored-by: Mohamed Abdeen <[email protected]>
  • Loading branch information
2 people authored and findepi committed Jul 16, 2024
1 parent a4a9a7a commit 2db75de
Show file tree
Hide file tree
Showing 6 changed files with 271 additions and 261 deletions.
Loading

0 comments on commit 2db75de

Please sign in to comment.