[
https://issues.apache.org/jira/browse/SPARK-23769?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Hyukjin Kwon resolved SPARK-23769.
----------------------------------
Resolution: Fixed
Assignee: Riaas Mokiem
Fix Version/s: 2.4.0
2.3.1
Fixed in https://github.com/apache/spark/pull/20880
> Remove unnecessary scalastyle check disabling
> ---------------------------------------------
>
> Key: SPARK-23769
> URL: https://issues.apache.org/jira/browse/SPARK-23769
> Project: Spark
> Issue Type: Improvement
> Components: Spark Core
> Affects Versions: 2.4.0
> Reporter: Riaas Mokiem
> Assignee: Riaas Mokiem
> Priority: Minor
> Fix For: 2.3.1, 2.4.0
>
>
> In `org/apache/spark/util/CompletionIterator.scala` the Scalastyle checker is
> disabled for 1 line of code. However, this line of code doesn't seem to
> violate any of the currently active rules for Scalastyle. So the Scalastyle
> checker doesn't need to be disabled there.
> I've tested this by removing the comments that disable the checker and
> running the checker with `build/mv scalastyle:check`. With the comments
> removed (so with the checker active for that line) the build still succeeds
> and no violations are shown in `core/target/scalastyle-output.xml`. I'll
> create a pull request to remove the comments that disable the checker.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
