Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Won't Fix
    • Affects Version/s: 1.2
    • Fix Version/s: 1.2
    • Component/s: Uncategorized
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      77

      Description

      We need to remove the synchronization block on the static variable
      "descriptors". Under the current implementation, any component created, the
      current classloader (application wide) has to be synchronized with every other
      component. This is not acceptable.

      Since loading PropertyDescriptors is not required to be synchronous, it is
      acceptable to have more than one constructor actually populate the static map in
      a non-blocking manner.

      Code changes would require simply removing the synchronization block in the
      constructor. Also, the static WeakHashMap should probably be indexed not by
      Class, but by ClassName. With all of the oddball Classloaders and byte weavers
      developers are using, I wouldn't trust Class.hashCode().

      1. changebundle.txt
        26 kB
        rogerk

        Activity

        Hide
        rogerk added a comment -

        Created an attachment (id=48)
        change bundle

        Show
        rogerk added a comment - Created an attachment (id=48) change bundle
        Hide
        rogerk added a comment -

        Created an attachment (id=50)
        changes - part 2

        Show
        rogerk added a comment - Created an attachment (id=50) changes - part 2
        Hide
        Ed Burns added a comment -

        reassign to Roger.

        Show
        Ed Burns added a comment - reassign to Roger.
        Hide
        rogerk added a comment -

        Ed, can you take alook at this one please?

        Show
        rogerk added a comment - Ed, can you take alook at this one please?
        Hide
        Ed Burns added a comment -

        This is an implementation issue. Moving to
        <https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=255>.

        Show
        Ed Burns added a comment - This is an implementation issue. Moving to < https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=255 >.
        Hide
        Ed Burns added a comment -

        Prepare to delete api subcomponent

        Show
        Ed Burns added a comment - Prepare to delete api subcomponent
        Hide
        Ed Burns added a comment -

        Move all to 1.2

        Show
        Ed Burns added a comment - Move all to 1.2
        Hide
        Manfred Riem added a comment -

        Closing resolved issue out

        Show
        Manfred Riem added a comment - Closing resolved issue out

          People

          • Assignee:
            Ed Burns
            Reporter:
            jhook
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: