[SLING-8236] remove toString from ModelAdapterFactory.clearDisposalCallbackRegistryQueue debug log

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Models Impl 1.4.10
• Fix Version/s: Models Impl 1.4.12
• Component/s: General
• Assignee: Nicolas Peltier
• Reporter: Nicolas Peltier
• Created: 24/Jan/19 13:53
• Updated: 24/Jan/19 14:21
• Resolved: 24/Jan/19 14:21

Description

log.debug("calling disposal for {}.", ref.toString());
itself is responsible for huge char[] allocation. Removing it is very quick and painless perf improvement (figures coming ahead), as toString will be called as well, but only in debug mode.

Issue Links

• links to: GitHub Pull Request #12