You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@spark.apache.org by "Apache Spark (Jira)" <ji...@apache.org> on 2021/07/30 17:28:00 UTC
[jira] [Commented] (SPARK-36362) Omnibus Java code static analyzer
warning fixes
[ https://issues.apache.org/jira/browse/SPARK-36362?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17390705#comment-17390705 ]
Apache Spark commented on SPARK-36362:
--------------------------------------
User 'srowen' has created a pull request for this issue:
https://github.com/apache/spark/pull/33594
> Omnibus Java code static analyzer warning fixes
> -----------------------------------------------
>
> Key: SPARK-36362
> URL: https://issues.apache.org/jira/browse/SPARK-36362
> Project: Spark
> Issue Type: Improvement
> Components: Spark Core, SQL, Tests
> Affects Versions: 3.2.0
> Reporter: Sean R. Owen
> Assignee: Sean R. Owen
> Priority: Minor
>
> Inspired by a recent Java code touch-up, I wanted to fix in one pass several lingering non-trivial issues with the Java code that a static analyzer turns up. Only a few of these have material effects, but some do, and figured we could avoid taking N PRs over time to address these.
> * Some int*int multiplications that widen to long maybe could overflow
> * Unnecessarily non-static inner classes
> * Some tests "catch (AssertionError)" and do nothing
> * Manual array iteration vs very slightly faster/simpler foreach
> * Incorrect generic types that just happen to not cause a runtime error
> * Missed opportunities for try-close
> * Mutable enums which shouldn't be
> * .. and a few other minor things
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@spark.apache.org
For additional commands, e-mail: issues-help@spark.apache.org