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-4056][Java] Add time-nanos logical type #3158

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

glywk
Copy link
Contributor

@glywk glywk commented Sep 11, 2024

AVRO-4056

What is the purpose of the change

To align with nanoseconds precision of timestamps and local timestamps, I would propose adding this precision to the time logical type Avro specification as well. Propose a Java implementation of AVRO-4043

Verifying this change

This change added tests and can be verified as follows:

  • Add test on schema validation when time_nanos is applied on good and wrong avro primitive type.

Documentation

  • Does this pull request introduce a new feature? (yes)
  • If yes, how is the feature documented? (docs in AVRO-4043)

@github-actions github-actions bot added the Java Pull Requests for Java binding label Sep 11, 2024
@martin-g martin-g marked this pull request as draft September 12, 2024 08:31
@martin-g
Copy link
Member

The PR is set to Draft until #3125 is merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Java Pull Requests for Java binding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants