Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20878
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20831
thanks, merging to master/2.3!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20851#discussion_r176636849
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -353,6 +353,13 @@ object SQLConf {
.booleanConf
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20831
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20861
@hvanhovell Ok. But this needs #20817. Since #20817 just adds new class and
doesn't change existing code, I think it can be directly merged into 2.3.
Should I create a backport PR of it too? Or you c
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20831
Thanks! @cloud-fan
---
-
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/19840
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/1718/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19840
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20633
@jkbradley Thanks for your comments! I've addressed them. Please review it
again. Thank you.
---
-
To unsubscribe, e-mail: review
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/20888
I mean on my machine the stage ID is zero for long long time here:
```
DataFrameRangeSuite.stageToKill = TaskContext.get().stageId()
```
and after 200 seconds the other t
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/20888
Uploaded logs for the jira. You're right when you pointed out a second
issue with the `stageToKill`. The `onJobStart` tries to cancel the same ID
twice.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20887
**[Test build #88535 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88535/testReport)**
for PR 20887 at commit
[`c6d12e8`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20887
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88535/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20887
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/20657
Thanks, merging to master branch!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
GitHub user Lemonjing opened a pull request:
https://github.com/apache/spark/pull/20889
[MINOR][DOC] Fix ml-guide markdown typos
## What changes were proposed in this pull request?
Easy fix in the markdown.
## How was this patch tested?
Tested manually
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20889
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20889
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20657
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20879
thanks! those are on me then.
are we not running lintr? or these checks are on the newer version only?
---
-
To unsubscr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20888
**[Test build #88536 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88536/testReport)**
for PR 20888 at commit
[`42c930d`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20888
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/20888
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88536/
Test PASSed.
---
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/20873#discussion_r176646160
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -776,6 +776,9 @@ object SparkSubmit extends CommandLineUtils with
Logging {
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20860#discussion_r176646535
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/IsolatedClientLoader.scala
---
@@ -179,7 +179,7 @@ private[hive] class IsolatedClien
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20864
Jenkins, ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20889
can you check if we have other typo in the md file?
or similar type of typo in other md files?
---
-
To unsubscribe, e-ma
Github user LantaoJin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20873#discussion_r176647022
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -776,6 +776,9 @@ object SparkSubmit extends CommandLineUtils with
Logging {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20864
**[Test build #88537 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88537/testReport)**
for PR 20864 at commit
[`b177bf4`](https://github.com/apache/spark/commit/b1
Github user LantaoJin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20873#discussion_r176647523
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -776,6 +776,9 @@ object SparkSubmit extends CommandLineUtils with
Logging {
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/20873#discussion_r176647902
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -776,6 +776,9 @@ object SparkSubmit extends CommandLineUtils with
Logging {
Github user Lemonjing commented on the issue:
https://github.com/apache/spark/pull/20889
@felixcheung Yes, I read the entire docs of Spark Mlib. There is no problem
with other mds (I may not have seen it or other contributors solved it). This
error is obvious, and it affects the link
Github user LantaoJin closed the pull request at:
https://github.com/apache/spark/pull/20873
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user LantaoJin commented on the issue:
https://github.com/apache/spark/pull/20873
Close it while found a work around way.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20771
ping @hvanhovell
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
301 - 335 of 335 matches
Mail list logo