You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/12/29 18:35:07 UTC

[GitHub] [arrow] Dandandan commented on pull request #9043: ARROW-11058: [Rust] [DataFusion] Implement coalesce batches operator

Dandandan commented on pull request #9043:
URL: https://github.com/apache/arrow/pull/9043#issuecomment-752197412


   LGTM 👍  😎 very nice speedup, good to see it also works on the bigger batch sizes (so it's not only because of the "problematic" join). I think it is a nice building block which we can decide to use in optimizations.
   We should still look at improving the join implementation, although this hides the problem quite a bit.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org