Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
merged to master, thanks @hthuynh2
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
+1
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apach
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21653
**[Test build #4222 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4222/testReport)**
for PR 21653 at commit
[`b6585da`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21653
**[Test build #4222 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4222/testReport)**
for PR 21653 at commit
[`b6585da`](https://github.com/apache/spark/commit/b
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21653
I kicked off the test manually at
https://spark-prs.appspot.com/users/hthuynh2. I dunno why the test triggering
via comments stops workign on some prs
---
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93474/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21653
**[Test build #93474 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93474/testReport)**
for PR 21653 at commit
[`b6585da`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
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/21653
**[Test build #93474 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93474/testReport)**
for PR 21653 at commit
[`b6585da`](https://github.com/apache/spark/commit/b6
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21653
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93447/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
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/21653
**[Test build #93447 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93447/testReport)**
for PR 21653 at commit
[`b6585da`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21653
**[Test build #93447 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93447/testReport)**
for PR 21653 at commit
[`b6585da`](https://github.com/apache/spark/commit/b6
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user hthuynh2 commented on the issue:
https://github.com/apache/spark/pull/21653
@tgravescs Can you please run the test again, thank you.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
Merged build finished. Test FAILed.
---
-
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/21653
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93418/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21653
**[Test build #93418 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93418/testReport)**
for PR 21653 at commit
[`b6585da`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21653
**[Test build #93418 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93418/testReport)**
for PR 21653 at commit
[`b6585da`](https://github.com/apache/spark/commit/b6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93302/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user hthuynh2 commented on the issue:
https://github.com/apache/spark/pull/21653
@tgravescs I updated it. Can you please have a look at it when you have
time. Thank you.
---
-
To unsubscribe, e-mail: reviews-
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
#21729 has been merged @hthuynh2 can you update this one?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
F
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
lets review https://github.com/apache/spark/pull/21729 before this since
its changing the type on killedByOtherAttempt
---
-
T
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92690/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
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/21653
**[Test build #92690 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92690/testReport)**
for PR 21653 at commit
[`3c655f2`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21653
**[Test build #92690 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92690/testReport)**
for PR 21653 at commit
[`3c655f2`](https://github.com/apache/spark/commit/3c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92688/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21653
**[Test build #92688 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92688/testReport)**
for PR 21653 at commit
[`f66fcab`](https://github.com/apache/spark/commit/f
Github user hthuynh2 commented on the issue:
https://github.com/apache/spark/pull/21653
@squito Thanks for the suggestions. I updated it. Could you please have a
look at it to see if there is anything else I need to change? Thanks.
---
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
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/21653
**[Test build #92688 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92688/testReport)**
for PR 21653 at commit
[`f66fcab`](https://github.com/apache/spark/commit/f6
Github user hthuynh2 commented on the issue:
https://github.com/apache/spark/pull/21653
@jiangxb1987 yes, you are correct that it is actually ignored. I think it
doesn't worth to add a new TaskState because we might need to add changes in
many places but does not add much benefit. Ins
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21653
IIUC this speculative task is not really killed right ? It is actually
ignored. Does that worth it to add a new TaskState for this case ?
---
--
Github user hthuynh2 commented on the issue:
https://github.com/apache/spark/pull/21653
I updated it. Thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
@hthuynh2 can you fix the description "as success of failures" , this is
just a copy of my typo in the jira. Can you just change to be "as success"
---
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
+1, changes look good to me.
@squito see any problems with this approach?
---
-
To unsubscribe, e-mail: reviews-uns
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
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/21653
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92426/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21653
**[Test build #92426 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92426/testReport)**
for PR 21653 at commit
[`980a933`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21653
**[Test build #92426 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92426/testReport)**
for PR 21653 at commit
[`980a933`](https://github.com/apache/spark/commit/98
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
Merged build finished. Test FAILed.
---
-
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/21653
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92425/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21653
**[Test build #92425 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92425/testReport)**
for PR 21653 at commit
[`8f7d981`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21653
**[Test build #92425 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92425/testReport)**
for PR 21653 at commit
[`8f7d981`](https://github.com/apache/spark/commit/8f
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21653
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/21653
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/21653
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user hthuynh2 commented on the issue:
https://github.com/apache/spark/pull/21653
cc @tgravescs
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@
56 matches
Mail list logo
