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.

Sign up for GitHub

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

Jump to bottom

unify the schemaWrap4Owner into schema storage implementation #10332

Closed
3AceShowHand opened this issue Dec 20, 2023 · 0 comments · Fixed by #10327
Closed

unify the schemaWrap4Owner into schema storage implementation #10332

3AceShowHand opened this issue Dec 20, 2023 · 0 comments · Fixed by #10327
Assignees
@3AceShowHand
Labels
area/ticdc Issues or PRs related to TiCDC. type/enhancement The issue or PR belongs to an enhancement.

Comments

@3AceShowHand
Copy link
Contributor

At the moment, there are 2 schema storage implementations, each one for the processor and the owner. It's redundant, we can unify they into one implementation, to simplify the code and improve code maintainance.
@3AceShowHand 3AceShowHand self-assigned this Dec 20, 2023
@3AceShowHand 3AceShowHand added type/enhancement The issue or PR belongs to an enhancement. area/ticdc Issues or PRs related to TiCDC. labels Dec 20, 2023
@3AceShowHand 3AceShowHand mentioned this issue Dec 20, 2023
Merged
ti-chi-bot bot pushed a commit that referenced this issue Dec 25, 2023
@3AceShowHand
schemaStorage(ticdc): unify 2 schema storage implementation (#10327)
40cd9c8 
close #10332
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@3AceShowHand 3AceShowHand

Labels
area/ticdc Issues or PRs related to TiCDC. type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

schemaStorage(ticdc): unify 2 schema storage implementation 3AceShowHand/tiflow
1 participant
@3AceShowHand