[GitHub] [arrow] nevi-me commented on pull request #7319: ARROW-8289: [Rust] Parquet Arrow writer with nested support

2020-08-13 Thread GitBox
nevi-me commented on pull request #7319: URL: https://github.com/apache/arrow/pull/7319#issuecomment-673589031 Merged as https://github.com/apache/arrow/commit/80a9c027b7c356f25a4c22e71587936a54959db6, not sure why the merge tool didn't close the issue

[GitHub] [arrow] nevi-me commented on pull request #7319: ARROW-8289: [Rust] Parquet Arrow writer with nested support

2020-08-08 Thread GitBox
nevi-me commented on pull request #7319: URL: https://github.com/apache/arrow/pull/7319#issuecomment-670899165 @sunchao @andygrove (CC @wesm @kszucs @emkornfield) in the past few months we haven't had enough review bandwidth on Rust's Parquet implementation (mostly relying on Chao for

[GitHub] [arrow] nevi-me commented on pull request #7319: ARROW-8289: [Rust] Parquet Arrow writer with nested support

2020-08-08 Thread GitBox
nevi-me commented on pull request #7319: URL: https://github.com/apache/arrow/pull/7319#issuecomment-670878573 > ... One thing to note which is currently a bug in C++ is once rep/def levels are computed for any anything > with deep nesting (any leaf column one or more direct

[GitHub] [arrow] nevi-me commented on pull request #7319: ARROW-8289: [Rust] Parquet Arrow writer with nested support

2020-08-05 Thread GitBox
nevi-me commented on pull request #7319: URL: https://github.com/apache/arrow/pull/7319#issuecomment-669128126 @maxburke there's been some interest from other people on this PR. I haven't been able to continue working on it because where I have a bit of free time I've been looking at the