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

AVRO-3886: [Rust] Serialize attribute in schema to support custom logical type #2557

Merged
merged 2 commits into from
Oct 18, 2023

Conversation

ZENOTME
Copy link
Contributor

@ZENOTME ZENOTME commented Oct 17, 2023

AVRO-3886

What is the purpose of the change

To support custom logical type, this PR add serialize attribute in enum, fixed, record schema.

Verifying this change

  • New unit tests are added

Documentation

  • Does this pull request introduce a new feature? yes
  • If yes, how is the feature documented? not documented

@github-actions github-actions bot added the Rust label Oct 17, 2023
@martin-g martin-g changed the title AVRO-3886: Serialize attribute in schema to support custom logical type AVRO-3886: [Rust] Serialize attribute in schema to support custom logical type Oct 18, 2023
@martin-g martin-g merged commit d6837ed into apache:main Oct 18, 2023
martin-g pushed a commit that referenced this pull request Oct 18, 2023
…gical type (#2557)

* AVRO-3886: serialize attribute in schema to support custom logical type

* AVRO-3886: [Rust] Use all kinds of JSON values for the attributes to serialize

Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>

---------

Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>
Co-authored-by: ZENOTME <[email protected]>
Co-authored-by: Martin Tzvetanov Grigorov <[email protected]>
(cherry picked from commit d6837ed)
@martin-g
Copy link
Member

Thank you, @ZENOTME !

RanbirK pushed a commit to RanbirK/avro that referenced this pull request May 13, 2024
…gical type (apache#2557)

* AVRO-3886: serialize attribute in schema to support custom logical type

* AVRO-3886: [Rust] Use all kinds of JSON values for the attributes to serialize

Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>

---------

Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>
Co-authored-by: ZENOTME <[email protected]>
Co-authored-by: Martin Tzvetanov Grigorov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants