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.

Sign up for GitHub

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

Jump to bottom

Use last lexigraphical key for parquet schema inference #4783

Merged
merged 3 commits into from
Nov 7, 2023
Merged

Use last lexigraphical key for parquet schema inference #4783

merged 3 commits into from
Nov 7, 2023

Conversation

devinrsmith
Copy link
Member

@devinrsmith devinrsmith commented Nov 7, 2023 •
edited
Loading

In support of #4738

@devinrsmith devinrsmith added bug Something isn't working parquet Related to the Parquet integration labels Nov 7, 2023
@devinrsmith devinrsmith added this to the November 2023 milestone Nov 7, 2023
@devinrsmith devinrsmith self-assigned this Nov 7, 2023
@devinrsmith devinrsmith changed the title Use first lexigraphical key for parquet schema inference Use last lexigraphical key for parquet schema inference Nov 7, 2023
@devinrsmith devinrsmith merged commit 09d18f1 into deephaven:main Nov 7, 2023
10 checks passed
@devinrsmith devinrsmith deleted the parquet-infer-from-first-sorted-key branch November 7, 2023 17:03
@github-actions github-actions bot locked and limited conversation to collaborators Nov 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@rcaudy rcaudy rcaudy approved these changes

Assignees

@devinrsmith devinrsmith

Labels
bug Something isn't working NoDocumentationNeeded NoReleaseNotesNeeded No release notes are needed. parquet Related to the Parquet integration
Projects
None yet
Milestone
November 2023
Development

Successfully merging this pull request may close these issues.
2 participants
@devinrsmith @rcaudy