zhouyuan commented on PR #5033: URL: https://github.com/apache/incubator-gluten/pull/5033#issuecomment-2100043286
> hi @zhouyuan if there any blocker on this ? @ulysses-you hi Xiduo, we are actually breaking up this patch into smaller ones to get merged. still following on this. The first half is merged already however we do find some unit tests failed due to the new compile method. `(free(): invalid pointer:)` https://github.com/apache/incubator-gluten/pull/5515 We should fix the unit tests firstly, @JkSelf is working on this. For static building with arrow and parquet(the second half of this patch) it's working only local test. However it requires to modify the library loading code path - which is actually conflicting with the original dynamic packaging method(as we still need to load libarrow/libparquet there). We may need to keep the both code path for a while as static packaging is not verified in production env. thanks -yuan -- 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]
