onichols-pivotal edited a comment 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 as POJOs (i.e. the test expects to see 
the internal serialized fields of a LocalDate object).  Jackson 2.12 no longer 
allows that.  I've updated the test 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:
[email protected]


Reply via email to