[jira] [Created] (SIS-570) Assign loggers to static final fields

["Martin Desruisseaux (Jira)" <ji...@apache.org>]

Martin Desruisseaux created SIS-570:
---------------------------------------

             Summary: Assign loggers to static final fields
                 Key: SIS-570
                 URL: https://issues.apache.org/jira/browse/SIS-570
             Project: Spatial Information Systems
          Issue Type: Task
          Components: Coverage, Features, GUI, Metadata, Referencing, Storage, Utilities
    Affects Versions: 1.3
            Reporter: Martin Desruisseaux
            Assignee: Martin Desruisseaux
             Fix For: 1.4


Most of the time during logging, SIS fetches the logger by a call to {{Logger.getLogger(String)}} at each log event. The recommended practice is rather to assign the logger in a static field instead, in order to keep the configuration stable (otherwise the {{Logger}} instance may be garbage collected and its configuration lost). It may also have a performance advantage, but uncertain given that the majority of SIS logging are for reporting recoverable exceptions.

We should replace many (not all) calls to {{Logger.getLogger(String)}} by static constants. We do not necessarily replace all calls because we do not want to make those loggers visible in public API and we may want to keep the unconditional creation of {{Logger}} in e.g. the root of a hierarchy of classes using that logger.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)