Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20864
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88537/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20864
**[Test build #88537 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88537/testReport)**
for PR 20864 at commit
[`b177bf4`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20864
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user jongyoul commented on a diff in the pull request:
https://github.com/apache/spark/pull/20860#discussion_r176652812
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/IsolatedClientLoader.scala
---
@@ -179,7 +179,7 @@ private[hive] class IsolatedClientLo
Github user jongyoul commented on a diff in the pull request:
https://github.com/apache/spark/pull/20860#discussion_r176654007
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/IsolatedClientLoader.scala
---
@@ -179,7 +179,7 @@ private[hive] class IsolatedClientLo
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20860
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1719/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20860
Merged build finished. Test PASSed.
---
-
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/20860
**[Test build #88538 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88538/testReport)**
for PR 20860 at commit
[`2ea9b7a`](https://github.com/apache/spark/commit/2e
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20836#discussion_r176656905
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaDataConsumer.scala
---
@@ -279,9 +279,8 @@ private[kafka010] case
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/20836
I don't see the problem. You see an exception which tells exactly what can
be done:
> If you don't want your streaming query to fail on such cases, set the
source option "failOnDataLo
Github user yucai commented on a diff in the pull request:
https://github.com/apache/spark/pull/20851#discussion_r176660685
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -353,6 +353,13 @@ object SQLConf {
.booleanConf
.cr
Github user zuotingbing commented on the issue:
https://github.com/apache/spark/pull/20864
@felixcheung Thanks. The test result is 0 failed, what is meaning of
> This patch fails due to an unknown error code, -9
? what should i do to fix the test fails?
---
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20885
**[Test build #88539 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88539/testReport)**
for PR 20885 at commit
[`d632706`](https://github.com/apache/spark/commit/d6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20885
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88539/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20885
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/20885
**[Test build #88539 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88539/testReport)**
for PR 20885 at commit
[`d632706`](https://github.com/apache/spark/commit/d
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20851#discussion_r176676405
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -353,6 +353,13 @@ object SQLConf {
.booleanConf
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/20541
oh, I see, I fallback to the modification of the non-deterministic
expression, and to keep the newly added test cases for a+1 and a+b, can you?
---
--
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20860
**[Test build #88538 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88538/testReport)**
for PR 20860 at commit
[`2ea9b7a`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20860
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88538/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20860
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user andrusha commented on a diff in the pull request:
https://github.com/apache/spark/pull/20811#discussion_r176687092
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodFactory.scala
---
@@ -108,6 +109,8 @@ private
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20811
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/1701/
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20811
Merged build finished. Test PASSed.
---
-
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/20811
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1720/
Tes
Github user Lemonjing commented on the issue:
https://github.com/apache/spark/pull/20889
@felixcheung hiï¼In my reading processï¼I found another md file typos and
fix it. Do I need to close this pr and merge it into one commit and give a new
pr? Thanks.
---
-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20811
Kubernetes integration test status failure
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/1701/
---
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/20877
> Does that fix actual usecases?
I see the following use cases:
1. Jsons coming usually from embedded systems have not-standard separators
(invisible in some cases). It is very conv
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20885
**[Test build #88540 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88540/testReport)**
for PR 20885 at commit
[`bbff402`](https://github.com/apache/spark/commit/bb
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20879
Yup, we are running old lint in PR builders and those are on the newer one.
---
-
To unsubscribe, e-mail: reviews-unsubscr...
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20879
Merged to master and branch-2.3.
Thanks for reviewing this @shaneknapp and @felixcheung.
---
-
To unsubscribe, e-mai
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20880
Merged to master and branch-2.3.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20880
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20879
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20889
I think you can add all other typos into here.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additio
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20884
Shall we add a configuration to control its behaviour if this is something
we need to support?
---
-
To unsubscribe, e-mail:
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20884#discussion_r176715906
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/json/JacksonGeneratorSuite.scala
---
@@ -56,7 +56,7 @@ class JacksonGeneratorSuite
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20877
Yup, yup. I don't object for now. Shall we merge this one first and talk
more about it in your PR?
I believe this PR itself proposes a complete option and I saw many the
requests for this fe
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20885
@cloud-fan and @hvanhovell. Do you think we need the flexible option for
line separator?
https://github.com/apache/spark/blob/bbff40206e6871ea9ab035e7a8876f495bdf3d90/sql/catalyst/src/m
Github user Lemonjing commented on the issue:
https://github.com/apache/spark/pull/20889
@HyukjinKwon Thanks a lot, could u help me to review these commits if u
have timeã
---
-
To unsubscribe, e-mail: reviews-uns
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/20877
I have only one concern: if we merge this PR, we close the possibility for
changing format of `lineSep` and future extensions. Your changes allow any
sequence of chars. It is not clear for me, how w
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/19881#discussion_r176727530
--- Diff:
core/src/test/scala/org/apache/spark/ExecutorAllocationManagerSuite.scala ---
@@ -145,6 +145,39 @@ class ExecutorAllocationManagerSuite
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/19881
+1
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apach
Github user goungoun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20800#discussion_r176728379
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -511,6 +511,14 @@ class Dataset[T] private[sql](
*/
def isLocal:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20877
I think we are fine to change the behaviour of `lineSep` before the release
..
---
-
To unsubscribe, e-mail: reviews-unsubsc
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20888
ah ok, yes when run in isolation, the stage will be 0, so your change makes
sense. But that is not what is making it flaky in a full test run
---
--
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20883
please update the description moving
> Fixes SPARK-23759 by moving connector.start() after connector.setHost()
after the title (What changes...) and please remove the sentence:
GitHub user kiszk opened a pull request:
https://github.com/apache/spark/pull/20890
[WIP][SPARK-23779][SQL] TaskMemoryManager and UnsafeSorter related classes
use MemoryBlock
## What changes were proposed in this pull request?
Waiting for merging #19222
This PR tri
Github user felixalbani commented on the issue:
https://github.com/apache/spark/pull/20883
@mgaido91 Thanks, I updated description with your suggestions
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20890
**[Test build #88541 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88541/testReport)**
for PR 20890 at commit
[`fce58f8`](https://github.com/apache/spark/commit/fc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20890
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1721/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20890
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20883
Thanks, LGTM
---
-
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/20890
**[Test build #88541 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88541/testReport)**
for PR 20890 at commit
[`fce58f8`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20890
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88541/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20890
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20717#discussion_r176750470
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -341,6 +341,26 @@ case class Join(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20717
**[Test build #88542 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88542/testReport)**
for PR 20717 at commit
[`5cadd86`](https://github.com/apache/spark/commit/5c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20717
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1722/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20717
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/20818#discussion_r176753823
--- Diff: core/src/main/scala/org/apache/spark/ui/UIUtils.scala ---
@@ -265,6 +266,7 @@ private[spark] object UIUtils extends Logging {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20885
**[Test build #88540 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88540/testReport)**
for PR 20885 at commit
[`bbff402`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20885
Merged build finished. Test PASSed.
---
-
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/20885
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88540/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20890
Merged build finished. Test PASSed.
---
-
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/20890
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1723/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20890
**[Test build #88543 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88543/testReport)**
for PR 20890 at commit
[`33b692a`](https://github.com/apache/spark/commit/33
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20890
**[Test build #88543 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88543/testReport)**
for PR 20890 at commit
[`33b692a`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20890
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88543/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20890
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20851#discussion_r176762766
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -353,6 +353,13 @@ object SQLConf {
.booleanConf
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20811
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1724/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20811
Merged build finished. Test PASSed.
---
-
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/20811
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/1705/
---
--
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20890
**[Test build #88544 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88544/testReport)**
for PR 20890 at commit
[`4a0a5e3`](https://github.com/apache/spark/commit/4a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20890
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1725/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20890
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/20811
LGTM. @foxish.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h..
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20811
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/1705/
---
Github user cclauss commented on the issue:
https://github.com/apache/spark/pull/20838
@HyukjinKwon Was there something more to do on this one?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For a
Github user sameeragarwal commented on a diff in the pull request:
https://github.com/apache/spark/pull/20884#discussion_r176796998
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/json/JacksonGeneratorSuite.scala
---
@@ -56,7 +56,7 @@ class JacksonGeneratorSuit
Github user liufengdb commented on the issue:
https://github.com/apache/spark/pull/20864
I thought the directory is also created from this line:
https://github.com/apache/spark/blob/master/sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/session/HiveSessionImpl.java#L143
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20883
Merging to master, 2.3, 2.2.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: r
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20883
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20717
**[Test build #88542 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88542/testReport)**
for PR 20717 at commit
[`5cadd86`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20717
Merged build finished. Test PASSed.
---
-
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/20717
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88542/
Test PASSed.
---
Github user gerashegalov commented on a diff in the pull request:
https://github.com/apache/spark/pull/20327#discussion_r176816848
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnClusterSuite.scala
---
@@ -136,6 +135,39 @@ class YarnClusterSuite ex
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20327#discussion_r176817667
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnClusterSuite.scala
---
@@ -136,6 +135,39 @@ class YarnClusterSuite extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20885#discussion_r176820312
--- Diff: python/pyspark/sql/readwriter.py ---
@@ -770,12 +773,15 @@ def json(self, path, mode=None, compression=None,
dateFormat=None, timestampForm
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20885#discussion_r176821110
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala
---
@@ -85,6 +85,38 @@ private[sql] class JSONOptions(
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19876#discussion_r176821060
--- Diff: mllib/src/main/scala/org/apache/spark/ml/util/ReadWrite.scala ---
@@ -86,7 +88,80 @@ private[util] sealed trait BaseReadWrite {
}
/
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19876#discussion_r176821909
--- Diff: mllib/src/main/scala/org/apache/spark/ml/util/ReadWrite.scala ---
@@ -86,7 +88,80 @@ private[util] sealed trait BaseReadWrite {
}
/
Github user gerashegalov commented on a diff in the pull request:
https://github.com/apache/spark/pull/20327#discussion_r176822894
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnClusterSuite.scala
---
@@ -136,6 +135,39 @@ class YarnClusterSuite ex
GitHub user mgaido91 opened a pull request:
https://github.com/apache/spark/pull/20891
[SPARK-23782][CORE][UI] SHS should list only applications with read
permissions
## What changes were proposed in this pull request?
Before the PR, all applications were returned to all us
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20891
Merged build finished. Test PASSed.
---
-
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/20891
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1726/
Tes
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20327#discussion_r176823881
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnClusterSuite.scala
---
@@ -136,6 +135,39 @@ class YarnClusterSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20891
**[Test build #88545 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88545/testReport)**
for PR 20891 at commit
[`bc87945`](https://github.com/apache/spark/commit/bc
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/18982#discussion_r176825209
--- Diff: python/pyspark/ml/wrapper.py ---
@@ -118,11 +118,18 @@ def _transfer_params_to_java(self):
"""
Transforms the embedde
1 - 100 of 219 matches
Mail list logo
