[GitHub] [arrow] nevi-me commented on pull request #7037: ARROW-6718: [DRAFT] [Rust] Remove packed_simd

2020-05-06 Thread GitBox
nevi-me commented on pull request #7037: URL: https://github.com/apache/arrow/pull/7037#issuecomment-624877425 I'm happy with turning SIMD off by default. @paddyhoran may you please port the `divide` changes that I made here to #7057 (with improvement if you see any scope)? -

[GitHub] [arrow] nevi-me commented on pull request #7037: ARROW-6718: [DRAFT] [Rust] Remove packed_simd

2020-05-02 Thread GitBox
nevi-me commented on pull request #7037: URL: https://github.com/apache/arrow/pull/7037#issuecomment-622931053 That's interesting, and how are other benchmarks affected, or are you only focusing on filter? To the extent that there's overlap with `take()`, it would be interesting to also se