Re: [PR] PARQUET-2249: Introduce IEEE 754 total order & NaN-counts [parquet-format]

["etseidl (via GitHub)" <gi...@apache.org>]

etseidl commented on PR #514:
URL: https://github.com/apache/parquet-format/pull/514#issuecomment-4188369174

   I've now verified the file @wgtmac produced, and verified that parquet-rs will produce the same statistics from the same input. I'd like to do some benchmarking early next week to see if there is any serious performance hit from this change. I hope to update the PoC for #221 to see if one approach is significantly faster than the other. Once that's done I think we'll be ready to finally put this issue to rest.


-- 
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: issues-unsubscribe@parquet.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@parquet.apache.org
For additional commands, e-mail: issues-help@parquet.apache.org