reta commented on PR #2056: URL: https://github.com/apache/cxf/pull/2056#issuecomment-2344420243
> Even though it may turn out to be JDK issue, I think using DateTimeFormatter for parsing LocalDate is more appropriate and performance efficient (no need to create instances of SimpleDateFormat, Date, Instant, ZonedDateTime) The `DateTimeFormatter` is certainly never API than `SimpleDateFormat` but to my knowledge, `DateTimeFormatter` and `SimpleDateFormat` are not compatible [1], [2] (please correct me if I am wrong here). We could bring the changes than risk to break existing applications but those won't be backported. To your point, it would make sense to migrate to `java.time` APIs but that should not be limited to `LocalDate` only (for example, the `SearchUtils` still use `SimpleDateFormat`). [1] https://docs.oracle.com/javase/8/docs/api/java/time/format/DateTimeFormatter.html [2] https://docs.oracle.com/javase/8/docs/api/java/text/SimpleDateFormat.html -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
