Uploaded image for project: 'Log4j 2'
  1. Log4j 2
  2. LOG4J2-1858

Memory issues with ParameterizedMessage

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.9.0
    • Component/s: None
    • Labels:
      None

      Description

      ParameterizedMessage keeps track of a ThreadLocal<StringBuilder> in order to save object creations. However, the reused string builders can only grow over time, which may end up causing memory issues after some large messages have been logged. This is especially true if the application uses a fixed thread pool since the string builders cannot be collected at all.

      One way to address that issue would be to drop the string builder if it grows too large, but I have concerns that this could cause garbage collection issues if this happens too often (since those string builders might not die young). So maybe this class should go back to create the StringBuilder on demand and make sure it always dies young?

      For the record, this problem seems to have been introduced in LOG4J2-1271 / https://git-wip-us.apache.org/repos/asf?p=logging-log4j2.git;a=blobdiff;f=log4j-api/src/main/java/org/apache/logging/log4j/message/ParameterizedMessage.java;h=d315c1345b5fb72c8d88f5e1aa177011c7376fb9;hp=334e19ba7c188e8ac862863830cf17dca7b7007c;hb=dca586c;hpb=047565e8928b0c9893c25ee92ffdf48dbcd6965c.

        Attachments

          Activity

            People

            • Assignee:
              remkop@yahoo.com Remko Popma
              Reporter:
              jpountz Adrien Grand
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: