mbutrovich commented on PR #3519: URL: https://github.com/apache/datafusion-comet/pull/3519#issuecomment-3909336457
Interesting PR, thanks @Shekharrajak! To help start the review process, could you: 1. Provide a high level architecture diagram? 2. Explain where the performance benefit comes from, and why is it so much faster to pass batches over this JNI interface than the existing interface? 3. As part of (1), maybe we can find a way to break this down into smaller PRs. -- 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]
