Skip to content
This repository has been archived by the owner on Feb 6, 2024. It is now read-only.

refactor: refactor ddl procedures #155

Merged
merged 7 commits into from
Apr 14, 2023

Conversation

ZuLiangWang
Copy link
Contributor

Which issue does this PR close?

Closes #

Rationale for this change

Adapt the ddl procedures to the current version of the procedure interface.

What changes are included in this PR?

  • Adapt the CreateTableProcedureDropTableProcedureCreatePartitionTableProcedureDropPartitionTableProcedure to the current version of the procedure interface.
  • Remove the code related to opening and closing the partition table, which is now handled by the proxy module of CeresDB.

Are there any user-facing changes?

None.

How does this change test

Unit tests are currently not working properly and will be fixed in a subsequent pull request.

@ZuLiangWang ZuLiangWang changed the base branch from main to refactor_cluster April 12, 2023 09:32
@ZuLiangWang ZuLiangWang force-pushed the refactor_ddl branch 2 times, most recently from 684242a to c94e502 Compare April 12, 2023 09:33
Copy link
Member

@ShiKaiWi ShiKaiWi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ShiKaiWi ShiKaiWi merged commit 1dadcb1 into apache:refactor_cluster Apr 14, 2023
ShiKaiWi pushed a commit that referenced this pull request Apr 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants