New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use tidb as metastore #544

Merged
merged 7 commits into from Jan 29, 2019

Conversation

Projects
None yet
3 participants
@zhexuany
Copy link
Member

zhexuany commented Jan 4, 2019

In this PR, we add a hive-site.xml template which can be used as an example if user wants to store their metastore in tidb. In addition, to make user happy, a doc is also created to teach user how to set tidb up.

@zhexuany zhexuany requested review from xiaojingchen and ilovesoup Jan 4, 2019

@zhexuany zhexuany force-pushed the zhexuany:use_tidb_as_metastore branch from 6ddacd9 to 6ebde1f Jan 4, 2019

@birdstorm
Copy link
Member

birdstorm left a comment

PTAL

Show resolved Hide resolved docs/how_to_use_tidb_as_metastore_db.md Outdated
Show resolved Hide resolved docs/how_to_use_tidb_as_metastore_db.md Outdated
Show resolved Hide resolved docs/how_to_use_tidb_as_metastore_db.md Outdated
Show resolved Hide resolved docs/how_to_use_tidb_as_metastore_db.md Outdated
Show resolved Hide resolved docs/how_to_use_tidb_as_metastore_db.md Outdated
Show resolved Hide resolved config/hive-site.xml Outdated
Show resolved Hide resolved docs/how_to_use_tidb_as_metastore_db.md
Show resolved Hide resolved docs/how_to_use_tidb_as_metastore_db.md Outdated
Show resolved Hide resolved docs/how_to_use_tidb_as_metastore_db.md Outdated
@ilovesoup

This comment has been minimized.

Copy link
Collaborator

ilovesoup commented Jan 4, 2019

So current TiDB version support direct metastore initialize already?

@zhexuany zhexuany removed the request for review from xiaojingchen Jan 7, 2019

zhexuany added some commits Jan 21, 2019

@zhexuany

This comment has been minimized.

Copy link
Member Author

zhexuany commented Jan 21, 2019

/run-all-tests

zhexuany added some commits Jan 28, 2019

@zhexuany

This comment has been minimized.

Copy link
Member Author

zhexuany commented Jan 28, 2019

Why tidb after 2.1 release canot be used please refer to pingcap/tidb#8793

@birdstorm
Copy link
Member

birdstorm left a comment

LGTM

Already addressed.

@zhexuany zhexuany merged commit cec026f into pingcap:master Jan 29, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
jenkins-ci-tispark/build Jenkins job succeeded.
Details

@zhexuany zhexuany deleted the zhexuany:use_tidb_as_metastore branch Jan 29, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment