emilie-wang opened a new pull request, #526:
URL: https://github.com/apache/incubator-xtable/pull/526

   ## *Important Read*
   
   ## What is the purpose of the pull request
   
   The pr aims to fix: https://github.com/apache/incubator-xtable/issues/524.
   When reading the delta snapshot and load the information into Delta object 
AddFile, the non-numeric values of float or double type (example, "NaN", 
"-Infinity") from col stats become string type. These special values need 
special handling and this pr used the same idea how delta new API handled this: 
https://github.com/delta-io/delta/blob/master/kernel/kernel-defaults/src/main/java/io/delta/kernel/defaults/internal/data/DefaultJsonRow.java#L210
   ## Brief change log
   - Add logic to handle special String value ("NaN", "-Infinity", "Infinity") 
when column type is Double or Float during translating Delta column stat to 
InternalType 
   - Add unit test
   
   ## Verify this pull request
   
   This change added tests and can be verified as follows:
   
   - Added unit test for this edge case
   


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

Reply via email to