Skip to content

Conversation

@wangyum
Copy link
Member

@wangyum wangyum commented May 15, 2019

What changes were proposed in this pull request?

This pr should after #24620.

How was this patch tested?

Exsting test

@SparkQA
Copy link

SparkQA commented May 15, 2019

Test build #105434 has finished for PR 24621 at commit a93fbf6.

  • This patch fails build dependency tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@wangyum
Copy link
Member Author

wangyum commented May 17, 2019

Sorry @dongjoon-hyun
The reason why I seperator it to 2 PRS is I'd like to discuss shoud we change the hive.version.short once we upgrade built-in Hive to 2.3.5 for hadoop-3.2. I have 3 options:

  1. Do not change it, still use 2.3.4.
  2. Update to 2.3.5.
  3. Update to 2.3.0 because the shoutVersion is 2.3.0 for Hive branch-2.3:

cc @gatorsmile @srowen @HyukjinKwon @felixcheung

@dongjoon-hyun
Copy link
Member

We should show the correct value if you don't have any reason to disguise this. And let's discuss on #24620 together.

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.

3 participants