-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[core]delete created_from_snapshot and created_from_snapshot from branchTable #4159
[core]delete created_from_snapshot and created_from_snapshot from branchTable #4159
Conversation
It is possible that the branches tag does not exist but the snapshot exists? |
Yes, if you create empty branch then insert some data to this branch, or you create branch from tag then delete branch's tag. In above two case snapshot will exists,tag will not exists. In this case created_from_tag will be null. |
|
That's okay, I'll modify it. |
With the current architecture, we cannot accurately obtain the origin snapshot of the branch. Maybe we should open a discussion thread to discuss this issue. |
I create a issue #4166 |
Can we just delete created_from_tag and created_from_snapshot first? |
I think it's ok if we can't get the exact tag and snapshot.I'll change it. |
f0b7ed6
to
25cef19
Compare
have changed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
Purpose
created_from_snapshot
should be the origin of the tag,not the earliest snapshot of the branch.Linked issue: close #xxx
Tests
API and Format
Documentation