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

feat: introduce java.time #1410

Merged
merged 6 commits into from
Nov 26, 2024
Merged

feat: introduce java.time #1410

merged 6 commits into from
Nov 26, 2024

Conversation

diegomarquezp
Copy link
Contributor

@diegomarquezp diegomarquezp commented Nov 19, 2024

This PR introduces java.time alternatives to existing org.threeten.bp.* methods, as well as switching internal variables (if any) to java.time

The main constraint is to keep the changes backwards compatible, so for each existing threeten method "method1(org.threeten.bp.Duration)" we will add an alternative with a Duration (or Timestamp when applicable) suffix: "method1Duration(java.time.Duration)".

For most cases, the implementation will be held in the java.time method and the old threeten method will just delegate the call to it. However, for the case of abstract classes, the implementation will be kept in the threeten method to avoid breaking changes (i.e. users that already overloaded the method in their user code).

@diegomarquezp diegomarquezp requested review from a team as code owners November 19, 2024 18:05
@product-auto-label product-auto-label bot added size: xs Pull request size is extra small. api: logging Issues related to the googleapis/java-logging-logback API. labels Nov 19, 2024
@diegomarquezp diegomarquezp requested review from lqiu96 and removed request for daniel-sanche November 19, 2024 19:38
@diegomarquezp diegomarquezp requested a review from a team as a code owner November 19, 2024 20:21
pom.xml Show resolved Hide resolved
@product-auto-label product-auto-label bot added size: s Pull request size is small. and removed size: xs Pull request size is extra small. labels Nov 20, 2024
@diegomarquezp diegomarquezp requested a review from lqiu96 November 20, 2024 21:44
Copy link

@lqiu96 lqiu96 left a comment

Choose a reason for hiding this comment

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

LGTM. Cindy for final review.

@product-auto-label product-auto-label bot added size: xs Pull request size is extra small. and removed size: s Pull request size is small. labels Nov 21, 2024
Copy link
Contributor

@cindy-peng cindy-peng left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you for working on it!

@diegomarquezp diegomarquezp merged commit 05b12ae into main Nov 26, 2024
19 checks passed
@diegomarquezp diegomarquezp deleted the introduce-java-time branch November 26, 2024 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: logging Issues related to the googleapis/java-logging-logback API. size: xs Pull request size is extra small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants