Skip to content

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 <[email protected]>

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 <[email protected]>

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 <[email protected]>
@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