bmscomp commented on PR #13662:
URL: https://github.com/apache/kafka/pull/13662#issuecomment-1637174843

   @divijvaidya  The third point you mentioned 
   Modified handling of datetime with zone information in it. See: 
https://github.com/FasterXML/jackson/wiki/Jackson-Release-2.15#changes-behavior-other
 does not affect the existing code base, it solves an issue, it's a correction 
of an existing bug in previous jackson version that prevent producing that kind 
of bugs 
   
   ```
   ZonedDateTime now = ZonedDateTime.now();
   System.out.println(now.withZoneSameInstant(ZoneOffset.UTC)
           .equals(now.withZoneSameInstant(ZoneId.of("UTC"))));
   ```
   
   I use to think that we need to write tests on  json Serialization and 
Deserialization of zoned datetime values 
   
   
   
   


-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to