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

domain: use snapshot interface to get commit ts of schema version #48283

Open
cfzjywxk opened this issue Nov 3, 2023 · 0 comments
Open

domain: use snapshot interface to get commit ts of schema version #48283

cfzjywxk opened this issue Nov 3, 2023 · 0 comments
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@cfzjywxk
Copy link
Contributor

cfzjywxk commented Nov 3, 2023

Enhancement

Currently, it's needed to get commit_ts of a schema version when inserting schema cache, the mvccGetByKey interface is used to do so. But this makes the entire schema reload process appear inconsistent. Snapshot-related interfaces are used during the reading of schema version and schema diff version, while mvcc interfaces are used when it's necessary to read the commit_ts of a schema diff transaction. Moreover, during the process of using mvccGetByKey, we have already encountered the following issues leading to schema cache invalidation:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Development

No branches or pull requests

1 participant