-
Notifications
You must be signed in to change notification settings - Fork 433
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
docs(python): update docs #1155
Conversation
Just went through our issues, and I guess we can close #373, once this is merged. |
62cf21e
to
d5c4b87
Compare
@@ -585,7 +585,7 @@ def test_partition_overwrite( | |||
tmp_path, | |||
sample_data, | |||
mode="overwrite", | |||
partitions_filters=[("p1", "=", "1")], | |||
partition_filters=[("p1", "=", "1")], |
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.
We haven't yet released with this feature, so this API change should be fine.
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.
Excellent update!
.. warning:: | ||
The writer is currently *experimental*. Please use on test data first, not | ||
on production data. Report any issues at https://github.com/delta-io/delta-rs/issues. | ||
|
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.
🎉
Co-authored-by: Robert Pack <42610831+roeap@users.noreply.github.com>
# Description The description of the main changes of your pull request # Related Issue(s) - closes delta-io#715 - closes delta-io#373 # Documentation <!--- Share links to useful documentation ---> --------- Co-authored-by: Robert Pack <42610831+roeap@users.noreply.github.com>
Description
The description of the main changes of your pull request
Related Issue(s)
Documentation