Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/17300
Thanks @cloud-fan @mridulm @gatorsmile
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
en
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/17300
Thanks for merging @cloud-fan, this PR kept dropping form my list ...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17300
thanks, merging to master!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wi
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/17300
Thanks for your review.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17300
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the featur
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17300
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17300
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76502/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17300
**[Test build #76502 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76502/testReport)**
for PR 17300 at commit
[`a3e979f`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17300
**[Test build #76502 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76502/testReport)**
for PR 17300 at commit
[`a3e979f`](https://github.com/apache/spark/commit/a3
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17300
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17300
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76500/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17300
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17300
**[Test build #76500 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76500/testReport)**
for PR 17300 at commit
[`a3e979f`](https://github.com/apache/spark/commit/a
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/17300
That is weird, jenkins should have restarted build when I commented ...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17300
**[Test build #76500 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76500/testReport)**
for PR 17300 at commit
[`a3e979f`](https://github.com/apache/spark/commit/a3
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17300
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/17300
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, o
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17300
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17300
**[Test build #76473 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76473/testReport)**
for PR 17300 at commit
[`56f5231`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17300
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76473/
Test FAILed.
---
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/17300
Thanks both of you for review, I have addressed the comments and modified
the test case. Please help calling jenkins for test, because I can't trigger
that. Thanks again.
---
If your project is
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17300
**[Test build #76473 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76473/testReport)**
for PR 17300 at commit
[`56f5231`](https://github.com/apache/spark/commit/56
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17300
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so,
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/17300
Will merge when tests pass.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wis
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/17300
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17300
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76453/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17300
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17300
**[Test build #76453 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76453/testReport)**
for PR 17300 at commit
[`56f5231`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17300
**[Test build #76453 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76453/testReport)**
for PR 17300 at commit
[`56f5231`](https://github.com/apache/spark/commit/56
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17300
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the fea
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17300
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so,
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/17300
Hi, @cloud-fan @zsxwing Can you take a look? Thanks a lot.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/17300
Ok, thanks a lot.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, o
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/17300
Hi, @jerryshao @mridulm Thanks for your review, I have updated the code.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your proj
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17300
The fix LGTM, I think it is nice to have such topology priority. CC
@mridulm .
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17300
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feat
36 matches
Mail list logo