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

support partition value serialization for more data types #357

Closed
houqp opened this issue Aug 7, 2021 · 1 comment
Closed

support partition value serialization for more data types #357

houqp opened this issue Aug 7, 2021 · 1 comment
Labels
binding/rust Issues for the Rust crate enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@houqp
Copy link
Member

houqp commented Aug 7, 2021 •
edited
Loading

Description

In rust/src/checkpoints.rs, we currently only support partition value deserialization for string, byte, short, integer, long and boolean.

Based on https://github.com/delta-io/delta/blob/master/PROTOCOL.md#partition-value-serialization, we also need to support double, date, timestamp and binary
@houqp houqp added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers binding/rust Issues for the Rust crate labels Aug 7, 2021
@houqp houqp mentioned this issue Aug 8, 2021
Merged
@zijie0 zijie0 mentioned this issue Aug 10, 2021
Merged
@houqp
Copy link
Member Author

houqp commented Jul 11, 2022

already implemented

@houqp houqp closed this as completed Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
binding/rust Issues for the Rust crate enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@houqp