kevinw66 commented on PR #8895: URL: https://github.com/apache/incubator-gluten/pull/8895#issuecomment-2698127021
> Thanks for your fix. We noticed here was core dump, but didn't find the root cause. This function origins from arrow https://github.com/apache/arrow-java/blob/main/adapter/orc/src/main/cpp/jni_wrapper.cpp#L83, maybe you also are interested on fixing it on arrow. @jinchengchenghh Thanks. However, I think the code in Apache Arrow should be fine. Since it allocates memory using vector, which is more intelligent in this regard. But considering that we don't use vector perhaps to improve performance, so I made the changes in our own way. -- 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]
