Skip to content

Conversation

@liancheng
Copy link
Contributor

This PR is exactly the same as #3178 except it reverts the FileStatus.isDir to FileStatus.isDirectory change, since it doesn't compile with Hadoop 1.

Review on Reviewable

@SparkQA
Copy link

SparkQA commented Nov 17, 2014

Test build #23450 has started for PR 3298 at commit 866037e.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Nov 17, 2014

Test build #23450 has finished for PR 3298 at commit 866037e.

  • 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/23450/
Test PASSed.

@marmbrus
Copy link
Contributor

Thanks for fixing this. Merged again to master and 1.2

@asfgit asfgit closed this in 6b7f2f7 Nov 18, 2014
asfgit pushed a commit that referenced this pull request Nov 18, 2014
…d fixes for complex types

This PR is exactly the same as #3178 except it reverts the `FileStatus.isDir` to `FileStatus.isDirectory` change, since it doesn't compile with Hadoop 1.

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

Author: Cheng Lian <[email protected]>

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

866037e [Cheng Lian] Revers isDirectory to isDir (it breaks Hadoop 1 profile)
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)

(cherry picked from commit 6b7f2f7)
Signed-off-by: Michael Armbrust <[email protected]>
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