mapleFU commented on PR #55323: URL: https://github.com/apache/arrow/pull/55323#issuecomment-6634880823
> What is the unit of the numbers? GreaterThan output a uint64_t word, it accepts 1-64 rows and generate the word to write the bitmap? so I use `n` as input count here > I don't really understand what you're proposing, but the PR looks fine to me. Well, I mean should I leaving the original avx2 implementation of cpp/src/parquet/level_comparison_avx2_internal.h (it was deleted in this pull request), or just leaving it here, until the xsimd optimization is merged and released -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
