Skip to content

Conversation

@liancheng
Copy link
Contributor

Ideally we should convert Metastore Parquet tables with our own Parquet implementation on both read path and write path. However, the write path is not well covered, and causes this test failure. This PR is a hotfix to bring back Jenkins PR builder. A proper fix will be delivered in a follow-up PR.

Review on Reviewable

@SparkQA
Copy link

SparkQA commented Feb 6, 2015

Test build #26882 has started for PR 4413 at commit 5291289.

  • This patch merges cleanly.

@liancheng
Copy link
Contributor Author

Gonna merge it since it's breaking Jenkins PR builds now.

@pwendell
Copy link
Contributor

pwendell commented Feb 6, 2015

@liancheng thanks cheng, please do

asfgit pushed a commit that referenced this pull request Feb 6, 2015
…erySuite.CTAS with serde"

Ideally we should convert Metastore Parquet tables with our own Parquet implementation on both read path and write path. However, the write path is not well covered, and causes this test failure. This PR is a hotfix to bring back Jenkins PR builder. A proper fix will be delivered in a follow-up PR.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/apache/spark/4413)
<!-- Reviewable:end -->

Author: Cheng Lian <[email protected]>

Closes #4413 from liancheng/hotfix-parquet-ctas and squashes the following commits:

5291289 [Cheng Lian] Hot fix for "SQLQuerySuite.CTAS with serde"

(cherry picked from commit 7c0a648)
Signed-off-by: Cheng Lian <[email protected]>
@asfgit asfgit closed this in 7c0a648 Feb 6, 2015
@liancheng liancheng deleted the hotfix-parquet-ctas branch February 6, 2015 02:10
@SparkQA
Copy link

SparkQA commented Feb 6, 2015

Test build #26882 has finished for PR 4413 at commit 5291289.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • protected class CaseInsensitiveMap(map: Map[String, String]) extends Map[String, String]
    • trait CreatableRelationProvider

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26882/
Test PASSed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants