-
Notifications
You must be signed in to change notification settings - Fork 400
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
feat: add metadata for operations::write::WriteBuilder #1584
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.
I would love to see a test included but this is still an improvement
Thanks for the review. I'm not much aware about the test setup in the repo. Could you please let me know which place I can add the test for this change? |
9b55f4d
to
361d1b3
Compare
@rtyler I have added tests. Please have a look and let me know if there's anything else required. Also, if you could please let me know approximately when the next version of delta-rs would be released with these changes? Thanks |
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.
Thanks for adding this!
Description
This PR simply provides a way to add commit metadata during
DeltaOps::write()
operation similar toDeltaOps::update()
.Related Issue(s)
None
Documentation
None