asfimport commented on issue #406:
URL: https://github.com/apache/parquet-format/issues/406#issuecomment-2184154156

   [Jan 
Finis](https://issues.apache.org/jira/browse/PARQUET-2249?#comment-17691525) / 
@jfinis:
   @wgtmac my proposal would be backward compatible. It would only add optional 
fields, so that legacy readers not implementing the logic could still read the 
file and legacy writers could also still write files not using this without any 
code changes.
   
   @mapleFU I agree. Treating NaN as larger or smaller than any other value 
doesn't fit the semantics of all engines. Therefore, my fix for this would 
enable both types of semantics (and a third semantics, where NaN should be 
treated as neither larger nor smaller) to work.
   
   I guess I'll just create a pull request. We can then start the discussion 
based on this. Sounds good?


-- 
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]

Reply via email to