Github user gczsjdy commented on the issue:
https://github.com/apache/spark/pull/19862
@cloud-fan Ok, thanks for your time, I will close this.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19862
I don't agree this is a small change, and users using spark prior to 2.0
won't get this patch, as we don't backport performance improvement patches.
Overall this patch won't bring much
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19862
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/19862
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/19862
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gczsjdy commented on the issue:
https://github.com/apache/spark/pull/19862
cc @cloud-fan @hvanhovell @viirya
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user gczsjdy commented on the issue:
https://github.com/apache/spark/pull/19862
This is actually a small change, but it can provide not small optimization
for users who don't use `WholeStageCodegen`, for example there're still some
users who use under 2.0 version of Spark.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19862
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/19862
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84842/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19862
**[Test build #84842 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84842/testReport)**
for PR 19862 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19862
**[Test build #84842 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84842/testReport)**
for PR 19862 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19862
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/19862
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84836/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19862
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84840/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19862
**[Test build #84840 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84840/testReport)**
for PR 19862 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19862
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/19862
**[Test build #84836 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84836/testReport)**
for PR 19862 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19862
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84839/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19862
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/19862
**[Test build #84839 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84839/testReport)**
for PR 19862 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19862
**[Test build #84840 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84840/testReport)**
for PR 19862 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19862
**[Test build #84839 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84839/testReport)**
for PR 19862 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19862
**[Test build #84836 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84836/testReport)**
for PR 19862 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19862
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/19862
**[Test build #84835 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84835/testReport)**
for PR 19862 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19862
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84835/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19862
**[Test build #84835 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84835/testReport)**
for PR 19862 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19862
**[Test build #84471 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84471/testReport)**
for PR 19862 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19862
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84471/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19862
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19862
I'm not sure we should spend time to optimize non-whole-stage-codegen path,
which is only used as a safe fallback if whole stage codegen is not applicable.
I think the more important thing to do
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19862
**[Test build #84471 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84471/testReport)**
for PR 19862 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19862
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
33 matches
Mail list logo
