Azure / Azure/azure-libraries-for-java
Use ZonedDateTime for ISO-8601 timestamp types
- Dominant language
- Java
- Stars
- 97
- Forks
- 102
- PR merge metrics
- No merged PRs in 30d
Description
The ZonedDateTime type was introduced in Java 8 and is functionally equivalent to the Joda DateTime type. Joda itself recommends using the java.time types starting with Java SE 8.
It would be preferable to depend on standard Java implementations for handling ISO-8601 types, as modern Java applications are more and more depending on ZonedDateTime rather than joda libraries for time-related types. Java SE 7 has been end-of-life since April 2015, so I believe backward compatibility should not be a relevant concern anymore.
Example code which uses Joda DateTime:
`azure.activityLogs().defineQuery().startingFrom().endsBefore()`
The startingFrom() and endsBefore() methods accept a Joda DateTime as an argument.
Contributor guide
Research direction
The issue identifies startingFrom() and endsBefore() as the affected methods and gives azure.activityLogs().defineQuery() as an entry point; begin by locating their signatures and callers. Determine the scope of replacing Joda DateTime with Java's ZonedDateTime, and confirm that the affected ISO-8601 APIs and their usage are consistently updated.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- api
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100