Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23262
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/23262
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99903/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23262
**[Test build #99903 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99903/testReport)**
for PR 23262 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23262
LGTM, can you update the PR title and description?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user eatoncys commented on the issue:
https://github.com/apache/spark/pull/23262
@HyukjinKwon Ok, removed it, thanks for review.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23262
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/23262
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99899/
Test FAILed.
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23262
Let's remove. No point of keeping unused method. The code will remain in
the commit anyway. Also, there's no quite good point of keeping few lines
method that's called only at one place.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23262
**[Test build #99899 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99899/testReport)**
for PR 23262 at commit
Github user eatoncys commented on the issue:
https://github.com/apache/spark/pull/23262
@HyukjinKwon @mgaido91 Thanks for review. @cloud-fan @kiszk Would you like
to give some suggestions: remove the object `RDDConversions` , or leave it
there?
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23262
LGTM otheriwse
---
-
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/23262
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/23262
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/23262
**[Test build #99903 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99903/testReport)**
for PR 23262 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23262
**[Test build #99899 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99899/testReport)**
for PR 23262 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23262
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/23262
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/23262
Good catch, LGTM
cc @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23262
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99866/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23262
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/23262
**[Test build #99866 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99866/testReport)**
for PR 23262 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23262
**[Test build #99866 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99866/testReport)**
for PR 23262 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23262
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/23262
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user eatoncys commented on the issue:
https://github.com/apache/spark/pull/23262
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/23262
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99861/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23262
**[Test build #99861 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99861/testReport)**
for PR 23262 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23262
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/23262
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23262
**[Test build #99861 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99861/testReport)**
for PR 23262 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23262
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
31 matches
Mail list logo
