Github user dhruve commented on the issue:
https://github.com/apache/spark/pull/22288
Thanks for the reviews and feedback @tgravescs , @squito !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/22288
merged to master and 2.4 branch, thanks @dhruve
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/22288
+1
---
-
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/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98481/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
**[Test build #98481 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98481/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
**[Test build #98481 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98481/testReport)**
for PR 22288 at commit
Github user dhruve commented on the issue:
https://github.com/apache/spark/pull/22288
@squito I have tested it again with both scenarios and I was able to verify
the expected behavior. For the cases that are not covered in the PR, i will
mention them in the jira.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98325/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #98325 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98325/testReport)**
for PR 22288 at commit
Github user squito commented on the issue:
https://github.com/apache/spark/pull/22288
You mentioned in the description that you did some manual testing -- since
this has been through some changes since the initial versions, can you do that
again? please be sure to run some manual
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #98325 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98325/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user dhruve commented on the issue:
https://github.com/apache/spark/pull/22288
@tgravescs I have fixed a nit and its good to be reviewed. @squito I have
updated the comment, let me know if its okay.
Thanks for the reviews.
---
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/22288
@dhruve is this ready to review again?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98107/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #98107 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98107/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
**[Test build #98107 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98107/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97943/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
**[Test build #97943 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97943/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user dhruve commented on the issue:
https://github.com/apache/spark/pull/22288
@squito for the locality wait, it would be the same as the condition where
it is not completely blacklisted. I have added a test for this. If we want to
ensure the sequence for the timeout expiring
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97943 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97943/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97879/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97879 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97879/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
**[Test build #97879 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97879/testReport)**
for PR 22288 at commit
Github user dhruve commented on the issue:
https://github.com/apache/spark/pull/22288
It applies to both DA and SA. I have updated the description.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user squito commented on the issue:
https://github.com/apache/spark/pull/22288
you're still mentioning dynamic allocation in the description, but this
isn't actually dynamic allocation specific, is it?
---
-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97850 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97850/testReport)**
for PR 22288 at commit
Github user dhruve commented on the issue:
https://github.com/apache/spark/pull/22288
@squito I have made the changes and updated the description.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97625/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97625 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97625/testReport)**
for PR 22288 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97625 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97625/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dhruve commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97622/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97622 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97622/testReport)**
for PR 22288 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97622 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97622/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97556/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97556 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97556/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
**[Test build #97556 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97556/testReport)**
for PR 22288 at commit
Github user dhruve commented on the issue:
https://github.com/apache/spark/pull/22288
Failure is unrelated.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dhruve commented on the issue:
https://github.com/apache/spark/pull/22288
retest 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/22288
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/22288
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97542/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97542 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97542/testReport)**
for PR 22288 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97542 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97542/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97499/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97499 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97499/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
**[Test build #97499 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97499/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97469/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97469 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97469/testReport)**
for PR 22288 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97469 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97469/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97208/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
**[Test build #97208 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97208/testReport)**
for PR 22288 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97208 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97208/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97176/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
**[Test build #97176 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97176/testReport)**
for PR 22288 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97176 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97176/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
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/22288
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97168/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #97168 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97168/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
1 - 100 of 136 matches
Mail list logo