Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13870
Thank you for merging, @liancheng and @davies .
---
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 liancheng commented on the issue:
https://github.com/apache/spark/pull/13870
LGTM, merging to master and branch-2.0. Thanks!
---
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 f
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13870
No problem! And, thank you for attention! :)
---
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
Github user davies commented on the issue:
https://github.com/apache/spark/pull/13870
@dongjoon-hyun Sorry, I misunderstood it, I thought it was batchStats. The
changes look good.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub a
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13870
Oh, thank you for review, @davies .
But, sorry. I'm not sure what you mean. Do you mean there is a reason that
`readBatches` should be `0` when the option
`spark.sql.inMemoryColumnarS
Github user davies commented on the issue:
https://github.com/apache/spark/pull/13870
@dongjoon-hyun I think that is also used to have better estimation on the
dataset, could be used by planner to have better physical plan.
---
If your project is set up for it, you can reply to this
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13870
Hi, @liancheng .
Could you review this PR, too?
This was initially introduced in your
https://github.com/apache/spark/commit/74049249abb952ad061c0e221c22ff894a9e9c8d#diff-b47dac3d980148
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13870
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61100/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13870
**[Test build #61100 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61100/consoleFull)**
for PR 13870 at commit
[`b1a80dd`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13870
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/13870
**[Test build #61100 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61100/consoleFull)**
for PR 13870 at commit
[`b1a80dd`](https://github.com/apache/spark/commit/b
11 matches
Mail list logo