rmannibucau commented on pull request #71: URL: https://github.com/apache/commons-logging/pull/71#issuecomment-1006350332
@cardamon was "likely" (so not always true) true until java 8, since java 8u<something> (think it is a 6x but no more sure) and more aggressive GC it is likely not true which means you get a perf issue now so until a benchmark on some real application prooves it wrong I think we shoudn't keep this change "as it" - but fully agree the impl is not correct in several env too, this is why i mentionned maybe it should move a bit. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
