geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Udo Kohlmeyer <>
Subject Re: Backwards compatibility issue with JSONFormatter
Date Tue, 14 May 2019 17:47:24 GMT
How did this slip the review process that the signature of a public API 
was changed?

Well done Gester for finding this!!


On 5/14/19 10:40, Dan Smith wrote:
> I think the changes for GEODE-6327 broke backwards compatibility in
> JSONFormatter with the change from fromJSON(String jsonString) to
> fromJSON(String jsonString, String... identityFields)
> Adding an additional varargs parameter to the method breaks code that was
> compiled against the non-varargs version. I think we need to overload the
> method instead.
> Thanks to Gester for discovering this with his test!
> -Dan

View raw message