Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/21447
I want to give a follow up PR and cc @gatorsmile @maropu for a review.
---
-
To unsubscribe, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21447
@maropu Do you want to take this over and add such a project in
`ColumnPruning`?
---
-
To unsubscribe, e-mail:
Github user xdcjie commented on the issue:
https://github.com/apache/spark/pull/21447
Okï¼
---
-
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/21447
@xdcjie, mind closing this one?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21447
Sorry, the fix does not look good to me. We should let the optimizer add
the project automatically.
---
-
To unsubscribe,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21447
Yea, usually we do it for master first and then backport it to other
branches to reduce the diff and master has the fix. I would appreciate it if
you go in this way.
---
Github user xdcjie commented on the issue:
https://github.com/apache/spark/pull/21447
@HyukjinKwon Our project is based on the branch-2.2, we will merge the
patch to local branch manually if it is against master. we prefer following
community to local branch.
If you don't
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21447
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/21447
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93103/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21447
**[Test build #93103 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93103/testReport)**
for PR 21447 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21447
Wait, why is it against branch-2.2 not master?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21447
**[Test build #93103 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93103/testReport)**
for PR 21447 at commit
Github user xdcjie commented on the issue:
https://github.com/apache/spark/pull/21447
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user xdcjie commented on the issue:
https://github.com/apache/spark/pull/21447
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/21447
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/21447
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92916/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21447
**[Test build #92916 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92916/testReport)**
for PR 21447 at commit
Github user xdcjie commented on the issue:
https://github.com/apache/spark/pull/21447
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/21447
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92842/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21447
**[Test build #92842 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92842/testReport)**
for PR 21447 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21447
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/21447
**[Test build #92842 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92842/testReport)**
for PR 21447 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21447
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21447
kindly pining @gatorsmile @HyukjinKwon @ueshin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21447
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user xdcjie commented on the issue:
https://github.com/apache/spark/pull/21447
@maropu @gatorsmile Can you help me review this pr ? Thanksã #
---
-
To unsubscribe, e-mail:
Github user xdcjie commented on the issue:
https://github.com/apache/spark/pull/21447
@maropu @gatorsmile Do you have any comment/suggestion for this PR? Thanks.
---
-
To unsubscribe, e-mail:
Github user xdcjie commented on the issue:
https://github.com/apache/spark/pull/21447
jenkins, test this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user xdcjie commented on the issue:
https://github.com/apache/spark/pull/21447
@maropu I updated the commet. In summary, with this pr can reduce the time
of scan and assemble data. In our scenario, the relation(table) have 700
columns.
---
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21447
Could you add explain result differences with/without this pr in the
description?
---
-
To unsubscribe, e-mail:
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21447
@gatorsmile Can you trigger this?
---
-
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/21447
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/21447
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
33 matches
Mail list logo