apache / dolphinscheduler

Apache DolphinScheduler is the modern data orchestration platform. Agile to create high performance workflow with low-code
https://dolphinscheduler.apache.org/
Apache License 2.0
12.88k stars 4.63k forks source link

[Fix-16279] Support Java 8 date/time type `java.time.OffsetDateTime` #16304

Closed xxzuo closed 4 months ago

xxzuo commented 4 months ago

Purpose of the pull request

fix #16279

refer https://github.com/FasterXML/jackson-modules-java8/issues/219

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

Pull Request Notice

Pull Request Notice

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

xxzuo commented 4 months ago

When adding support for the java.time.OffsetDateTime type, it is better to add the entire JavaTimeModule or just the serializers and deserializers related to OffsetDateTime ?

@ruanwenjun

ruanwenjun commented 4 months ago

JavaTimeModule

Add JavaTimeModule is better.

sonarcloud[bot] commented 4 months ago

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud