Github user imatiach-msft commented on the issue:

    https://github.com/apache/spark/pull/17059
  
    @datumbox I like the changes, I just had a minor concern about the code 
where we call v.intValue and then compare this to v.doubleValue -- due to 
precision issues, I'm not sure if this is desirable, since the data could come 
from any source and be slightly modified outside the Int range, and the 
previous code does not do this check


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to