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

Execute create database in procedure #2601

Closed
Tracked by #286 ...
WenyXu opened this issue Oct 13, 2023 · 11 comments
Closed
Tracked by #286 ...

Execute create database in procedure #2601

WenyXu opened this issue Oct 13, 2023 · 11 comments
Assignees
Labels
C-enhancement Category Enhancements good first issue Good for newcomers help wanted Extra attention is needed

Comments

@WenyXu
Copy link
Member

WenyXu commented Oct 13, 2023

What type of enhancement is this?

Tech debt reduction

What does the enhancement do?

Before implementing the drop database #825, we should move the create database execution into the procedure. It needs to require a schema-level lock before execution.

Implementation challenges

No response

@WenyXu WenyXu added C-enhancement Category Enhancements good first issue Good for newcomers help wanted Extra attention is needed labels Oct 13, 2023
@WenyXu WenyXu changed the title execute create database in procedure Execute create database in procedure Oct 13, 2023
@john8628
Copy link

can i have a try? @WenyXu

@WenyXu
Copy link
Member Author

WenyXu commented Oct 13, 2023

can i have a try? @WenyXu

Thanks ❤️

@john8628
Copy link

@WenyXu I think I need some help for working on adding support for a new CreateDatabaseProcedure by:

  1. Adding a new variant CreateDatabase(CreateDatabaseTask) to the ddlTask enum
  2. Implementing CreateDatabaseTask in ddl.rs
  3. Implementing submit_create_database_task async function in ddl_taskmanager
    However, when running the project I'm getting the following error:
    error[E0432]: unresolved import api::v1::meta::CreateDatabaseTask
    --> src/common/meta/src/rpc/ddl.rs:22:5
    |
    22 | CreateDatabaseTask as PbCreateDatabaseTask
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    | |
    | no CreateDatabaseTask in v1::meta
    | help: a similar name exists in the module: CreateTableTask

@WenyXu
Copy link
Member Author

WenyXu commented Oct 24, 2023

@WenyXu I think I need some help for working on adding support for a new CreateDatabaseProcedure by:

  1. Adding a new variant CreateDatabase(CreateDatabaseTask) to the ddlTask enum
  2. Implementing CreateDatabaseTask in ddl.rs
  3. Implementing submit_create_database_task async function in ddl_taskmanager
    However, when running the project I'm getting the following error:
    error[E0432]: unresolved import api::v1::meta::CreateDatabaseTask
    --> src/common/meta/src/rpc/ddl.rs:22:5
    |
    22 | CreateDatabaseTask as PbCreateDatabaseTask
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    | |
    | no CreateDatabaseTask in v1::meta
    | help: a similar name exists in the module: CreateTableTask

You need to define the it in this repo

@john8628
Copy link

OK ,got it , thanks

@WenyXu
Copy link
Member Author

WenyXu commented Nov 22, 2023

Hi @john8628, How is it going? Do you have any updates?

@john8628
Copy link

you have any updates

I am sorry, it seems difficult to me; I cannot do it alone.

@WenyXu
Copy link
Member Author

WenyXu commented Nov 24, 2023

you have any updates

I am sorry, it seems difficult to me; I cannot do it alone.

It's fine. Maybe take a look at other good first issues.

@john8628
Copy link

you have any updates

I am sorry, it seems difficult to me; I cannot do it alone.

It's fine. Maybe take a look at other good first issues.

thanks

@QuenKar
Copy link
Contributor

QuenKar commented Dec 4, 2023

hi! is there someone solving the issue? can I have a try?

@fengjiachun
Copy link
Collaborator

Close this issue, the detail in this comment: #2880 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category Enhancements good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants