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

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-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org

Reply via email to