Skip to content

Pushdown integer upcasts to source operators in LocalPlanner#1354

Open
yingsu00 wants to merge 4 commits intoIBM:mainfrom
yingsu00:pushdownCastToSourceLocal
Open

Pushdown integer upcasts to source operators in LocalPlanner#1354
yingsu00 wants to merge 4 commits intoIBM:mainfrom
yingsu00:pushdownCastToSourceLocal

Conversation

@yingsu00
Copy link
Collaborator

PUshdown integer upcasts to source operators

Only when there are join nodes or result output nodes in the fragment that an integer upcast is needed. In this case, pushdown the upcasts to source operators like TableScanNode or ExchangeNode.

@prestodb-ci prestodb-ci force-pushed the main branch 15 times, most recently from a2ae60d to cb5caac Compare November 25, 2025 04:51
@yingsu00 yingsu00 force-pushed the pushdownCastToSourceLocal branch from 66ceb0a to b8a2817 Compare November 25, 2025 07:47
@prestodb-ci prestodb-ci force-pushed the main branch 5 times, most recently from b3ffe3f to a32ad3c Compare November 26, 2025 04:21
@yingsu00 yingsu00 force-pushed the pushdownCastToSourceLocal branch 6 times, most recently from 86894a7 to 380428a Compare December 2, 2025 08:12
@prestodb-ci prestodb-ci force-pushed the main branch 9 times, most recently from 9eadceb to 2e30c70 Compare December 17, 2025 07:41
@yingsu00 yingsu00 force-pushed the pushdownCastToSourceLocal branch 2 times, most recently from d8fee39 to b627700 Compare December 19, 2025 08:37
@prestodb-ci prestodb-ci force-pushed the main branch 15 times, most recently from 0e9fc69 to ae17272 Compare January 8, 2026 23:57
yingsu00 and others added 4 commits February 16, 2026 04:25
Changed pushdown_integer_upcasts_to_scan to pushdown_integer_upcasts_to_source

PUshdown integer upcasts to source operators

Only when there are join nodes or result output nodes in the fragment
that an integer upcast is needed. In this case, pushdown the upcasts to
source operators like TableScanNode or ExchangeNode.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants