-
Type: Clean up
-
Status: Resolved
-
Priority: Minor
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 2023.0
-
Component/s: Code Refactoring (Global)
-
Epic Link:
-
Tags:
-
Sprint:nxplatform #79, nxplatform #80
-
Story Points:5
Since we introduced log4j2 we want to remove usage of Apache Commons Logging and SLF4J APIs and unify all logging to use log4j2 API.
Then, we could probably remove our dependencies to these logging API (without removing their bridges to log4j2).
For example from :
import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; [...] public static final Log log = LogFactory.getLog(RunConverter.class);
or
import org.slf4j.Logger; import org.slf4j.LoggerFactory; [...] private static final Logger LOG = LoggerFactory.getLogger(NuxeoCmisBrowserBindingServlet.class);
to :
import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; [..] private static final Logger log = LogManager.getLogger(BulkDownload.class);
This could also be the moment to get rid of the isInfoEnabled/isDebugEnabled/isTraceEnabled calls by replacing them to lambda evaluation where it is possible.