LiuYC3213687107 commented on PR #26718: URL: https://github.com/apache/datafusion/pull/26718#issuecomment-6654701314
Thanks for catching the run-end encoded case! Fixed in 51cf54b: `is_float_zero` now looks inside `RunEndEncoded`, with your query in `topk.slt` (it returned `7, 1` before the fix) and the case in `is_float_zero_matches_both_zeros`. I also took the comment suggestion and added the parquet case, with `pushdown_filters` both off and on. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
