You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "zanmato1984 (via GitHub)" <gi...@apache.org> on 2025/03/21 08:37:03 UTC
[I] [C++][Acero] Cleanup 64-bit temp states of Swiss join by using 32-bit [arrow]
zanmato1984 opened a new issue, #45877:
URL: https://github.com/apache/arrow/issues/45877
### Describe the enhancement requested
Since we have already addressed almost all potential overflow issues of Swiss join by #45336, #45515, we are now quite positive that:
1. The limit of number of rows in a Swiss table is `uint32_max`.
2. Any computation on "row id" that is possible of overflow has been properly handled to prevent overflow from happening.
It's time to finally cleanup the temp states remaining unnecessarily to be 64-bit to tolerant certain overflow, e.g. https://github.com/apache/arrow/pull/45336#discussion_r1926710909.
### Component(s)
C++
--
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: issues-unsubscribe@arrow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [C++][Acero] Cleanup 64-bit temp states of Swiss join by using 32-bit [arrow]
Posted by "zanmato1984 (via GitHub)" <gi...@apache.org>.
zanmato1984 closed issue #45877: [C++][Acero] Cleanup 64-bit temp states of Swiss join by using 32-bit
URL: https://github.com/apache/arrow/issues/45877
--
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: issues-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [C++][Acero] Cleanup 64-bit temp states of Swiss join by using 32-bit [arrow]
Posted by "zanmato1984 (via GitHub)" <gi...@apache.org>.
zanmato1984 commented on issue #45877:
URL: https://github.com/apache/arrow/issues/45877#issuecomment-2746914448
Issue resolved by pull request 45878
https://github.com/apache/arrow/pull/45878
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org