glassfish
  1. glassfish
  2. GLASSFISH-14199

[PERF] Orb deserializing creating LogManager$Cleaner threads

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 3.1
    • Fix Version/s: 3.1_ms07
    • Component/s: orb
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      14,199
    • Tags:

      Description

      Objects that have cached the result of LogManager.getLogger() and are then
      copied via the corba copying code will end up with new instances of the
      LogManager class, which is expected to be a singleton. When the new instances of
      the class are created, the LogManager.<init> also constructs a new Cleaner thread.

      The end result is a very large memory leak; this is also at least partly
      responsible for the deserialization performance regression we are seeing.

        Issue Links

          Activity

          Hide
          Ken Cavanaugh added a comment -

          I have a possible workaround for this delivered to Scott for testing.
          An improved object copier may need to wait until GF 3.2.

          Show
          Ken Cavanaugh added a comment - I have a possible workaround for this delivered to Scott for testing. An improved object copier may need to wait until GF 3.2.
          Hide
          Ken Cavanaugh added a comment -

          I modified the ORB object copier in b010 to avoid copying Logger and LogManager.
          According to an email from Joe Fialli, this issue has been fixed, so I am
          closing this issue.

          Show
          Ken Cavanaugh added a comment - I modified the ORB object copier in b010 to avoid copying Logger and LogManager. According to an email from Joe Fialli, this issue has been fixed, so I am closing this issue.
          Hide
          preston001 added a comment -

          What files specifically were modified to resolve this?

          Show
          preston001 added a comment - What files specifically were modified to resolve this?

            People

            • Assignee:
              Ken Cavanaugh
              Reporter:
              Scott Oaks
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: