Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20288
@gatorsmile, I am sorry i don't know why I missed this comment ..
---
-
To unsubscribe, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20288
LGTM except one minor comment.
Could you submit a follow-up PR?
---
-
To unsubscribe, e-mail:
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/20288
Thanks! merging to master/2.3.
---
-
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/20288
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/20288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86309/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86309 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86309/testReport)**
for PR 20288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
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/20288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86306/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86308/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
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/20288
**[Test build #86306 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86306/testReport)**
for PR 20288 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86308 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86308/testReport)**
for PR 20288 at commit
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/20288
LGTM pending Jenkins.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20288
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86309 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86309/testReport)**
for PR 20288 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86308 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86308/testReport)**
for PR 20288 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86306 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86306/testReport)**
for PR 20288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86274/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
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/20288
**[Test build #86274 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86274/testReport)**
for PR 20288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
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/20288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86273/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86273 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86273/testReport)**
for PR 20288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
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/20288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86265/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86265 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86265/testReport)**
for PR 20288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86264/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
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/20288
**[Test build #86264 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86264/testReport)**
for PR 20288 at commit
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20288
LGTM
---
-
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/20288
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86266/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
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/20288
**[Test build #86266 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86266/testReport)**
for PR 20288 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86274 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86274/testReport)**
for PR 20288 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20288
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20288
Just fixed minor doc nits and double checked the built API documentation.
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86273 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86273/testReport)**
for PR 20288 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20288
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/20288
LGTM pending Jenkins.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86266 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86266/testReport)**
for PR 20288 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86265 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86265/testReport)**
for PR 20288 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86264 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86264/testReport)**
for PR 20288 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20288
Will try to double check and clean up soon. Comments so far above look all
valid.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86247/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
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/20288
**[Test build #86247 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86247/testReport)**
for PR 20288 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86247 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86247/testReport)**
for PR 20288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
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/20288
**[Test build #86234 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86234/testReport)**
for PR 20288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86234/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20288
**[Test build #86234 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86234/testReport)**
for PR 20288 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20288
@ueshin and @icexelloss (docstring reassignment) and @cloud-fan
(deprecation), could you guys take a look and see if I understood your
suggestions correctly?
---
52 matches
Mail list logo
