You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2019/07/23 09:33:06 UTC
[GitHub] [incubator-iceberg] electrum commented on issue #295: Normalize NaN
values for bucketing
electrum commented on issue #295: Normalize NaN values for bucketing
URL: https://github.com/apache/incubator-iceberg/issues/295#issuecomment-514132500
I just noticed that bucketing by float or double is not actually allowed by the specification, nor does the Iceberg library implement. However, the specification mentions how to hash these types, for seemingly no purpose. Would it be better to remove the hash specification to avoid confusion?
FWIW, I agree with the decision to not include bucketing on floating point numbers, since this will almost certainly cause problems and user confusion in practice.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org