spark-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hyukjin Kwon (JIRA)" <j...@apache.org>
Subject [jira] [Resolved] (SPARK-25655) Add Pspark-ganglia-lgpl to the scala style check
Date Sat, 06 Oct 2018 06:27:00 GMT

     [ https://issues.apache.org/jira/browse/SPARK-25655?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Hyukjin Kwon resolved SPARK-25655.
----------------------------------
       Resolution: Fixed
    Fix Version/s: 3.0.0

Issue resolved by pull request 22647
[https://github.com/apache/spark/pull/22647]

> Add Pspark-ganglia-lgpl to the scala style check
> ------------------------------------------------
>
>                 Key: SPARK-25655
>                 URL: https://issues.apache.org/jira/browse/SPARK-25655
>             Project: Spark
>          Issue Type: Bug
>          Components: Build
>    Affects Versions: 3.0.0
>            Reporter: Xiao Li
>            Assignee: Xiao Li
>            Priority: Major
>             Fix For: 3.0.0
>
>
> Our lint failed due to the following errors:
> {code}
> [INFO] --- scalastyle-maven-plugin:1.0.0:check (default) @ spark-ganglia-lgpl_2.11 ---
> error file=/home/jenkins/workspace/spark-master-maven-snapshots/spark/external/spark-ganglia-lgpl/src/main/scala/org/apache/spark/metrics/sink/GangliaSink.scala
message=
>       Are you sure that you want to use toUpperCase or toLowerCase without the root locale?
In most cases, you
>       should use toUpperCase(Locale.ROOT) or toLowerCase(Locale.ROOT) instead.
>       If you must use toUpperCase or toLowerCase without the root locale, wrap the code
block with
>       // scalastyle:off caselocale
>       .toUpperCase
>       .toLowerCase
>       // scalastyle:on caselocale
>      line=67 column=49
> error file=/home/jenkins/workspace/spark-master-maven-snapshots/spark/external/spark-ganglia-lgpl/src/main/scala/org/apache/spark/metrics/sink/GangliaSink.scala
message=
>       Are you sure that you want to use toUpperCase or toLowerCase without the root locale?
In most cases, you
>       should use toUpperCase(Locale.ROOT) or toLowerCase(Locale.ROOT) instead.
>       If you must use toUpperCase or toLowerCase without the root locale, wrap the code
block with
>       // scalastyle:off caselocale
>       .toUpperCase
>       .toLowerCase
>       // scalastyle:on caselocale
>      line=71 column=32
> Saving to outputFile=/home/jenkins/workspace/spark-master-maven-snapshots/spark/external/spark-ganglia-lgpl/target/scalastyle-output.xml
> {code}
> See https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Compile/job/spark-master-lint/8890/



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@spark.apache.org
For additional commands, e-mail: issues-help@spark.apache.org


Mime
View raw message