Skip to content

Conversation

@gatorsmile
Copy link
Member

@gatorsmile gatorsmile commented Feb 25, 2018

What changes were proposed in this pull request?

This is based on #20668 for supporting Hive 2.2 and Hive 2.3 metastore.

When we merge the PR, we should give the major credit to @wangyum

How was this patch tested?

Added the test cases

@SparkQA
Copy link

SparkQA commented Feb 25, 2018

Test build #87653 has finished for PR 20671 at commit 0fdf29b.

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

@SparkQA
Copy link

SparkQA commented Feb 25, 2018

Test build #87654 has finished for PR 20671 at commit 2e12c10.

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

@gatorsmile
Copy link
Member Author

retest this please

@SparkQA
Copy link

SparkQA commented Feb 28, 2018

Test build #87795 has finished for PR 20671 at commit 2e12c10.

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

@gatorsmile
Copy link
Member Author

cc @cloud-fan

@cloud-fan
Copy link
Contributor

LGTM. In the future we should refactor this part so that adding a new version doesn't need to touch so many files.

@cloud-fan
Copy link
Contributor

thanks, merging to master!

@asfgit asfgit closed this in ff14801 Mar 1, 2018
asfgit pushed a commit that referenced this pull request Mar 9, 2018
## What changes were proposed in this pull request?
In the PR #20671, I forgot to update the doc about this new support.

## How was this patch tested?
N/A

Author: gatorsmile <[email protected]>

Closes #20789 from gatorsmile/docUpdate.
tkakantousis pushed a commit to tkakantousis/spark that referenced this pull request Mar 11, 2018
## What changes were proposed in this pull request?
This is based on apache#20668 for supporting Hive 2.2 and Hive 2.3 metastore.

When we merge the PR, we should give the major credit to wangyum

## How was this patch tested?
Added the test cases

Author: Yuming Wang <[email protected]>
Author: gatorsmile <[email protected]>

Closes apache#20671 from gatorsmile/pr-20668.
peter-toth pushed a commit to peter-toth/spark that referenced this pull request Oct 6, 2018
## What changes were proposed in this pull request?
This is based on apache#20668 for supporting Hive 2.2 and Hive 2.3 metastore.

When we merge the PR, we should give the major credit to wangyum

## How was this patch tested?
Added the test cases

Author: Yuming Wang <[email protected]>
Author: gatorsmile <[email protected]>

Closes apache#20671 from gatorsmile/pr-20668.
ghost pushed a commit to tubular/spark that referenced this pull request Feb 19, 2019
## What changes were proposed in this pull request?
This is based on apache#20668 for supporting Hive 2.2 and Hive 2.3 metastore.

When we merge the PR, we should give the major credit to wangyum

## How was this patch tested?
Added the test cases

Author: Yuming Wang <[email protected]>
Author: gatorsmile <[email protected]>

Closes apache#20671 from gatorsmile/pr-20668.
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