Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/8070#issuecomment-129703530
Merged to master and branch-1.5.
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8070
---
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
Github user dguy commented on the pull request:
https://github.com/apache/spark/pull/8070#issuecomment-129559546
LGTM - working as expected.
---
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
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/8070#discussion_r36676679
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/parquet/CatalystRowConverter.scala
---
@@ -446,4 +478,61 @@ private[parquet] class
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8070#issuecomment-129590965
[Test build #40304 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40304/console)
for PR 8070 at commit
Github user rdblue commented on the pull request:
https://github.com/apache/spark/pull/8070#issuecomment-129649565
This looks good to me overall.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8070#issuecomment-129651534
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8070#issuecomment-129651523
Merged build triggered.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8070#issuecomment-129651564
[Test build #40355 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40355/consoleFull)
for PR 8070 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8070#issuecomment-129591067
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8070#issuecomment-129684756
[Test build #40355 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40355/console)
for PR 8070 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8070#issuecomment-129684809
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
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/8070#discussion_r36700528
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/parquet/CatalystRowConverter.scala
---
@@ -446,4 +478,61 @@ private[parquet] class
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/8070#issuecomment-129658453
@dguy @rdblue Thanks for the review! I just rebased this PR to resolve
conflicts introduced by #8056. Will merge this pending Jenkins.
---
If your project is set up
14 matches
Mail list logo