You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2021/02/01 01:31:36 UTC

[GitHub] [geode] onichols-pivotal commented on pull request #5828: GEODE-8496: Bump jackson from 2.11.3 to 2.12.1

onichols-pivotal commented on pull request #5828:
URL: https://github.com/apache/geode/pull/5828#issuecomment-770501970


   @jinmei it looks like the test DistributedSystemMBeanIntegrationTest.queryAllUsingGfshDoesNotFormatDate was expecting Date objects to be serialized a POJOs (i.e. the test expects to see the internal serialized fields of a Date object).  I've updated it to instead expect the new failure message.  Is this an acceptable change in default behavior?  Perhaps better would be to start including the jsr310 jar in the shipping product, instead of as test-runtime-only dependency, so that date formatting always displays *something* out-of-the-box?


----------------------------------------------------------------
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.

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