yangzhg commented on PR #5326: URL: https://github.com/apache/incubator-gluten/pull/5326#issuecomment-2044079819
> Thanks. Does this cause overflow in real cases? The uint16_t is used on purpose to benefit L2 cache, and is not supposed to cause the overflow with the 32k limitation of the batch size. Please check [#4401 (comment)](https://github.com/apache/incubator-gluten/pull/4401#issuecomment-1895360554) > > cc: @FelixYBW Indeed, using uint16_t in this scenario can lead to overflow. The specific statements and conditions for reproducing this issue have already been outlined in issue #5307, and I will be providing a dataset shortly to further illustrate this point. -- 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]
