[GitHub] [arrow] andygrove commented on pull request #8029: ARROW-9464: [Rust] [DataFusion] Remove Partition trait

2020-08-23 Thread GitBox
andygrove commented on pull request #8029: URL: https://github.com/apache/arrow/pull/8029#issuecomment-678784408 > This is an impressive simplification and improvement. Really great work, @andygrove ! > > I went through it and could not find any issue with it, only benefits. >

[GitHub] [arrow] andygrove commented on pull request #8029: ARROW-9464: [Rust] [DataFusion] Remove Partition trait

2020-08-22 Thread GitBox
andygrove commented on pull request #8029: URL: https://github.com/apache/arrow/pull/8029#issuecomment-678728434 Yes, exactly. We could do more efficient things in the future such as perform sorts in parallel and then do a sort-merge join on the results. On Sat, Aug 22, 2020,

[GitHub] [arrow] andygrove commented on pull request #8029: ARROW-9464: [Rust] [DataFusion] Remove Partition trait

2020-08-22 Thread GitBox
andygrove commented on pull request #8029: URL: https://github.com/apache/arrow/pull/8029#issuecomment-678704604 @jorgecarleitao @alamb Sorry, this is a bit of a larger change than usual. This is an automated message from