Details

      Description

      3 changes:

      1) Remove all the inner isXXXEnabled() check, all of our logger usages for low levels are doing outer checking, this inner checking is duplicated. All log library impls, for the enabled check is at least a volatile read which is not cheap. For some rare third part lib logger usages who does not do the outer check, I'd rather let them suffer the potential waste sanitizing. in order to speed up most logger usages.

      2) Don't even create the SanitizerLogWrapper, when sanitizer is disabled.

      3) Move sanitizer properties from portal.properties to system.properties
      Some loggers are created and usage before portal.properties loading, it is wrong to depend on that, all properties need to be loaded from system.properties

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                justin.choi Justin Choi
                Reporter:
                shuyang.zhou Shuyang Zhou
                Recent user:
                Shalaka Tendulkar
                Participants of an Issue:
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Packages

                  Version Package
                  6.2.0 CE B2