Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/4156
---
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 enab
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71583576
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71583569
[Test build #26137 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26137/consoleFull)
for PR 4156 at commit
[`a403979`](https://gith
Github user saucam commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71578360
fixed the styling issues. @liancheng thanks for the feedback!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71578336
[Test build #26137 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26137/consoleFull)
for PR 4156 at commit
[`a403979`](https://githu
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71564983
LGTM except for several minor styling issue. Thanks for working on this!
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/4156#discussion_r23576414
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/parquet/ParquetFilterSuite.scala
---
@@ -91,6 +91,32 @@ class ParquetFilterSuite extends QueryTest w
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/4156#discussion_r23576409
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/parquet/ParquetFilterSuite.scala
---
@@ -21,10 +21,10 @@ import parquet.filter2.predicate.Operators.
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/4156#discussion_r23576410
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/parquet/ParquetFilterSuite.scala
---
@@ -91,6 +91,32 @@ class ParquetFilterSuite extends QueryTest w
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/4156#discussion_r23576413
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/parquet/ParquetFilterSuite.scala
---
@@ -91,6 +91,32 @@ class ParquetFilterSuite extends QueryTest w
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71436639
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71436629
[Test build #26090 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26090/consoleFull)
for PR 4156 at commit
[`d029866`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71429249
[Test build #26090 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26090/consoleFull)
for PR 4156 at commit
[`d029866`](https://githu
Github user saucam commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71428920
Added test case
---
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
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71400636
Hi @saucam, would you please add tests for this PR? Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71336956
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71336955
[Test build #26055 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26055/consoleFull)
for PR 4156 at commit
[`cb2e0d9`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71334179
[Test build #26055 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26055/consoleFull)
for PR 4156 at commit
[`cb2e0d9`](https://githu
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-71334132
ok to test
---
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
ena
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-70976504
Can one of the admins verify this patch?
---
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 pro
Github user saucam commented on the pull request:
https://github.com/apache/spark/pull/4156#issuecomment-70976498
done
---
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
21 matches
Mail list logo
