Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19524
Merged to master.
---
-
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/19524
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82918/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19524
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/19524
**[Test build #82918 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82918/testReport)**
for PR 19524 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19524
**[Test build #82918 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82918/testReport)**
for PR 19524 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19524
Thanks for your review @shaneknapp.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19524
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/19524
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82900/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19524
**[Test build #82900 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82900/testReport)**
for PR 19524 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19524
**[Test build #82900 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82900/testReport)**
for PR 19524 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19524
Sure, thanks. Let me update soon.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user shaneknapp commented on the issue:
https://github.com/apache/spark/pull/19524
yeah, now that i'm looking at this w/a fresh cup of coffee, i agree w/
@holdenk about shellutil.py being a weird place to exit for a python version
check.
run-tests.py and
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19524
@holdenk, hm, should I maybe make add this to `run-tests.py` and
`run-tests-jenkins.py`? I wasn't sure where I should put this.
---
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/19524
Looks reasonable, my only thing is putting in a forced exited inside of
shellutil.py is sort of a random place to the version check.
---
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/19524
seems fine.
---
-
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/19524
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/19524
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82871/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19524
**[Test build #82871 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82871/testReport)**
for PR 19524 at commit
Github user shaneknapp commented on the issue:
https://github.com/apache/spark/pull/19524
yeah, this makes sense. i don't think we've officially supported 2.6 in a
while, esp for tests and i'd be ok w/removing the backports. this makes for a
much clearer exit case.
@rxin
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19524
**[Test build #82871 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82871/testReport)**
for PR 19524 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19524
cc @JoshRosen, @holdenk and @shaneknapp, could you take a look and see if
makes sense please?
---
-
To unsubscribe, e-mail:
21 matches
Mail list logo