Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/19285
thanks all.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spa
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19285
thanks, merging 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/19285
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86674/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86674 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86674/testReport)**
for PR 19285 at commit
[`9d1aeef`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86674 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86674/testReport)**
for PR 19285 at commit
[`9d1aeef`](https://github.com/apache/spark/commit/9d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
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/19285
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86634/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86634 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86634/testReport)**
for PR 19285 at commit
[`40bdcac`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86634 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86634/testReport)**
for PR 19285 at commit
[`40bdcac`](https://github.com/apache/spark/commit/40
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19285
retest this please
---
-
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/19285
**[Test build #86629 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86629/testReport)**
for PR 19285 at commit
[`9e0759f`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86630/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
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/19285
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/19285
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86629/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86630 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86630/testReport)**
for PR 19285 at commit
[`40bdcac`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86630 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86630/testReport)**
for PR 19285 at commit
[`40bdcac`](https://github.com/apache/spark/commit/40
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86629 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86629/testReport)**
for PR 19285 at commit
[`9e0759f`](https://github.com/apache/spark/commit/9e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86619/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86619 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86619/testReport)**
for PR 19285 at commit
[`ded080d`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86620/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86620 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86620/testReport)**
for PR 19285 at commit
[`b41f1bb`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86620 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86620/testReport)**
for PR 19285 at commit
[`b41f1bb`](https://github.com/apache/spark/commit/b4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86619 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86619/testReport)**
for PR 19285 at commit
[`ded080d`](https://github.com/apache/spark/commit/de
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86565 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86565/testReport)**
for PR 19285 at commit
[`f392217`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86565/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86565 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86565/testReport)**
for PR 19285 at commit
[`f392217`](https://github.com/apache/spark/commit/f3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86557/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86557 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86557/testReport)**
for PR 19285 at commit
[`c988762`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86557 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86557/testReport)**
for PR 19285 at commit
[`c988762`](https://github.com/apache/spark/commit/c9
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/19285
Thanks for your valuable suggestion, the code has been updated.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
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/19285
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86455/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86455 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86455/testReport)**
for PR 19285 at commit
[`c442494`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86455 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86455/testReport)**
for PR 19285 at commit
[`c442494`](https://github.com/apache/spark/commit/c4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
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/19285
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86429/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86429 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86429/testReport)**
for PR 19285 at commit
[`9d7c52d`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86427/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86427 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86427/testReport)**
for PR 19285 at commit
[`a2b9513`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86427 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86427/testReport)**
for PR 19285 at commit
[`a2b9513`](https://github.com/apache/spark/commit/a2
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/19285
Thanks for reviewing. The code has updated, pls help to review. Thanks
again.
---
-
To unsubscribe, e-mail: reviews-unsubscr...
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86429 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86429/testReport)**
for PR 19285 at commit
[`9d7c52d`](https://github.com/apache/spark/commit/9d
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19285
It's just a refactor so I'd like to target it for 2.4
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For ad
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19285
Are we targeting this to 2.3 or 2.4?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19285
overall looks good
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
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/19285
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83575/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #83575 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83575/testReport)**
for PR 19285 at commit
[`bc3ad4e`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #83575 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83575/testReport)**
for PR 19285 at commit
[`bc3ad4e`](https://github.com/apache/spark/commit/bc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83527/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #83527 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83527/testReport)**
for PR 19285 at commit
[`3a90ad1`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #83527 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83527/testReport)**
for PR 19285 at commit
[`3a90ad1`](https://github.com/apache/spark/commit/3a
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/19285
It's updated. Thanks a lot.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19285
@ConeyLiu Could you rebase this with the latest master so we can continue
review it? Thanks!
---
-
To unsubscribe, e-mail: r
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #82084 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82084/testReport)**
for PR 19285 at commit
[`d0fcf4f`](https://github.com/apache/spark/commit/d0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
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/19285
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81981/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #81981 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81981/testReport)**
for PR 19285 at commit
[`9ea8f49`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #81981 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81981/testReport)**
for PR 19285 at commit
[`9ea8f49`](https://github.com/apache/spark/commit/9e
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19285
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81971/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #81971 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81971/testReport)**
for PR 19285 at commit
[`9ea8f49`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #81971 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81971/testReport)**
for PR 19285 at commit
[`9ea8f49`](https://github.com/apache/spark/commit/9e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
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/19285
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81961/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #81961 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81961/testReport)**
for PR 19285 at commit
[`d2b8ccd`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #81961 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81961/testReport)**
for PR 19285 at commit
[`d2b8ccd`](https://github.com/apache/spark/commit/d2
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19285
ok to test.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/19285
Hi @cloud-fan @jiangxb1987 , would you mind take a look ? Thanks a lot.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.
82 matches
Mail list logo