[spark] branch master updated (b344e91 -> 9270238)

2021-02-07 Thread kabhwan
This is an automated email from the ASF dual-hosted git repository.

kabhwan pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git.


from b344e91  [SPARK-34375][CORE][K8S][TEST] Replaces 'Mockito.initMocks' 
with 'Mockito.openMocks'
 add 9270238  [SPARK-34355][SQL] Add log and time cost for commit job

No new revisions were added by this update.

Summary of changes:
 .../apache/spark/sql/execution/datasources/FileFormatWriter.scala| 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)


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



[spark] branch master updated (329f945 -> b344e91)

2021-02-07 Thread gurwls223
This is an automated email from the ASF dual-hosted git repository.

gurwls223 pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git.


from 329f945  [SPARK-34391][BUILD] Upgrade commons-io to 2.8.0
 add b344e91  [SPARK-34375][CORE][K8S][TEST] Replaces 'Mockito.initMocks' 
with 'Mockito.openMocks'

No new revisions were added by this update.

Summary of changes:
 .../java/org/apache/spark/shuffle/sort/UnsafeShuffleWriterSuite.java  | 4 ++--
 .../org/apache/spark/unsafe/map/AbstractBytesToBytesMapSuite.java | 4 ++--
 .../spark/util/collection/unsafe/sort/UnsafeExternalSorterSuite.java  | 4 ++--
 core/src/test/scala/org/apache/spark/deploy/worker/WorkerSuite.scala  | 2 +-
 .../test/scala/org/apache/spark/shuffle/ShuffleBlockPusherSuite.scala | 2 +-
 .../apache/spark/shuffle/sort/BypassMergeSortShuffleWriterSuite.scala | 2 +-
 .../apache/spark/shuffle/sort/IndexShuffleBlockResolverSuite.scala| 2 +-
 .../scala/org/apache/spark/shuffle/sort/SortShuffleWriterSuite.scala  | 2 +-
 .../spark/shuffle/sort/io/LocalDiskShuffleMapOutputWriterSuite.scala  | 2 +-
 .../test/scala/org/apache/spark/deploy/k8s/submit/ClientSuite.scala   | 2 +-
 .../scala/org/apache/spark/deploy/k8s/submit/K8sSubmitOpSuite.scala   | 2 +-
 .../spark/scheduler/cluster/k8s/ExecutorPodsAllocatorSuite.scala  | 2 +-
 .../scheduler/cluster/k8s/ExecutorPodsLifecycleManagerSuite.scala | 2 +-
 .../cluster/k8s/ExecutorPodsPollingSnapshotSourceSuite.scala  | 2 +-
 .../scheduler/cluster/k8s/ExecutorPodsWatchSnapshotSourceSuite.scala  | 2 +-
 .../cluster/k8s/KubernetesClusterSchedulerBackendSuite.scala  | 2 +-
 16 files changed, 19 insertions(+), 19 deletions(-)


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



[spark] branch master updated (dcaf62a -> 329f945)

2021-02-07 Thread dongjoon
This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git.


from dcaf62a  [SPARK-34346][CORE][TESTS][FOLLOWUP] Fix UT by removing 
core-site.xml
 add 329f945  [SPARK-34391][BUILD] Upgrade commons-io to 2.8.0

No new revisions were added by this update.

Summary of changes:
 dev/deps/spark-deps-hadoop-3.2-hive-2.3 | 2 +-
 pom.xml | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)


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



[spark] branch branch-3.0 updated: [SPARK-34346][CORE][TESTS][FOLLOWUP] Fix UT by removing core-site.xml

2021-02-07 Thread gurwls223
This is an automated email from the ASF dual-hosted git repository.

gurwls223 pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.0 by this push:
 new 579e158  [SPARK-34346][CORE][TESTS][FOLLOWUP] Fix UT by removing 
core-site.xml
579e158 is described below

commit 579e15873c5a8ff3fcaafec5ce99e37884768f51
Author: Dongjoon Hyun 
AuthorDate: Mon Feb 8 11:32:23 2021 +0900

[SPARK-34346][CORE][TESTS][FOLLOWUP] Fix UT by removing core-site.xml

### What changes were proposed in this pull request?

This is a follow-up for SPARK-34346 which causes a flakiness due to 
`core-site.xml` test resource file addition. This PR aims to remove the test 
resource `core/src/test/resources/core-site.xml` from `core` module.

### Why are the changes needed?

Due to the test resource `core-site.xml`, YARN UT becomes flaky in GitHub 
Action and Jenkins.
```
$ build/sbt "yarn/testOnly *.YarnClusterSuite -- -z SPARK-16414" -Pyarn
...
[info] YarnClusterSuite:
[info] - yarn-cluster should respect conf overrides in SparkHadoopUtil 
(SPARK-16414, SPARK-23630) *** FAILED *** (20 seconds, 209 milliseconds)
[info]   FAILED did not equal FINISHED (stdout/stderr was not captured) 
(BaseYarnClusterSuite.scala:210)
```

To isolate more, we may use `SPARK_TEST_HADOOP_CONF_DIR` like `yarn` 
module's `yarn/Client`, but it seems an overkill in `core` module.
```
// SPARK-23630: during testing, Spark scripts filter out hadoop conf dirs 
so that user's
// environments do not interfere with tests. This allows a special env 
variable during
// tests so that custom conf dirs can be used by unit tests.
val confDirs = Seq("HADOOP_CONF_DIR", "YARN_CONF_DIR") ++
  (if (Utils.isTesting) Seq("SPARK_TEST_HADOOP_CONF_DIR") else Nil)
```

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Pass the CIs.

Closes #31515 from dongjoon-hyun/SPARK-34346-2.

Authored-by: Dongjoon Hyun 
Signed-off-by: HyukjinKwon 
(cherry picked from commit dcaf62afea8791e49a44c2062fe14bafdcc0e92f)
Signed-off-by: HyukjinKwon 
---
 core/src/test/resources/core-site.xml  | 24 --
 .../scala/org/apache/spark/SparkContextSuite.scala |  3 ++-
 2 files changed, 2 insertions(+), 25 deletions(-)

diff --git a/core/src/test/resources/core-site.xml 
b/core/src/test/resources/core-site.xml
deleted file mode 100644
index 84eddf8..000
--- a/core/src/test/resources/core-site.xml
+++ /dev/null
@@ -1,24 +0,0 @@
-
-
-
-
-hadoop.tmp.dir
-/tmp/hive_zero
-default is /tmp/hadoop-${user.name} and will be 
overridden
-
-
diff --git a/core/src/test/scala/org/apache/spark/SparkContextSuite.scala 
b/core/src/test/scala/org/apache/spark/SparkContextSuite.scala
index e0a5f52..01b1deb 100644
--- a/core/src/test/scala/org/apache/spark/SparkContextSuite.scala
+++ b/core/src/test/scala/org/apache/spark/SparkContextSuite.scala
@@ -979,11 +979,12 @@ class SparkContextSuite extends SparkFunSuite with 
LocalSparkContext with Eventu
 val testKey = "hadoop.tmp.dir"
 val bufferKey = "io.file.buffer.size"
 val hadoopConf0 = new Configuration()
+hadoopConf0.set(testKey, "/tmp/hive_zero")
 
 val hiveConfFile = 
Utils.getContextOrSparkClassLoader.getResource("hive-site.xml")
 assert(hiveConfFile != null)
 hadoopConf0.addResource(hiveConfFile)
-assert(hadoopConf0.get(testKey) === "/tmp/hive_one")
+assert(hadoopConf0.get(testKey) === "/tmp/hive_zero")
 assert(hadoopConf0.get(bufferKey) === "201811")
 
 val sparkConf = new SparkConf()


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



[spark] branch branch-3.1 updated: [SPARK-34346][CORE][TESTS][FOLLOWUP] Fix UT by removing core-site.xml

2021-02-07 Thread gurwls223
This is an automated email from the ASF dual-hosted git repository.

gurwls223 pushed a commit to branch branch-3.1
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.1 by this push:
 new 2fe20ef  [SPARK-34346][CORE][TESTS][FOLLOWUP] Fix UT by removing 
core-site.xml
2fe20ef is described below

commit 2fe20ef060a80aff1a20ef30f5e61992efd0fe50
Author: Dongjoon Hyun 
AuthorDate: Mon Feb 8 11:32:23 2021 +0900

[SPARK-34346][CORE][TESTS][FOLLOWUP] Fix UT by removing core-site.xml

### What changes were proposed in this pull request?

This is a follow-up for SPARK-34346 which causes a flakiness due to 
`core-site.xml` test resource file addition. This PR aims to remove the test 
resource `core/src/test/resources/core-site.xml` from `core` module.

### Why are the changes needed?

Due to the test resource `core-site.xml`, YARN UT becomes flaky in GitHub 
Action and Jenkins.
```
$ build/sbt "yarn/testOnly *.YarnClusterSuite -- -z SPARK-16414" -Pyarn
...
[info] YarnClusterSuite:
[info] - yarn-cluster should respect conf overrides in SparkHadoopUtil 
(SPARK-16414, SPARK-23630) *** FAILED *** (20 seconds, 209 milliseconds)
[info]   FAILED did not equal FINISHED (stdout/stderr was not captured) 
(BaseYarnClusterSuite.scala:210)
```

To isolate more, we may use `SPARK_TEST_HADOOP_CONF_DIR` like `yarn` 
module's `yarn/Client`, but it seems an overkill in `core` module.
```
// SPARK-23630: during testing, Spark scripts filter out hadoop conf dirs 
so that user's
// environments do not interfere with tests. This allows a special env 
variable during
// tests so that custom conf dirs can be used by unit tests.
val confDirs = Seq("HADOOP_CONF_DIR", "YARN_CONF_DIR") ++
  (if (Utils.isTesting) Seq("SPARK_TEST_HADOOP_CONF_DIR") else Nil)
```

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Pass the CIs.

Closes #31515 from dongjoon-hyun/SPARK-34346-2.

Authored-by: Dongjoon Hyun 
Signed-off-by: HyukjinKwon 
(cherry picked from commit dcaf62afea8791e49a44c2062fe14bafdcc0e92f)
Signed-off-by: HyukjinKwon 
---
 core/src/test/resources/core-site.xml  | 24 --
 .../scala/org/apache/spark/SparkContextSuite.scala |  3 ++-
 2 files changed, 2 insertions(+), 25 deletions(-)

diff --git a/core/src/test/resources/core-site.xml 
b/core/src/test/resources/core-site.xml
deleted file mode 100644
index 84eddf8..000
--- a/core/src/test/resources/core-site.xml
+++ /dev/null
@@ -1,24 +0,0 @@
-
-
-
-
-hadoop.tmp.dir
-/tmp/hive_zero
-default is /tmp/hadoop-${user.name} and will be 
overridden
-
-
diff --git a/core/src/test/scala/org/apache/spark/SparkContextSuite.scala 
b/core/src/test/scala/org/apache/spark/SparkContextSuite.scala
index 7751918..0c0a9b8 100644
--- a/core/src/test/scala/org/apache/spark/SparkContextSuite.scala
+++ b/core/src/test/scala/org/apache/spark/SparkContextSuite.scala
@@ -1040,11 +1040,12 @@ class SparkContextSuite extends SparkFunSuite with 
LocalSparkContext with Eventu
 val testKey = "hadoop.tmp.dir"
 val bufferKey = "io.file.buffer.size"
 val hadoopConf0 = new Configuration()
+hadoopConf0.set(testKey, "/tmp/hive_zero")
 
 val hiveConfFile = 
Utils.getContextOrSparkClassLoader.getResource("hive-site.xml")
 assert(hiveConfFile != null)
 hadoopConf0.addResource(hiveConfFile)
-assert(hadoopConf0.get(testKey) === "/tmp/hive_one")
+assert(hadoopConf0.get(testKey) === "/tmp/hive_zero")
 assert(hadoopConf0.get(bufferKey) === "201811")
 
 val sparkConf = new SparkConf()


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



[spark] branch master updated: [SPARK-34346][CORE][TESTS][FOLLOWUP] Fix UT by removing core-site.xml

2021-02-07 Thread gurwls223
This is an automated email from the ASF dual-hosted git repository.

gurwls223 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
 new dcaf62a  [SPARK-34346][CORE][TESTS][FOLLOWUP] Fix UT by removing 
core-site.xml
dcaf62a is described below

commit dcaf62afea8791e49a44c2062fe14bafdcc0e92f
Author: Dongjoon Hyun 
AuthorDate: Mon Feb 8 11:32:23 2021 +0900

[SPARK-34346][CORE][TESTS][FOLLOWUP] Fix UT by removing core-site.xml

### What changes were proposed in this pull request?

This is a follow-up for SPARK-34346 which causes a flakiness due to 
`core-site.xml` test resource file addition. This PR aims to remove the test 
resource `core/src/test/resources/core-site.xml` from `core` module.

### Why are the changes needed?

Due to the test resource `core-site.xml`, YARN UT becomes flaky in GitHub 
Action and Jenkins.
```
$ build/sbt "yarn/testOnly *.YarnClusterSuite -- -z SPARK-16414" -Pyarn
...
[info] YarnClusterSuite:
[info] - yarn-cluster should respect conf overrides in SparkHadoopUtil 
(SPARK-16414, SPARK-23630) *** FAILED *** (20 seconds, 209 milliseconds)
[info]   FAILED did not equal FINISHED (stdout/stderr was not captured) 
(BaseYarnClusterSuite.scala:210)
```

To isolate more, we may use `SPARK_TEST_HADOOP_CONF_DIR` like `yarn` 
module's `yarn/Client`, but it seems an overkill in `core` module.
```
// SPARK-23630: during testing, Spark scripts filter out hadoop conf dirs 
so that user's
// environments do not interfere with tests. This allows a special env 
variable during
// tests so that custom conf dirs can be used by unit tests.
val confDirs = Seq("HADOOP_CONF_DIR", "YARN_CONF_DIR") ++
  (if (Utils.isTesting) Seq("SPARK_TEST_HADOOP_CONF_DIR") else Nil)
```

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Pass the CIs.

Closes #31515 from dongjoon-hyun/SPARK-34346-2.

Authored-by: Dongjoon Hyun 
Signed-off-by: HyukjinKwon 
---
 core/src/test/resources/core-site.xml  | 24 --
 .../scala/org/apache/spark/SparkContextSuite.scala |  3 ++-
 2 files changed, 2 insertions(+), 25 deletions(-)

diff --git a/core/src/test/resources/core-site.xml 
b/core/src/test/resources/core-site.xml
deleted file mode 100644
index 84eddf8..000
--- a/core/src/test/resources/core-site.xml
+++ /dev/null
@@ -1,24 +0,0 @@
-
-
-
-
-hadoop.tmp.dir
-/tmp/hive_zero
-default is /tmp/hadoop-${user.name} and will be 
overridden
-
-
diff --git a/core/src/test/scala/org/apache/spark/SparkContextSuite.scala 
b/core/src/test/scala/org/apache/spark/SparkContextSuite.scala
index 4319761..7a4970e 100644
--- a/core/src/test/scala/org/apache/spark/SparkContextSuite.scala
+++ b/core/src/test/scala/org/apache/spark/SparkContextSuite.scala
@@ -1155,11 +1155,12 @@ class SparkContextSuite extends SparkFunSuite with 
LocalSparkContext with Eventu
 val testKey = "hadoop.tmp.dir"
 val bufferKey = "io.file.buffer.size"
 val hadoopConf0 = new Configuration()
+hadoopConf0.set(testKey, "/tmp/hive_zero")
 
 val hiveConfFile = 
Utils.getContextOrSparkClassLoader.getResource("hive-site.xml")
 assert(hiveConfFile != null)
 hadoopConf0.addResource(hiveConfFile)
-assert(hadoopConf0.get(testKey) === "/tmp/hive_one")
+assert(hadoopConf0.get(testKey) === "/tmp/hive_zero")
 assert(hadoopConf0.get(bufferKey) === "201811")
 
 val sparkConf = new SparkConf()


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



[spark] branch branch-3.1 updated: [SPARK-34158] Incorrect url of the only developer Matei in pom.xml

2021-02-07 Thread gurwls223
This is an automated email from the ASF dual-hosted git repository.

gurwls223 pushed a commit to branch branch-3.1
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.1 by this push:
 new 14d0419  [SPARK-34158] Incorrect url of the only developer Matei in 
pom.xml
14d0419 is described below

commit 14d041986759c0721420b2c1f343fb00b7b31515
Author: Kevin 
AuthorDate: Mon Feb 8 09:15:05 2021 +0900

[SPARK-34158] Incorrect url of the only developer Matei in pom.xml

### What changes were proposed in this pull request?

Update the Incorrect URL of the only developer Matei in pom.xml

### Why are the changes needed?

The current link was broken

### Does this PR introduce _any_ user-facing change?

No

### How was this patch tested?

change the link to https://cs.stanford.edu/people/matei/

Closes #31512 from pingsutw/SPARK-34158.

Authored-by: Kevin 
Signed-off-by: HyukjinKwon 
(cherry picked from commit 30ef3d6e1c00bd1f28e71511576daad223ba8b22)
Signed-off-by: HyukjinKwon 
---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 68eea2f..9811826 100644
--- a/pom.xml
+++ b/pom.xml
@@ -48,7 +48,7 @@
   matei
   Matei Zaharia
   matei.zaha...@gmail.com
-  http://www.cs.berkeley.edu/~matei
+  https://cs.stanford.edu/people/matei
   Apache Software Foundation
   http://spark.apache.org
 


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



[spark] branch master updated: [SPARK-34158] Incorrect url of the only developer Matei in pom.xml

2021-02-07 Thread gurwls223
This is an automated email from the ASF dual-hosted git repository.

gurwls223 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
 new 30ef3d6  [SPARK-34158] Incorrect url of the only developer Matei in 
pom.xml
30ef3d6 is described below

commit 30ef3d6e1c00bd1f28e71511576daad223ba8b22
Author: Kevin 
AuthorDate: Mon Feb 8 09:15:05 2021 +0900

[SPARK-34158] Incorrect url of the only developer Matei in pom.xml

### What changes were proposed in this pull request?

Update the Incorrect URL of the only developer Matei in pom.xml

### Why are the changes needed?

The current link was broken

### Does this PR introduce _any_ user-facing change?

No

### How was this patch tested?

change the link to https://cs.stanford.edu/people/matei/

Closes #31512 from pingsutw/SPARK-34158.

Authored-by: Kevin 
Signed-off-by: HyukjinKwon 
---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 8bb618c..348f9d6 100644
--- a/pom.xml
+++ b/pom.xml
@@ -48,7 +48,7 @@
   matei
   Matei Zaharia
   matei.zaha...@gmail.com
-  http://www.cs.berkeley.edu/~matei
+  https://cs.stanford.edu/people/matei
   Apache Software Foundation
   http://spark.apache.org
 


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



[spark] branch branch-3.1 updated: [SPARK-34398][DOCS] Fix PySpark migration link

2021-02-07 Thread gurwls223
This is an automated email from the ASF dual-hosted git repository.

gurwls223 pushed a commit to branch branch-3.1
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.1 by this push:
 new de51f25  [SPARK-34398][DOCS] Fix PySpark migration link
de51f25 is described below

commit de51f25c02894941029ea6e5a029e909465b658a
Author: raphaelauv 
AuthorDate: Mon Feb 8 09:12:15 2021 +0900

[SPARK-34398][DOCS] Fix PySpark migration link

### What changes were proposed in this pull request?

docs/pyspark-migration-guide.md

### Why are the changes needed?
broken link

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Manually build and check

Closes #31514 from raphaelauv/patch-2.

Authored-by: raphaelauv 
Signed-off-by: HyukjinKwon 
(cherry picked from commit 34a1a65b398c4469eb97cd458ee172dc76b7ef56)
Signed-off-by: HyukjinKwon 
---
 docs/pyspark-migration-guide.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/docs/pyspark-migration-guide.md b/docs/pyspark-migration-guide.md
index e117ed1..5a1ca9b 100644
--- a/docs/pyspark-migration-guide.md
+++ b/docs/pyspark-migration-guide.md
@@ -19,4 +19,4 @@ license: |
   limitations under the License.
 ---
 
-The migration guide is now archived on [this 
page](https://spark.apache.org/docs/latest/api/python/migration_guide/index.html).
\ No newline at end of file
+The migration guide is now archived on [this 
page](api/python/migration_guide/index.html).


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



[spark] branch master updated (eb5558e -> 34a1a65)

2021-02-07 Thread gurwls223
This is an automated email from the ASF dual-hosted git repository.

gurwls223 pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git.


from eb5558e  [SPARK-34390][CORE] Enable Zstandard buffer pool by default
 add 34a1a65  [SPARK-34398][DOCS] Fix PySpark migration link

No new revisions were added by this update.

Summary of changes:
 docs/pyspark-migration-guide.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


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