Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18704
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
@cloud-fan merged with the latest master and addressed your comment for
indent
---
-
To unsubscribe, e-mail: reviews-unsubscr...@s
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
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/18704
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82426/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #82426 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82426/testReport)**
for PR 18704 at commit
[`c16230d`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #82426 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82426/testReport)**
for PR 18704 at commit
[`c16230d`](https://github.com/apache/spark/commit/c1
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
I will rebase this next a few hours.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #82420 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82420/testReport)**
for PR 18704 at commit
[`549b10f`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82420/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
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/18704
**[Test build #82420 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82420/testReport)**
for PR 18704 at commit
[`549b10f`](https://github.com/apache/spark/commit/54
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18704
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18704
LGTM, pending jenkins
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
@cloud-fan could you please review this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
ping @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
@michal-databricks Thank you for your review and comments.
ping @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@s
Github user michal-databricks commented on the issue:
https://github.com/apache/spark/pull/18704
I don't fully understand the big picture here, but I am assuming the goal
is to have more efficient access to the data stored in compressible spark
columnar cache.
Either way this look
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
ping @cloud-fan & @michal-databricks
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
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/18704
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82125/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #82125 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82125/testReport)**
for PR 18704 at commit
[`549b10f`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #82125 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82125/testReport)**
for PR 18704 at commit
[`549b10f`](https://github.com/apache/spark/commit/54
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82124/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #82124 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82124/testReport)**
for PR 18704 at commit
[`b8d5dec`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
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/18704
**[Test build #82124 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82124/testReport)**
for PR 18704 at commit
[`b8d5dec`](https://github.com/apache/spark/commit/b8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
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/18704
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82122/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #82122 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82122/testReport)**
for PR 18704 at commit
[`1607bd1`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #82122 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82122/testReport)**
for PR 18704 at commit
[`1607bd1`](https://github.com/apache/spark/commit/16
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
@michal-databricks do you have any thoughts?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comm
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
@cloud-fan as you proposed before, we will first work for reading table
cache that are frequently executed.
Then, we will work for optimizing columnar table cache building in other
PRs.
---
---
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/18704
cc @michal-databricks any thoughts on this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comman
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
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/18704
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81924/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #81924 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81924/testReport)**
for PR 18704 at commit
[`2902c5b`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #81924 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81924/testReport)**
for PR 18704 at commit
[`2902c5b`](https://github.com/apache/spark/commit/29
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18704
LGTM, I think eventually we should simplify the columnar cache module and
codegen most of it to reduce code duplication.
---
-
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
@cloud-fan Could you please review this again?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional co
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
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/18704
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81883/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #81883 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81883/testReport)**
for PR 18704 at commit
[`bdecaaf`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #81883 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81883/testReport)**
for PR 18704 at commit
[`bdecaaf`](https://github.com/apache/spark/commit/bd
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81778/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
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/18704
**[Test build #81778 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81778/testReport)**
for PR 18704 at commit
[`6be96f8`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #81778 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81778/testReport)**
for PR 18704 at commit
[`6be96f8`](https://github.com/apache/spark/commit/6b
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
ping @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
ping @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
@cloud-fan Resolved conflict, could you please review?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have th
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81295/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #81295 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81295/testReport)**
for PR 18704 at commit
[`097fc05`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #81295 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81295/testReport)**
for PR 18704 at commit
[`097fc05`](https://github.com/apache/spark/commit/09
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
ping @cloud-fan
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
@cloud-fan could you please review this again?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this featur
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18704
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81093/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #81093 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81093/testReport)**
for PR 18704 at commit
[`fb0d4e5`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18704
**[Test build #81093 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81093/testReport)**
for PR 18704 at commit
[`fb0d4e5`](https://github.com/apache/spark/commit/fb
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
@cloud-fan I updated this implementation by using `ColumnVector`, as we
discussed. I would appreciate it if you could discuss two implementations
([on-demand approach](https://github.com/apache/spark/
61 matches
Mail list logo