[GitHub] [arrow] houqp commented on pull request #7666: ARROW-8559: [Rust] Consolidate Record Batch reader traits in main arrow crate

2020-07-11 Thread GitBox
houqp commented on pull request #7666: URL: https://github.com/apache/arrow/pull/7666#issuecomment-657125605 linter error will be addressed by https://github.com/apache/arrow/pull/7710 This is an automated message from the

[GitHub] [arrow] houqp commented on pull request #7666: ARROW-8559: [Rust] Consolidate Record Batch reader traits in main arrow crate

2020-07-10 Thread GitBox
houqp commented on pull request #7666: URL: https://github.com/apache/arrow/pull/7666#issuecomment-656943172 +1 on unifying to Arc for the initial release given it's not on the hot path. It's better to start simple and optimize later. Arc's atomic reference counting tend to mess with