ICEpush
  1. ICEpush
  2. PUSH-277

ConcurrentModificationException thrown for pendingNotificationSet in LocalPushGroupManager

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0.BETA, EE-4.0.0.GA
    • Fix Version/s: 4.0.BETA, 4.0
    • Component/s: Push Library
    • Labels:
      None
    • Environment:
      ICEpush

      Description

      A ConcurrentModificationException is thrown for concurrent access to the pendingNotificationSet in LocalPushGroupManager.

        Activity

        Hide
        Jack Van Ooststroom added a comment -

        Sending core/src/main/java/org/icepush/LocalPushGroupManager.java
        Transmitting file data .
        Committed revision 38062.

        Show
        Jack Van Ooststroom added a comment - Sending core/src/main/java/org/icepush/LocalPushGroupManager.java Transmitting file data . Committed revision 38062.
        Hide
        Jack Van Ooststroom added a comment -

        There is no ConcurrentSet or ConcurrentHashSet. As of JDK 1.6 there is a static method in the Collections class <E> Set<E> newSetFromMap(Map<e, Boolean> map) that can be used to create a Set backed by a ConcurrentMap. But ICEpush needs to be JDK 1.5 compatible. Therefor, a ReentrantLock is used for this Set. Marking this one as FIXED.

        Show
        Jack Van Ooststroom added a comment - There is no ConcurrentSet or ConcurrentHashSet. As of JDK 1.6 there is a static method in the Collections class <E> Set<E> newSetFromMap(Map<e, Boolean> map) that can be used to create a Set backed by a ConcurrentMap. But ICEpush needs to be JDK 1.5 compatible. Therefor, a ReentrantLock is used for this Set. Marking this one as FIXED.

          People

          • Assignee:
            Jack Van Ooststroom
            Reporter:
            Jack Van Ooststroom
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: