RussellSpitzer commented on issue #2405:
URL: https://github.com/apache/iceberg/issues/2405#issuecomment-826359240
I think I have this fixed in #2495
Sent from my iPhone
> On Apr 25, 2021, at 3:24 AM, Chen, Junjie ***@***.***> wrote:
>
>
> Still hit this problem. Now I know how to reproduce this.
>
> Write the iceberg table with the 0.10.0 version.
> Read the iceberg table manifests with the 0.11.1 version.
> The following code could fix the issue:
>
> ---
a/core/src/main/java/org/apache/iceberg/GenericPartitionFieldSummary.java
> +++
b/core/src/main/java/org/apache/iceberg/GenericPartitionFieldSummary.java
> @@ -121,7 +121,7 @@ public class GenericPartitionFieldSummary
>
> @Override
> public Boolean containsNaN() {
> - return containsNaN;
> + return containsNaN == null || containsNull;
> }
> @RussellSpitzer @yyanyy, We may need to re-consider this with #2492 again.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or unsubscribe.
--
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
