[bugfix] Java 8 date/time type LocalDateTime not supported by default

[yuluo-yx]

• For [Task] <fix bugs by e2e test> #2485
• refer: [Improve] add unit test #2454 (comment)

[pwallk]

Perhaps adding jackson-datatype-jsr310 would be a better choice,such as

    @Bean
    public Jackson2ObjectMapperBuilderCustomizer customizer() {
        return builder -> {
            JavaTimeModule javaTimeModule = new JavaTimeModule();
            final SimpleDateFormat simpleDateFormat = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss.SSSX");
            simpleDateFormat.setTimeZone(TimeZone.getDefault());

            javaTimeModule.addSerializer(BigDecimal.class, ToStringSerializer.instance);
            DateTimeFormatter formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd'T'HH:mm:ss.SSSX");
            javaTimeModule.addSerializer(LocalDateTime.class, new LocalDateTimeSerializer(formatter));
            javaTimeModule.addDeserializer(LocalDateTime.class, new LocalDateTimeDeserializer(formatter));

            builder.modules(javaTimeModule)
                    .timeZone(TimeZone.getDefault())
                    .dateFormat(simpleDateFormat)
        };
    }

[yuluo-yx]

Perhaps adding jackson-datatype-jsr310 would be a better choice,such as

    @Bean
    public Jackson2ObjectMapperBuilderCustomizer customizer() {
        return builder -> {
            JavaTimeModule javaTimeModule = new JavaTimeModule();
            final SimpleDateFormat simpleDateFormat = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss.SSSX");
            simpleDateFormat.setTimeZone(TimeZone.getDefault());

            javaTimeModule.addSerializer(BigDecimal.class, ToStringSerializer.instance);
            DateTimeFormatter formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd'T'HH:mm:ss.SSSX");
            javaTimeModule.addSerializer(LocalDateTime.class, new LocalDateTimeSerializer(formatter));
            javaTimeModule.addDeserializer(LocalDateTime.class, new LocalDateTimeDeserializer(formatter));

            builder.modules(javaTimeModule)
                    .timeZone(TimeZone.getDefault())
                    .dateFormat(simpleDateFormat)
        };
    }

Have you tested this locally to see if it works?

[yuluo-yx]

btw, This doesn't look like it's going to work. Maybe you have a better plan.

[pwallk]

The example is rough, DateTimeFormatter does not support yyyy-MM-dd'T 'hh :mm:ss.SSSX, change to yyyy-MM-dd'T'HH:mm:ss.SSSSSS

[yuluo-yx]

The example is rough, DateTimeFormatter does not support yyyy-MM-dd'T 'hh :mm:ss.SSSX, change to yyyy-MM-dd'T'HH:mm:ss.SSSSSS

In the demo you provided, the time format is also. And the previous code was in this date format, is there any advantage to changing it?

[pwallk]

Just to demonstrate how jackson supports LocalDateTime, the time format can be changed as needed,

[yuluo-yx]

Just to demonstrate how jackson supports LocalDateTime, the time format can be changed as needed,

cc @tomsun28. Do you have any ideas for this pr

[Calvin979]

@yuluo-yx Maybe you can refer #2122 to find out why LocalDateTime is supported by default.
Is the time correct in db on your pr?

[tomsun28]

hi, this also related the pr #2251

[yuluo-yx]

hi, this also related the pr #2251

Looks like there's a lot riding on this one. 🥲🥲

[yuluo-yx]

I'm going to close this pr for now, looking for a better solution in #2454