Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1959#issuecomment-53361811
Thanks! I've merged this into master and 1.1.
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/1959
---
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 liancheng commented on the pull request:
https://github.com/apache/spark/pull/1959#issuecomment-53180216
LGTM, 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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1959#issuecomment-53144788
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19096/consoleFull)
for PR 1959 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1959#issuecomment-53146504
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19096/consoleFull)
for PR 1959 at commit
Github user chutium commented on a diff in the pull request:
https://github.com/apache/spark/pull/1959#discussion_r16589482
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/parquet/ParquetTypes.scala ---
@@ -373,9 +373,11 @@ private[parquet] object ParquetTypesConverter
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/1959#discussion_r16628843
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/parquet/ParquetTypes.scala ---
@@ -373,9 +373,11 @@ private[parquet] object ParquetTypesConverter
Github user chutium commented on a diff in the pull request:
https://github.com/apache/spark/pull/1959#discussion_r16527681
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/parquet/ParquetTypes.scala ---
@@ -373,9 +373,11 @@ private[parquet] object ParquetTypesConverter
Github user chutium commented on a diff in the pull request:
https://github.com/apache/spark/pull/1959#discussion_r16530668
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/parquet/ParquetTypes.scala ---
@@ -373,9 +373,11 @@ private[parquet] object ParquetTypesConverter
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/1959#discussion_r16582511
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/parquet/ParquetTypes.scala ---
@@ -373,9 +373,11 @@ private[parquet] object ParquetTypesConverter
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/1959#discussion_r16284043
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/parquet/ParquetTypes.scala ---
@@ -373,9 +373,11 @@ private[parquet] object ParquetTypesConverter
Github user joesu commented on a diff in the pull request:
https://github.com/apache/spark/pull/1959#discussion_r16299347
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/parquet/ParquetTypes.scala ---
@@ -373,9 +373,11 @@ private[parquet] object ParquetTypesConverter
Github user joesu commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52147641
code fixed.
---
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52147673
test this please
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52147949
QA tests have started for PR 1924. This patch merges cleanly. brView
progress:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18526/consoleFull
---
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52152170
QA results for PR 1924:br- This patch FAILED unit tests.br- This patch
merges cleanlybr- This patch adds no public classesbrbrFor more
information see test
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52216775
The test failures look unrelated.
I've merge this to master and 1.1. Thanks!
---
If your project is set up for it, you can reply to this email and have your
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/1924
---
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 tgravescs commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52232598
I'm seeing build failures from this change on hadoop 0.23. I haven't had
time to dig into it.
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52234201
I'll revert this change.
---
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 tgravescs commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52236703
Thanks, I took a quick look to see what the problem is and it appears that
was removed in hadoop 0.23 and then put back in in hadoop 2.x to be compatible
with hadoop
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52236970
We could just hard code the string _temporary with a note about why.
On Aug 14, 2014 1:07 PM, Tom Graves notificati...@github.com wrote:
Thanks, I took a
GitHub user joesu opened a pull request:
https://github.com/apache/spark/pull/1959
[SPARK-3011][SQL] _temporary directory should be filtered out by
sqlContext.parquetFile
fix compile error on hadoop 0.23 for the pull request #1924.
You can merge this pull request into a Git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1959#issuecomment-52262477
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
Github user joesu commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52262657
Submitted another pull request: https://github.com/apache/spark/pull/1959
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1959#issuecomment-52263111
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1959#issuecomment-52263383
QA tests have started for PR 1959. This patch merges cleanly. brView
progress:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18586/consoleFull
---
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1959#issuecomment-52267094
QA results for PR 1959:br- This patch FAILED unit tests.br- This patch
merges cleanlybr- This patch adds no public classesbrbrFor more
information see test
GitHub user joesu opened a pull request:
https://github.com/apache/spark/pull/1924
[SPARK-3011][SQL] _temporary directory should be filtered out by
sqlContext.parquetFile
You can merge this pull request into a Git repository by running:
$ git pull
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52069053
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
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52084042
test this please
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52084440
QA tests have started for PR 1924. This patch merges cleanly. brView
progress:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18449/consoleFull
---
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1924#issuecomment-52084536
QA results for PR 1924:br- This patch FAILED unit tests.br- This patch
merges cleanlybr- This patch adds no public classesbrbrFor more
information see test
33 matches
Mail list logo