spark-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Apache Spark (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (SPARK-25655) Add Pspark-ganglia-lgpl to the scala style check
Date Fri, 05 Oct 2018 20:48:00 GMT

    [ https://issues.apache.org/jira/browse/SPARK-25655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16640328#comment-16640328
] 

Apache Spark commented on SPARK-25655:
--------------------------------------

User 'gatorsmile' has created a pull request for this issue:
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
>
> 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