Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19695
cc @holdenk and @xynny, mind taking a look please?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For add
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19695
**[Test build #83597 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83597/testReport)**
for PR 19695 at commit
[`c820237`](https://github.com/apache/spark/commit/c8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19695
**[Test build #83597 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83597/testReport)**
for PR 19695 at commit
[`c820237`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19695
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83597/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19695
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19695
I realized that some OSs puts `lsof` into /usr/bin and other OSs put `lsof`
into `/usr/sbin`.
Could you apply this to `dev/run-tests.py`, too?
---
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19695
Ah, for https://github.com/apache/spark/pull/19695#discussion_r149851570
and https://github.com/apache/spark/pull/19695#issuecomment-342865001, actually
the problem here is not the different pat
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19695
**[Test build #83625 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83625/testReport)**
for PR 19695 at commit
[`0dcc12b`](https://github.com/apache/spark/commit/0d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19695
**[Test build #83629 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83629/testReport)**
for PR 19695 at commit
[`a6642fa`](https://github.com/apache/spark/commit/a6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19695
**[Test build #83625 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83625/testReport)**
for PR 19695 at commit
[`0dcc12b`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19695
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/19695
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83625/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19695
**[Test build #83629 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83629/testReport)**
for PR 19695 at commit
[`a6642fa`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19695
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/19695
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83629/
Test FAILed.
---
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19695
Jenkins, retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19695
**[Test build #83638 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83638/testReport)**
for PR 19695 at commit
[`a6642fa`](https://github.com/apache/spark/commit/a6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19695
**[Test build #83638 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83638/testReport)**
for PR 19695 at commit
[`a6642fa`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19695
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/19695
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83638/
Test PASSed.
---
Github user xynny commented on the issue:
https://github.com/apache/spark/pull/19695
The script looks fine to me, but I think it's just still a problem that all
the jenkins slaves are actually running different OSes. It's just
inconsistent.
---
--
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19695
Let's go ahead with this first and make a followup if there is another
problem can be found, if you guys here wouldn't mind.
---
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19695
Just rebased and fixed the comments a little bit more. Let me merge this
one once it passes the tests.
---
-
To unsubscribe,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19695
**[Test build #83765 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83765/testReport)**
for PR 19695 at commit
[`5a513ac`](https://github.com/apache/spark/commit/5a
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/19695
LGTM pending jenkins.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19695
**[Test build #83765 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83765/testReport)**
for PR 19695 at commit
[`5a513ac`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19695
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/19695
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83765/
Test PASSed.
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19695
@felixcheung, I am about to merge this one to master, branch-2.2 and
branch-2.1. Would it be all okay to your release preparation?
---
-
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19695
Sorry I didnât see it before tagging
Please go ahead
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19695
Merged to master, branch-2.2 and branch-2.1.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additiona
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19695
Let me keep my eyes on the builds.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19695
Oh BTW thank you all for your review / comments, @srowen, @kiszk, @viirya,
@holdenk, @xynny and @felixcheung.
---
-
To unsub
33 matches
Mail list logo