At present, creating LoggingCacheErrorHandler
that logs stacktrace also requires supplying the logger to be used. This might be inconvenient for some users, as it requires usage of Commons Logging API.
This commit simplifies creation of such as LoggingCacheErrorHandler
instance by adding a constructor that only accepts a boolean flag indicating whether to log stacktrace.
This was originally a part of:
-
28648
For consideration:
The first commit could maybe be update to deprecate the existing constructor that takes
org.apache.commons.logging.Log
and replace it with the one that takesString
representing logger name as that way Commons Logging dependency wouldn't leak out at all. But I'd leave that decision to whoever reviews this PR.
Comment From: sbrannen
This has been merged into 5.3.x
and main
.
Thanks