Github user IgorBerman commented on the issue:
https://github.com/apache/spark/pull/20640
@squito ok, I've cherry-picked @ 2 commits, so the history will contain his
contribution and added on top of it commit with your proposition, hope now it's
ok
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20640
**[Test build #87563 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87563/testReport)**
for PR 20640 at commit
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/17619#discussion_r169450317
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -484,7 +481,6 @@
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20640
**[Test build #87564 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87564/testReport)**
for PR 20640 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20613
**[Test build #87565 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87565/testReport)**
for PR 20613 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20631
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20631
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87561/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20631
**[Test build #87561 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87561/testReport)**
for PR 20631 at commit
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/20604#discussion_r169436456
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -334,6 +336,10 @@ private[spark] class ExecutorAllocationManager(
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/20604#discussion_r169437530
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1643,7 +1646,10 @@ class SparkContext(config: SparkConf) extends
Logging {
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/20604#discussion_r169438419
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -455,7 +461,12 @@ private[spark] class ExecutorAllocationManager(
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20616
Could you review this, @zsxwing and @gatorsmile ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20603
**[Test build #87559 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87559/testReport)**
for PR 20603 at commit
Github user asolimando commented on the issue:
https://github.com/apache/spark/pull/20632
Given that we are converging I have squashed the commits into a single one.
My local `mvn scalastyle:check` was passing (as well as the check done via
the Scala plugin for IntelliiJ)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20640
**[Test build #87562 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87562/testReport)**
for PR 20640 at commit
GitHub user vanzin opened a pull request:
https://github.com/apache/spark/pull/20643
[SPARK-23468][core] Stringify auth secret before storing it in credentials.
The secret is used as a string in many parts of the code, so it has
to be turned into a hex string to avoid issues
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20643
@jerryshao @tgravescs
---
-
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/20640
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20604#discussion_r169452326
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1643,7 +1646,10 @@ class SparkContext(config: SparkConf) extends
Logging {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20640
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87563/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20640
**[Test build #87563 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87563/testReport)**
for PR 20640 at commit
301 - 321 of 321 matches
Mail list logo