Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197595389
**[Test build #53354 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53354/consoleFull)**
for PR 11749 at commit
[`0688cf8`](https://g
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11749
---
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 the feature is ena
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197556324
**[Test build #53354 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53354/consoleFull)**
for PR 11749 at commit
[`0688cf8`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197595610
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user nongli commented on a diff in the pull request:
https://github.com/apache/spark/pull/11749#discussion_r56416653
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnarBatch.java
---
@@ -345,15 +358,25 @@ public void setColumn(int ordinal, Co
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197550621
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197505322
**[Test build #53338 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53338/consoleFull)**
for PR 11749 at commit
[`2d1066f`](https://gi
Github user nongli commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197553112
LGTM
---
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 a
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197595608
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197550624
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user sameeragarwal commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197504528
thanks, all comments addressed!
---
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 do
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197550184
**[Test build #53338 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53338/consoleFull)**
for PR 11749 at commit
[`2d1066f`](https://g
Github user sameeragarwal commented on a diff in the pull request:
https://github.com/apache/spark/pull/11749#discussion_r56417640
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnarBatch.java
---
@@ -345,15 +358,25 @@ public void setColumn(int ordi
Github user nongli commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197135193
Can you add some test cases to columnarbatchsuite that exercises this?
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user nongli commented on a diff in the pull request:
https://github.com/apache/spark/pull/11749#discussion_r56277661
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnarBatch.java
---
@@ -58,6 +58,9 @@
// True if the row is filtered.
Github user nongli commented on a diff in the pull request:
https://github.com/apache/spark/pull/11749#discussion_r56277607
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnarBatch.java
---
@@ -284,11 +287,23 @@ public void reset() {
}
Github user nongli commented on a diff in the pull request:
https://github.com/apache/spark/pull/11749#discussion_r56277494
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetReadBenchmark.scala
---
@@ -299,10 +299,112 @@ object ParquetRead
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197123489
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197123483
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197122998
**[Test build #53253 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53253/consoleFull)**
for PR 11749 at commit
[`af217fe`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197093877
**[Test build #53253 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53253/consoleFull)**
for PR 11749 at commit
[`af217fe`](https://gi
Github user sameeragarwal commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197093496
cc @nongli
---
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 fea
GitHub user sameeragarwal opened a pull request:
https://github.com/apache/spark/pull/11749
[SPARK-13922][SQL] Filter rows with null attributes in parquet vectorized
reader
## What changes were proposed in this pull request?
It's common for many SQL operators to not care ab
23 matches
Mail list logo