Skip to content

Replace java.util.Date and TimeUnit usage in scheduling with appropiate java.time classes #28714

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
Closed
desiderantes opened this issue Jun 27, 2022 · 2 comments
Closed

Replace java.util.Date and TimeUnit usage in scheduling with appropiate java.time classes #28714

desiderantes opened this issue Jun 27, 2022 · 2 comments
Assignees
@poutsma
Labels
in: core Issues in core modules (aop, beans, core, context, expression) type: enhancement A general enhancement
Milestone
6.0.0-M5

Comments

@desiderantes
Copy link

Given that a lot of times we get the values for Triggers and such from configs or other places that use java.time types, and the fact that they're the proper types to model this sort of thing, it would be nice to reduce usage of java.util.Date (since it's so prone to improper handling).
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Jun 27, 2022
@poutsma poutsma self-assigned this Jul 4, 2022
@poutsma poutsma added type: enhancement A general enhancement in: core Issues in core modules (aop, beans, core, context, expression) and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Jul 4, 2022
@poutsma poutsma added this to the 6.0.0-M5 milestone Jul 8, 2022
@poutsma poutsma closed this as completed in 9b739a2 Jul 8, 2022
@poutsma
Copy link
Contributor

poutsma commented Jul 8, 2022

@desiderantes This is now complete, see 9b739a2. Let me know if there are java.util.Date usages that I missed.

@desiderantes
Copy link
Author

@poutsma Whoah, thanks a lot! I don't see other usages of java.util.Date in scheduling (and I can't judge the usages in the other parts)

@rwinch rwinch mentioned this issue Jul 8, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@poutsma poutsma

Labels
in: core Issues in core modules (aop, beans, core, context, expression) type: enhancement A general enhancement
Projects
None yet
Milestone
6.0.0-M5
Development

No branches or pull requests
3 participants
@poutsma @desiderantes @spring-projects-issues