Skip to content
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 versions to 0.7.0 and update changelog #1083

Closed
wants to merge 2 commits into from

Conversation

iajoiner
Copy link
Contributor

Description

The description of the main changes of your pull request

Related Issue(s)

Closes #1035.

Documentation

@github-actions github-actions bot added binding/rust Issues for the Rust crate delta-inspect rust labels Jan 19, 2023
@rtyler rtyler enabled auto-merge (rebase) January 19, 2023 23:50
@rtyler rtyler disabled auto-merge January 19, 2023 23:51
Copy link
Collaborator

@wjones127 wjones127 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm we seem to erroneously include irrelevant issues. I guess in order to fix this the right way, we should do a better job at grooming the issue labels? Also perhaps we can tag PRs with the right bindings label so we can filter for PRs that affect Rust (avoiding the Python-only ones).

Comment on lines +11 to +14
- Python package: Loosen version requirements for maturin [\#1004](https://github.com/delta-io/delta-rs/issues/1004)
- Remove `Cargo.lock` from library crates and add `Cargo.lock` to binary ones [\#1000](https://github.com/delta-io/delta-rs/issues/1000)
- More frequent Rust releases [\#969](https://github.com/delta-io/delta-rs/issues/969)
- Thoughts on adding read\_delta to pandas [\#869](https://github.com/delta-io/delta-rs/issues/869)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Python package: Loosen version requirements for maturin [\#1004](https://github.com/delta-io/delta-rs/issues/1004)
- Remove `Cargo.lock` from library crates and add `Cargo.lock` to binary ones [\#1000](https://github.com/delta-io/delta-rs/issues/1000)
- More frequent Rust releases [\#969](https://github.com/delta-io/delta-rs/issues/969)
- Thoughts on adding read\_delta to pandas [\#869](https://github.com/delta-io/delta-rs/issues/869)

I don't think these apply to the rust release, do they?

Comment on lines +23 to +24
- Overwrite creates new file [\#960](https://github.com/delta-io/delta-rs/issues/960)
- The written delta file has corrupted structure [\#956](https://github.com/delta-io/delta-rs/issues/956)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Overwrite creates new file [\#960](https://github.com/delta-io/delta-rs/issues/960)
- The written delta file has corrupted structure [\#956](https://github.com/delta-io/delta-rs/issues/956)

These two issues were just ones where users misunderstood how Delta tables works. They have no change set associated with them.

I wonder if the script that generates this could exclude issues that have no linked PRs?

- Overwrite creates new file [\#960](https://github.com/delta-io/delta-rs/issues/960)
- The written delta file has corrupted structure [\#956](https://github.com/delta-io/delta-rs/issues/956)
- Write mode doesn't work with Azure storage [\#955](https://github.com/delta-io/delta-rs/issues/955)
- Python: We don't error on reader protocol v2 [\#886](https://github.com/delta-io/delta-rs/issues/886)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Python: We don't error on reader protocol v2 [\#886](https://github.com/delta-io/delta-rs/issues/886)

This one is Python only.

- The written delta file has corrupted structure [\#956](https://github.com/delta-io/delta-rs/issues/956)
- Write mode doesn't work with Azure storage [\#955](https://github.com/delta-io/delta-rs/issues/955)
- Python: We don't error on reader protocol v2 [\#886](https://github.com/delta-io/delta-rs/issues/886)
- Cannot open a deltatable in S3 using AWS\_PROFILE based credentials from a local machine [\#855](https://github.com/delta-io/delta-rs/issues/855)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Cannot open a deltatable in S3 using AWS\_PROFILE based credentials from a local machine [\#855](https://github.com/delta-io/delta-rs/issues/855)

Duplicate of feature above.

Comment on lines +37 to +40
- Add a new release github action for Python binding: macos with universal2 wheel [\#976](https://github.com/delta-io/delta-rs/pull/976) ([fvaleye](https://github.com/fvaleye))
- Bump version of the Python binding to 0.6.4 [\#970](https://github.com/delta-io/delta-rs/pull/970) ([fvaleye](https://github.com/fvaleye))
- Handle pandas timestamps [\#958](https://github.com/delta-io/delta-rs/pull/958) ([hayesgb](https://github.com/hayesgb))
- test\(python\): add azure integration tests [\#912](https://github.com/delta-io/delta-rs/pull/912) ([wjones127](https://github.com/wjones127))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Add a new release github action for Python binding: macos with universal2 wheel [\#976](https://github.com/delta-io/delta-rs/pull/976) ([fvaleye](https://github.com/fvaleye))
- Bump version of the Python binding to 0.6.4 [\#970](https://github.com/delta-io/delta-rs/pull/970) ([fvaleye](https://github.com/fvaleye))
- Handle pandas timestamps [\#958](https://github.com/delta-io/delta-rs/pull/958) ([hayesgb](https://github.com/hayesgb))
- test\(python\): add azure integration tests [\#912](https://github.com/delta-io/delta-rs/pull/912) ([wjones127](https://github.com/wjones127))

Python only.

- crates.io entry is missing link to rustdoc documentation [\#1076](https://github.com/delta-io/delta-rs/issues/1076)
- DeltaTable DataFusion TableProvider does not support filter pushdown [\#1064](https://github.com/delta-io/delta-rs/issues/1064)
- DeltaTable DataFusion scan does not prune files properly [\#1063](https://github.com/delta-io/delta-rs/issues/1063)
- CI constantly fails for `python_build / Python Build (Python 3.10 PyArrow latest)` [\#1025](https://github.com/delta-io/delta-rs/issues/1025)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- CI constantly fails for `python_build / Python Build (Python 3.10 PyArrow latest)` [\#1025](https://github.com/delta-io/delta-rs/issues/1025)
- CI constantly fails for `python_build / Python Build (Python 3.10 PyArrow latest)` [\#1025](https://github.com/delta-io/delta-rs/issues/1025)

dev facing only

- Bump version of the Python binding to 0.6.4 [\#970](https://github.com/delta-io/delta-rs/pull/970) ([fvaleye](https://github.com/fvaleye))
- Handle pandas timestamps [\#958](https://github.com/delta-io/delta-rs/pull/958) ([hayesgb](https://github.com/hayesgb))
- test\(python\): add azure integration tests [\#912](https://github.com/delta-io/delta-rs/pull/912) ([wjones127](https://github.com/wjones127))
- ci: Merge several automatically generated labels [\#1081](https://github.com/delta-io/delta-rs/pull/1081) ([iajoiner](https://github.com/iajoiner))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- ci: Merge several automatically generated labels [\#1081](https://github.com/delta-io/delta-rs/pull/1081) ([iajoiner](https://github.com/iajoiner))

dev-only

- fix: azure sas key url encoding [\#1036](https://github.com/delta-io/delta-rs/pull/1036) ([roeap](https://github.com/roeap))
- chore: fix new lints from new cargo [\#1034](https://github.com/delta-io/delta-rs/pull/1034) ([wjones127](https://github.com/wjones127))
- feat: expose function to get table of add actions [\#1033](https://github.com/delta-io/delta-rs/pull/1033) ([wjones127](https://github.com/wjones127))
- refactor\(api!\): refactor Python APIs for getting file list [\#1032](https://github.com/delta-io/delta-rs/pull/1032) ([wjones127](https://github.com/wjones127))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Python-only

- feat: make `DeltaStorageHandler` pickle serializable [\#1016](https://github.com/delta-io/delta-rs/pull/1016) ([roeap](https://github.com/roeap))
- feat: clean up dependencies and feature flags [\#1014](https://github.com/delta-io/delta-rs/pull/1014) ([roeap](https://github.com/roeap))
- test\(python\): make sure integration tests wait for services to start [\#979](https://github.com/delta-io/delta-rs/pull/979) ([wjones127](https://github.com/wjones127))
- test\(python\): add read / write benchmarks [\#933](https://github.com/delta-io/delta-rs/pull/933) ([wjones127](https://github.com/wjones127))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- test\(python\): add read / write benchmarks [\#933](https://github.com/delta-io/delta-rs/pull/933) ([wjones127](https://github.com/wjones127))

Python-only

- Fix to support UTC timezone [\#1022](https://github.com/delta-io/delta-rs/pull/1022) ([andrei-ionescu](https://github.com/andrei-ionescu))
- feat: make `DeltaStorageHandler` pickle serializable [\#1016](https://github.com/delta-io/delta-rs/pull/1016) ([roeap](https://github.com/roeap))
- feat: clean up dependencies and feature flags [\#1014](https://github.com/delta-io/delta-rs/pull/1014) ([roeap](https://github.com/roeap))
- test\(python\): make sure integration tests wait for services to start [\#979](https://github.com/delta-io/delta-rs/pull/979) ([wjones127](https://github.com/wjones127))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- test\(python\): make sure integration tests wait for services to start [\#979](https://github.com/delta-io/delta-rs/pull/979) ([wjones127](https://github.com/wjones127))

dev-facing only.

@iajoiner
Copy link
Contributor Author

@wjones127 Sorry about that. Currently I filter out labels such as dependencies as well as non-Rust languages. Let me check tomorrow how to better achieve that.

@iajoiner
Copy link
Contributor Author

Sorry I will handle it tomorrow night.

@wjones127
Copy link
Collaborator

Closing in favor of #1137.

@wjones127 wjones127 closed this Feb 11, 2023
@iajoiner iajoiner deleted the prep_for_rustv0.7.0 branch February 27, 2023 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binding/rust Issues for the Rust crate delta-inspect rust
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Release delta-rs 0.7.0 (next release after 0.6.0)
3 participants