Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/22066
@cloud-fan , yeah, I will include it in 2.3.2.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additiona
Github user LantaoJin commented on the issue:
https://github.com/apache/spark/pull/22066
Thank you @yucai . New PR #22077 for branch-2.3. Cc: @cloud-fan @jerryshao
---
-
To unsubscribe, e-mail: reviews-unsubscr...@s
Github user yucai commented on the issue:
https://github.com/apache/spark/pull/22066
@cloud-fan Synced with @LantaoJin he will help port to 2.3 soon and I will
review it.
---
-
To unsubscribe, e-mail: reviews-unsubs
Github user yucai commented on the issue:
https://github.com/apache/spark/pull/22066
@cloud-fan @jerryshao sure, I will do it.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional command
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22066
cc @jerryshao this is a regression for 2.3, we should have it in 2.3.2
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22066
thanks, merging to master!
@yucai can you send a new PR for 2.3? We did a lot of codegen changes in
2.4 so it's safer to make sure we pass all the tests in 2.3.
---
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22066
> @viirya is that effort going on? I can help with the work if you want.
Thanks.
@mgaido91 Yeah, I'm still working on it. One of the PRs #21537 is still
waiting for review.
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22066
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94564/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22066
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/22066
**[Test build #94564 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94564/testReport)**
for PR 22066 at commit
[`5da6d4d`](https://github.com/apache/spark/commit/5
Github user yucai commented on the issue:
https://github.com/apache/spark/pull/22066
@LantaoJin I realized the initial way had some issue, so I marked it as WIP
to refine and add test. It is different from your original implementation, so I
would like to use this one.
---
-
Github user yucai commented on the issue:
https://github.com/apache/spark/pull/22066
@cloud-fan Jira and 1st is from this one. It is critical to our 2.3
migration.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@s
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22066
This PR looks identical to #22067 , which one is the first PR?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22066
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/22066
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94557/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22066
**[Test build #94557 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94557/testReport)**
for PR 22066 at commit
[`931fa28`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22066
**[Test build #94564 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94564/testReport)**
for PR 22066 at commit
[`5da6d4d`](https://github.com/apache/spark/commit/5d
Github user LantaoJin commented on the issue:
https://github.com/apache/spark/pull/22066
Since you refactor your code copying from #22067 . Would you mind just use
that?
---
-
To unsubscribe, e-mail: reviews-unsubsc
Github user yucai commented on the issue:
https://github.com/apache/spark/pull/22066
@cloud-fan @gatorsmile PR has been ready, kindly help review.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Fo
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22066
**[Test build #94557 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94557/testReport)**
for PR 22066 at commit
[`931fa28`](https://github.com/apache/spark/commit/93
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22066
**[Test build #94543 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94543/testReport)**
for PR 22066 at commit
[`8ee56bb`](https://github.com/apache/spark/commit/8e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22066
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/22066
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
23 matches
Mail list logo