-
Notifications
You must be signed in to change notification settings - Fork 242
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
Update sequences.mdx #6151
base: develop
Are you sure you want to change the base?
Update sequences.mdx #6151
Conversation
3553c36
to
bff5d56
Compare
bff5d56
to
2159476
Compare
@@ -229,6 +230,9 @@ support renaming `galloc` sequences or moving them to another namespace or | |||
renaming the namespace that contains a `galloc` sequence. Be | |||
mindful of this limitation while designing application schema. | |||
|
|||
Additionally, after restoring a node from a backup, all galloc sequences must manually |
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.
Not sure this is correct.
Additionally, after restoring a node from a backup, all galloc sequences must manually have their next value set to the maximum value of the related column. See "Set a new start value for the sequence".
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.
@wind39 This is what you told me and what we did for the customer. please confirm
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.
@djw-m is correct. We didn't set the maximum value of the sequence to the maximum value of the column.
@matthew123987 , check your text: have their maximum value set to the maximum value of
It's a paste-o, well spotted by @djw-m .
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.
@wind39 have popped in my suggested text from above. Would be good to get a review and approval on this so we can merge (and push the changes into 5.6 content too, as that's the new latest branch).
2159476
to
22286b2
Compare
What Changed?
@wind39