zhztheplayer opened a new pull request, #5305:
URL: https://github.com/apache/incubator-gluten/pull/5305
Explanation on terms:
1. cluster-canonical node:
A node with all children replaced by cluster key.
2. group-canonical node:
A node with all children replaced by group id.
After the patch, memo will use 1 as the cache key to look up resident
cluster for new node.
This will make the memo size and memo cache smaller during optimization
since we always create new cluster if cache was not hit when adding node to
memo.
Consider two plans are inserted to memo respectively:
```
Project(col1, col2)
\- Group(cluster0, ordering by col1)
```
```
Project(col1, col2)
\- Group(cluster0, ordering by col2)
```
Previously the second insertion would create a new cluster since `ordering
by col2` is considered in cache key. After the patch the second insertion will
no longer create new cluster.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
