Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19861
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19861
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84995/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19861
**[Test build #84995 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84995/testReport)**
for PR 19861 at commit
[`5292329`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19998
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84993/
Test FAILed.
---
Github user yashs360 commented on the issue:
https://github.com/apache/spark/pull/18029
Hi @brkyvz Thinking on these lines, Adding them as Java objects adds more
complexity to our design. We again have to think about making the objects
singleton and thread safe. The Scala case class w
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19998
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19998
**[Test build #84993 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84993/testReport)**
for PR 19998 at commit
[`964e5ff`](https://github.com/apache/spark/commit/9
Github user morenn520 commented on the issue:
https://github.com/apache/spark/pull/1
@gatorsmile we fix it in SPARK 1.6.2, and take in use for two month. For
further reason, I give one pr on master branch. I will test it next week.
---
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84998/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #84998 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84998/testReport)**
for PR 1 at commit
[`d1d310c`](https://github.com/apache/spark/commit/d
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/1
The support is interesting, but the current impl is not clean. cc
@dongjoon-hyun Could you help reviewing this PR?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #84998 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84998/testReport)**
for PR 1 at commit
[`d1d310c`](https://github.com/apache/spark/commit/d1
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/1
Please update the PR title
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19998
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19998
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84992/
Test FAILed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/1
Could you write a test case?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/1
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19998
**[Test build #84992 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84992/testReport)**
for PR 19998 at commit
[`b384336`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user morenn520 opened a pull request:
https://github.com/apache/spark/pull/1
JDBC support date/timestamp type as partitionColumn
Jira: https://issues.apache.org/jira/browse/SPARK-22814
PartitionColumn must be a numeric column from the table.
However, there are
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19594
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84991/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19594
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19594
**[Test build #84991 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84991/testReport)**
for PR 19594 at commit
[`2637429`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19998
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19998
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84990/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19998
**[Test build #84990 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84990/testReport)**
for PR 19998 at commit
[`969bc22`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19998
**[Test build #84997 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84997/testReport)**
for PR 19998 at commit
[`6c29a11`](https://github.com/apache/spark/commit/6c
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19998#discussion_r157334828
--- Diff: dev/run-tests.py ---
@@ -253,9 +253,11 @@ def kill_zinc_on_port(zinc_port):
"""
Kill the Zinc process running on the given port, if
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19995
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19995
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84986/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19995
**[Test build #84986 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84986/testReport)**
for PR 19995 at commit
[`b3e1af3`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19977
**[Test build #84996 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84996/testReport)**
for PR 19977 at commit
[`2d3926e`](https://github.com/apache/spark/commit/2d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19977
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84987/
Test FAILed.
---
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/19977
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h..
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19977
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19977
**[Test build #84987 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84987/testReport)**
for PR 19977 at commit
[`2d3926e`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19594
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19594
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84989/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19594
**[Test build #84989 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84989/testReport)**
for PR 19594 at commit
[`2a4ee99`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19861
**[Test build #84995 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84995/testReport)**
for PR 19861 at commit
[`5292329`](https://github.com/apache/spark/commit/52
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19861
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19861
Hm ..
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19861
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19861
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84988/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19861
**[Test build #84988 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84988/testReport)**
for PR 19861 at commit
[`5292329`](https://github.com/apache/spark/commit/5
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19998#discussion_r157333856
--- Diff: dev/run-tests.py ---
@@ -253,9 +253,11 @@ def kill_zinc_on_port(zinc_port):
"""
Kill the Zinc process running on the given po
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19981
**[Test build #84994 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84994/testReport)**
for PR 19981 at commit
[`bc300f9`](https://github.com/apache/spark/commit/bc
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/19981
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19981#discussion_r15724
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SharedState.scala ---
@@ -82,6 +82,19 @@ private[sql] class SharedState(val sparkContext:
Sp
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19998
**[Test build #84993 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84993/testReport)**
for PR 19998 at commit
[`964e5ff`](https://github.com/apache/spark/commit/96
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19998#discussion_r157333278
--- Diff: dev/run-tests.py ---
@@ -253,9 +253,11 @@ def kill_zinc_on_port(zinc_port):
"""
Kill the Zinc process running on the given port, if
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84985/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #84985 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84985/testReport)**
for PR 19954 at commit
[`46a8c99`](https://github.com/apache/spark/commit/4
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19998#discussion_r157333189
--- Diff: dev/run-tests.py ---
@@ -253,9 +253,11 @@ def kill_zinc_on_port(zinc_port):
"""
Kill the Zinc process running on the given po
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19998#discussion_r157333050
--- Diff: dev/run-tests.py ---
@@ -253,9 +253,11 @@ def kill_zinc_on_port(zinc_port):
"""
Kill the Zinc process running on the given port, if
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19998#discussion_r157332995
--- Diff: dev/run-tests.py ---
@@ -253,9 +253,11 @@ def kill_zinc_on_port(zinc_port):
"""
Kill the Zinc process running on the given po
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19998
**[Test build #84992 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84992/testReport)**
for PR 19998 at commit
[`b384336`](https://github.com/apache/spark/commit/b3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19955
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19955
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84983/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19955
**[Test build #84983 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84983/testReport)**
for PR 19955 at commit
[`59e7720`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19594
**[Test build #84991 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84991/testReport)**
for PR 19594 at commit
[`2637429`](https://github.com/apache/spark/commit/26
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19996
Oh, maybe it's not quite related but do you mind if I ask to fix the below
too?
https://github.com/apache/spark/blob/ccdf21f56e4ff5497d7770dcbee2f7a60bb9e3a7/docs/sql-programming-guide.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19984
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84981/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19984
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19984
**[Test build #84981 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84981/testReport)**
for PR 19984 at commit
[`f50488c`](https://github.com/apache/spark/commit/f
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19998#discussion_r157332103
--- Diff: dev/run-tests.py ---
@@ -253,9 +253,14 @@ def kill_zinc_on_port(zinc_port):
"""
Kill the Zinc process running on the given port, if
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19998#discussion_r157331922
--- Diff: dev/run-tests.py ---
@@ -253,9 +253,14 @@ def kill_zinc_on_port(zinc_port):
"""
Kill the Zinc process running on the given po
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19998
@srowen @HyukjinKwon could you please review this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additiona
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19998
**[Test build #84990 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84990/testReport)**
for PR 19998 at commit
[`969bc22`](https://github.com/apache/spark/commit/96
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/19594#discussion_r157331840
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/EstimationUtils.scala
---
@@ -114,4 +115,183 @@ object Esti
GitHub user kiszk opened a pull request:
https://github.com/apache/spark/pull/19998
[SPARK-22377][BUILD] Use lsof or /usr/sbin/lsof in run-tests.py
## What changes were proposed in this pull request?
In [the environment where `/usr/sbin/lsof` does not
exist](https://github.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19594
**[Test build #84989 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84989/testReport)**
for PR 19594 at commit
[`2a4ee99`](https://github.com/apache/spark/commit/2a
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/19594#discussion_r157331711
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/JoinEstimation.scala
---
@@ -191,8 +191,16 @@ case class Jo
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19981#discussion_r157331435
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SharedState.scala ---
@@ -82,6 +82,19 @@ private[sql] class SharedState(val sparkContext:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19995
Seems it's related with
https://github.com/apache/spark/commit/e58f275678fb4f904124a4a2a1762f04c835eb0e
somehow and then fine back now. I am not yet entirely sure how this change
relates to CRA
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19681#discussion_r157331270
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/ui/SQLListenerSuite.scala
---
@@ -36,13 +36,14 @@ import org.apache.spark.sql.catalyst.u
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19861
**[Test build #84988 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84988/testReport)**
for PR 19861 at commit
[`5292329`](https://github.com/apache/spark/commit/52
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19977
**[Test build #84987 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84987/testReport)**
for PR 19977 at commit
[`2d3926e`](https://github.com/apache/spark/commit/2d
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19997
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19995
The R tests are pretty flaky recently, any ideas @HyukjinKwon ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apach
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19997
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19861
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19995
**[Test build #84986 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84986/testReport)**
for PR 19995 at commit
[`b3e1af3`](https://github.com/apache/spark/commit/b3
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19995
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19997#discussion_r157330796
--- Diff: python/pyspark/ml/tests.py ---
@@ -44,6 +44,7 @@
import numpy as np
from numpy import abs, all, arange, array, array_equal, inf, ones
Github user foxish commented on the issue:
https://github.com/apache/spark/pull/19995
@ueshin @vanzin SparkR failure seems unrelated to me. Any ideas?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19995
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84979/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19995
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19995
**[Test build #84979 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84979/testReport)**
for PR 19995 at commit
[`b3e1af3`](https://github.com/apache/spark/commit/b
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/19995
LGTM pending tests.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user liyinan926 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19954#discussion_r157328425
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/rest/k8s/KubernetesSparkDependencyDownloadInitContainer.scala
---
@@
Github user liyinan926 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19954#discussion_r157328327
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/Config.scala
---
@@ -133,30 +132,78 @@ private[spark] object Conf
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19813
@mgaido91 Thanks for the comment. I agreed that to make the contract is the
easiest way. If we don't make this contract, seems to me a significant change
is needed.
---
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #84985 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84985/testReport)**
for PR 19954 at commit
[`46a8c99`](https://github.com/apache/spark/commit/46
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/19954
@vanzin Addressed your comments so far. PTAL. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For a
Github user liyinan926 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19954#discussion_r157327812
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/rest/k8s/FileFetcher.scala
---
@@ -0,0 +1,27 @@
+/*
+ * Licen
Github user foxish commented on a diff in the pull request:
https://github.com/apache/spark/pull/19954#discussion_r157327721
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/Config.scala
---
@@ -133,30 +132,78 @@ private[spark] object Config e
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19954#discussion_r157327642
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/rest/k8s/KubernetesSparkDependencyDownloadInitContainer.scala
---
@@ -0,0
1 - 100 of 425 matches
Mail list logo