Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19543
thanks @susanxhuynh
merged this to master.
the cherry pick to 2.2 was non-trivial, but merged, please keep an eye for
this and please help test 2.2.1!
---
Github user susanxhuynh commented on the issue:
https://github.com/apache/spark/pull/19543
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19543
@susanxhuynh, @ttashi-rms, @ArtRand please take a look
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19543
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/19543
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83718/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19543
**[Test build #83718 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83718/testReport)**
for PR 19543 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19543
**[Test build #83718 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83718/testReport)**
for PR 19543 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19543
add to white list
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19543
Jenkins, retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user pmackles commented on the issue:
https://github.com/apache/spark/pull/19543
@susanxhuynh, @ttashi-rms, @ArtRand - thanks for the feedback. I was able
to make all of the suggested changes.
---
-
To
Github user ttashi-rms commented on the issue:
https://github.com/apache/spark/pull/19543
@pmackles your changes works great. we had to update SJS (spark-job-server)
side to support the new param for dispatcher. Ended up updating the
manager_start.sh incase anyone is trying to figure
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19543
@susanxhuynh or anyone from the mesos side would you please review?
---
-
To unsubscribe, e-mail:
Github user pmackles commented on the issue:
https://github.com/apache/spark/pull/19543
@felixcheung - any chance of getting this merged into the upcoming 2.2.1
release? I cleaned up the merge conflict
---
-
To
Github user pmackles commented on the issue:
https://github.com/apache/spark/pull/19543
@ttashi-rms v2.2 and up
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user ttashi-rms commented on the issue:
https://github.com/apache/spark/pull/19543
which release version is this intended for ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user ArtRand commented on the issue:
https://github.com/apache/spark/pull/19543
LGTM.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user ArtRand commented on the issue:
https://github.com/apache/spark/pull/19543
Just one super-nit.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user pmackles commented on the issue:
https://github.com/apache/spark/pull/19543
@ArtRand - thanks for the feedback. I made the suggested changes to the
docs and added some more tests - including a test w/ multiple constraints
---
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19543
@susanxhuynh would you like to review?
also looks like @ArtRand has some comments
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19543
**[Test build #82963 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82963/testReport)**
for PR 19543 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19543
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82963/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19543
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19543
**[Test build #82963 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82963/testReport)**
for PR 19543 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19543
Jenkins, retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user pmackles commented on the issue:
https://github.com/apache/spark/pull/19543
My apologies. Reread the docs and ran style checks locally this time.
Fingers crossed
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19543
**[Test build #82946 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82946/testReport)**
for PR 19543 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19543
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19543
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82946/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19543
**[Test build #82946 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82946/testReport)**
for PR 19543 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19543
Jenkins, retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user pmackles commented on the issue:
https://github.com/apache/spark/pull/19543
@felixcheung - fixed scala-style issues and also updated the docs to
include the new property
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19543
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19543
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82937/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19543
**[Test build #82937 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82937/testReport)**
for PR 19543 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19543
**[Test build #82937 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82937/testReport)**
for PR 19543 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19543
test this please
---
-
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/19543
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
37 matches
Mail list logo