Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "[SPARK-4309][SPARK-4407][SQL] Date type support for Thrift server, and fixes for complex types" #3292

Closed
wants to merge 2 commits into from

Conversation

marmbrus
Copy link
Contributor

No description provided.

liancheng and others added 2 commits November 16, 2014 14:26
…d fixes for complex types

SPARK-4407 was detected while working on SPARK-4309. Merged these two into a single PR since 1.2.0 RC is approaching.

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

Author: Cheng Lian <lian@databricks.com>

Closes apache#3178 from liancheng/date-for-thriftserver and squashes the following commits:

6f71d0b [Cheng Lian] Makes toHiveString static
26fa955 [Cheng Lian] Fixes complex type support in Hive 0.13.1 shim
a92882a [Cheng Lian] Updates HiveShim for 0.13.1
73f442b [Cheng Lian] Adds Date support for HiveThriftServer2 (Hive 0.12.0)
…rver, and fixes for complex types"

This reverts commit cb6bd83.
@SparkQA
Copy link

SparkQA commented Nov 16, 2014

Test build #23441 has started for PR 3292 at commit 808e96e.

  • This patch merges cleanly.

asfgit pushed a commit that referenced this pull request Nov 16, 2014
…rver, and fixes for complex types"

Author: Michael Armbrust <michael@databricks.com>

Closes #3292 from marmbrus/revert4309 and squashes the following commits:

808e96e [Michael Armbrust] Revert "[SPARK-4309][SPARK-4407][SQL] Date type support for Thrift server, and fixes for complex types"

(cherry picked from commit 45ce327)
Signed-off-by: Michael Armbrust <michael@databricks.com>
@asfgit asfgit closed this in 45ce327 Nov 16, 2014
@SparkQA
Copy link

SparkQA commented Nov 17, 2014

Test build #23441 has finished for PR 3292 at commit 808e96e.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@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/23441/
Test PASSed.

@marmbrus marmbrus deleted the revert4309 branch November 19, 2014 02:47
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