Skip to content

[MINOR][BUILD] Add 2 maven properties(hive.classifier and hive.parquet.group)#23996

Closed
wangyum wants to merge 1 commit intoapache:masterfrom
wangyum:add_2_maven_properties
Closed

[MINOR][BUILD] Add 2 maven properties(hive.classifier and hive.parquet.group)#23996
wangyum wants to merge 1 commit intoapache:masterfrom
wangyum:add_2_maven_properties

Conversation

@wangyum
Copy link
Member

@wangyum wangyum commented Mar 7, 2019

What changes were proposed in this pull request?

This pr adds 2 maven properties to help us upgrade the built-in Hive.

Property Name Default In future
hive.classifier (none) core
hive.parquet.group com.twitter org.apache.parquet

How was this patch tested?

existing tests

@SparkQA
Copy link

SparkQA commented Mar 7, 2019

Test build #103114 has finished for PR 23996 at commit 1f977a5.

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

@wangyum
Copy link
Member Author

wangyum commented Mar 7, 2019

cc @srowen In order to make the upgrade built-in Hive change smaller, I'd like to change this first.

@srowen
Copy link
Member

srowen commented Mar 7, 2019

I see, this is so we can try Hive 2/3? OK.

@wangyum
Copy link
Member Author

wangyum commented Mar 7, 2019

cc @srowen In order to make the upgrade built-in Hive change smaller, I'd like to change this first.

Yes. I have tried it: https://github.com/wangyum/spark/tree/hadoop-3.2
and all test passed.

@wangyum
Copy link
Member Author

wangyum commented Mar 7, 2019

@srowen Could we merge this to the master?

@srowen
Copy link
Member

srowen commented Mar 7, 2019

Merged to master

@srowen srowen closed this in d70b6a3 Mar 7, 2019
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