Repository: spark
Updated Branches:
  refs/heads/master 0558d021c -> 297b81e0e


[SPARK-20156][SQL][ML][FOLLOW-UP] Java String toLowerCase with Locale.ROOT

## What changes were proposed in this pull request?

Add `Locale.ROOT` to all internal calls to String `toLowerCase`, `toUpperCase`

## How was this patch tested?
existing tests

Closes #22975 from zhengruifeng/Tokenizer_Locale.

Authored-by: zhengruifeng <ruife...@foxmail.com>
Signed-off-by: Sean Owen <sean.o...@databricks.com>


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/297b81e0
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/297b81e0
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/297b81e0

Branch: refs/heads/master
Commit: 297b81e0eb1493b12838c3c48c6f754289ce1c1f
Parents: 0558d02
Author: zhengruifeng <ruife...@foxmail.com>
Authored: Fri Nov 9 07:55:02 2018 -0600
Committer: Sean Owen <sean.o...@databricks.com>
Committed: Fri Nov 9 07:55:02 2018 -0600

----------------------------------------------------------------------
 project/SparkBuild.scala | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/297b81e0/project/SparkBuild.scala
----------------------------------------------------------------------
diff --git a/project/SparkBuild.scala b/project/SparkBuild.scala
index ca57df0..5e034f9 100644
--- a/project/SparkBuild.scala
+++ b/project/SparkBuild.scala
@@ -17,6 +17,7 @@
 
 import java.io._
 import java.nio.file.Files
+import java.util.Locale
 
 import scala.io.Source
 import scala.util.Properties
@@ -650,10 +651,13 @@ object Assembly {
     },
     jarName in (Test, assembly) := 
s"${moduleName.value}-test-${version.value}.jar",
     mergeStrategy in assembly := {
-      case m if m.toLowerCase.endsWith("manifest.mf")          => 
MergeStrategy.discard
-      case m if m.toLowerCase.matches("meta-inf.*\\.sf$")      => 
MergeStrategy.discard
+      case m if m.toLowerCase(Locale.ROOT).endsWith("manifest.mf")
+                                                               => 
MergeStrategy.discard
+      case m if m.toLowerCase(Locale.ROOT).matches("meta-inf.*\\.sf$")
+                                                               => 
MergeStrategy.discard
       case "log4j.properties"                                  => 
MergeStrategy.discard
-      case m if m.toLowerCase.startsWith("meta-inf/services/") => 
MergeStrategy.filterDistinctLines
+      case m if m.toLowerCase(Locale.ROOT).startsWith("meta-inf/services/")
+                                                               => 
MergeStrategy.filterDistinctLines
       case "reference.conf"                                    => 
MergeStrategy.concat
       case _                                                   => 
MergeStrategy.first
     }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to