Skip to content
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

feat: List catalogs from metasrv #12219

Merged
merged 36 commits into from
Jul 31, 2023
Merged

Conversation

Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Jul 26, 2023

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/

Summary

This PR will allow list catalogs from metasrv.


This change is Reviewable

Signed-off-by: Xuanwo <github@xuanwo.io>
@vercel
Copy link

vercel bot commented Jul 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
databend ⬜️ Ignored (Inspect) Visit Preview Jul 31, 2023 8:20am

@github-actions github-actions bot added the pr-feature this PR introduces a new feature to the codebase label Jul 26, 2023
@Xuanwo Xuanwo marked this pull request as draft July 26, 2023 15:47
Xuanwo added 7 commits July 27, 2023 00:21
Signed-off-by: Xuanwo <github@xuanwo.io>
This reverts commit d524d4d.
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo marked this pull request as ready for review July 27, 2023 11:00
@Xuanwo Xuanwo requested a review from zhang2014 July 27, 2023 11:00
Xuanwo added 6 commits July 27, 2023 19:01
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Xuanwo added 3 commits July 28, 2023 13:49
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Xuanwo added 4 commits July 28, 2023 15:15
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo
Copy link
Member Author

Xuanwo commented Jul 29, 2023

Ok, most test caes passed. The only missing piece is drop_catalog support. I will implement that in another PR.

Xuanwo added 2 commits July 30, 2023 10:59
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo marked this pull request as ready for review July 30, 2023 11:11
@Xuanwo
Copy link
Member Author

Xuanwo commented Jul 30, 2023

After this pull request, our multi-catalog feature is now available! Users can create, list, show and drop catalogs that will be saved by Databend in Metasrv. At current stage, we support hive and iceberg catalogs.

Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo
Copy link
Member Author

Xuanwo commented Jul 30, 2023

Can't not reproduce locally:

Test finished, fail fast enabled, 1 out of 4102 records failed to run
0: query result mismatch:
[SQL] SHOW CATALOGS LIKE 'ctl';
[Diff] (-expected|+actual)
-   ctl
Error: SelfError("sqllogictest failed")
at tests/sqllogictests/suites/base/05_ddl/05_0029_ddl_create_catalog:12

I'm working on it.

@Xuanwo
Copy link
Member Author

Xuanwo commented Jul 31, 2023

Let's go!

@Xuanwo Xuanwo merged commit 744cf3a into databendlabs:main Jul 31, 2023
@Xuanwo Xuanwo deleted the catalog-in-metasrv branch July 31, 2023 08:53
andylokandy pushed a commit to andylokandy/databend that referenced this pull request Nov 27, 2023
* feat: List catalogs from metasrv

Signed-off-by: Xuanwo <github@xuanwo.io>

* try move

Signed-off-by: Xuanwo <github@xuanwo.io>

* Revert "try move"

This reverts commit d524d4d.

* Revert "feat: List catalogs from metasrv"

This reverts commit ec9939f.

* Add meta-store

Signed-off-by: Xuanwo <github@xuanwo.io>

* Add config

Signed-off-by: Xuanwo <github@xuanwo.io>

* Save work

Signed-off-by: Xuanwo <github@xuanwo.io>

* Fix build

Signed-off-by: Xuanwo <github@xuanwo.io>

* Fix typos

Signed-off-by: Xuanwo <github@xuanwo.io>

* format toml

Signed-off-by: Xuanwo <github@xuanwo.io>

* Implement list_catalogs

Signed-off-by: Xuanwo <github@xuanwo.io>

* Implement create catalog

Signed-off-by: Xuanwo <github@xuanwo.io>

* Fix build

Signed-off-by: Xuanwo <github@xuanwo.io>

* Fix test

Signed-off-by: Xuanwo <github@xuanwo.io>

* Fix test

Signed-off-by: Xuanwo <github@xuanwo.io>

* Refactor

Signed-off-by: Xuanwo <github@xuanwo.io>

* fix cargo lock

Signed-off-by: Xuanwo <github@xuanwo.io>

* Save work

Signed-off-by: Xuanwo <github@xuanwo.io>

* Add catalog info in plan

Signed-off-by: Xuanwo <github@xuanwo.io>

* Fix build

Signed-off-by: Xuanwo <github@xuanwo.io>

* Fix test

Signed-off-by: Xuanwo <github@xuanwo.io>

* Fix test

Signed-off-by: Xuanwo <github@xuanwo.io>

* Use debug for test

Signed-off-by: Xuanwo <github@xuanwo.io>

* Fix tenant

Signed-off-by: Xuanwo <github@xuanwo.io>

* fix test

Signed-off-by: Xuanwo <github@xuanwo.io>

* Revert "Use debug for test"

This reverts commit 9c11187.

---------

Signed-off-by: Xuanwo <github@xuanwo.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-feature this PR introduces a new feature to the codebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants