elexx commented on issue #55: JOHNZON-293 potential high memory consumption in dateconverter URL: https://github.com/apache/johnzon/pull/55#issuecomment-562033821 With all the new insights (not actually a memleak, just potential "high" memusage, not backwards compatible or introducing try/catch fallbacks), I'm not sure the change is worth the effort and risk. Maybe we just should add some Javadoc to the class, warning that it might consume a bit more memory than one might expect and provide some java.time.* Converters instead?
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services