Jetty
  1. Jetty
  2. JETTY-1114

WebAppClassLoader has unncessary synchronization

    Details

    • Type: Improvement Improvement
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 6.1.20
    • Fix Version/s: 6.1.21
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Description

      From this forum thread:

      The getResource() and loadClass() methods are marked synchronized in WebAppClassLoader.

      • Is this synchronization necessary?
      • If so, can the locking be made more granular?

        Activity

        Hide
        Greg Wilkins added a comment -

        I have unsynchronized getResource.
        loadClass will remain synchronized as there is a race between loading a class and resolving the class. The super implementation is also synchronized.

        Show
        Greg Wilkins added a comment - I have unsynchronized getResource. loadClass will remain synchronized as there is a race between loading a class and resolving the class. The super implementation is also synchronized.

          People

          • Assignee:
            Unassigned
            Reporter:
            Kevin Conaway
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: