Repository: spark
Updated Branches:
  refs/heads/master b73f76beb -> 6bbceb9fe


[SPARK-25726][SQL][TEST] Fix flaky test in SaveIntoDataSourceCommandSuite

## What changes were proposed in this pull request?

[SPARK-22479](https://github.com/apache/spark/pull/19708/files#diff-5c22ac5160d3c9d81225c5dd86265d27R31)
 adds a test case which sometimes fails because the used password string `123` 
matches `41230802`. This PR aims to fix the flakiness.

- 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97343/consoleFull

```scala
SaveIntoDataSourceCommandSuite:
- simpleString is redacted *** FAILED ***
"SaveIntoDataSourceCommand 
.org.apache.spark.sql.execution.datasources.jdbc.JdbcRelationProvider41230802, 
Map(password -> *********(redacted), url -> *********(redacted), driver -> 
mydriver), ErrorIfExists
+- Range (0, 1, step=1, splits=Some(2))
" contained "123" (SaveIntoDataSourceCommandSuite.scala:42)
```

## How was this patch tested?

Pass the Jenkins with the updated test case

Closes #22716 from dongjoon-hyun/SPARK-25726.

Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>


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

Branch: refs/heads/master
Commit: 6bbceb9fefe815d18001c6dd84f9ea2883d17a88
Parents: b73f76b
Author: Dongjoon Hyun <[email protected]>
Authored: Sat Oct 13 18:01:28 2018 -0700
Committer: Dongjoon Hyun <[email protected]>
Committed: Sat Oct 13 18:01:28 2018 -0700

----------------------------------------------------------------------
 .../sql/execution/datasources/SaveIntoDataSourceCommandSuite.scala | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/6bbceb9f/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/SaveIntoDataSourceCommandSuite.scala
----------------------------------------------------------------------
diff --git 
a/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/SaveIntoDataSourceCommandSuite.scala
 
b/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/SaveIntoDataSourceCommandSuite.scala
index a1da3ec..8b06b17 100644
--- 
a/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/SaveIntoDataSourceCommandSuite.scala
+++ 
b/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/SaveIntoDataSourceCommandSuite.scala
@@ -25,7 +25,7 @@ class SaveIntoDataSourceCommandSuite extends SharedSQLContext 
{
 
   test("simpleString is redacted") {
     val URL = "connection.url"
-    val PASS = "123"
+    val PASS = "mypassword"
     val DRIVER = "mydriver"
 
     val dataSource = DataSource(


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to