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
Add original value to Del in change feeds #3969
Conversation
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.
LGTM!
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.
looks good
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.
LGTM! Thank you very much for your continued efforts!
Thank you for submitting this pull request! We really appreciate you spending the time to work on these changes.
What is the motivation?
Add a delete variant for change feeds that includes the original value
What does this change do?
Add a variant onto CF that includes the original value
What is your testing strategy?
integration, unit, manual
Usage tests will be implemented alongside LQ
live select diff ...
and deleteIs this related to any issues?
Lq v2 del with diff/patch
Does this change need documentation?
Have you read the Contributing Guidelines?