Github user cloud-fan commented on the issue:

    https://github.com/apache/spark/pull/21481
  
    +1 on @JoshRosen 's ideas.
    
    I've already seen several overflow fixes from @kiszk , it will be good if 
we have some tools to check it, even we need to run the tool manually. One idea 
may be to force to use java 8's safe math functions: 
https://docs.oracle.com/javase/8/docs/api/java/lang/Math.html#addExact-int-int-


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to