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

[Breaking] Deleted deprecated parquet read/write APIs #5510

Merged
merged 4 commits into from
Jun 6, 2024

Conversation

malhotrashivam
Copy link
Contributor

@malhotrashivam malhotrashivam commented May 20, 2024

Follow up work from #5358 (more details there).

This is a breaking change, since we are deleting deprecated public APIs.

@malhotrashivam malhotrashivam added parquet Related to the Parquet integration NoDocumentationNeeded breaking NoReleaseNotesNeeded No release notes are needed. labels May 20, 2024
@malhotrashivam malhotrashivam added this to the 3. May 2024 milestone May 20, 2024
@malhotrashivam malhotrashivam self-assigned this May 20, 2024
@malhotrashivam malhotrashivam changed the title Deleted all deprecated parquet read/write APIs [Breaking] Deleted deprecated parquet read/write APIs May 20, 2024
Copy link
Member

@rcaudy rcaudy left a comment

Choose a reason for hiding this comment

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

.

@rcaudy rcaudy added the release blocker A bug/behavior that puts is below the "good enough" threshold to release. label May 20, 2024
Copy link
Contributor

@jmao-denver jmao-denver left a comment

Choose a reason for hiding this comment

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

The Python changes LGTM

@malhotrashivam malhotrashivam merged commit a8f121c into deephaven:main Jun 6, 2024
15 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jun 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
breaking NoDocumentationNeeded NoReleaseNotesNeeded No release notes are needed. parquet Related to the Parquet integration release blocker A bug/behavior that puts is below the "good enough" threshold to release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants