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

              Ascending order - Click to sort in descending order
              Jack Van Ooststroom created issue -
              Jack Van Ooststroom made changes -
              Field Original Value New Value
              Assignee Jack Van Ooststroom [ jack.van.ooststroom ]
              Jack Van Ooststroom made changes -
              Status Open [ 1 ] In Progress [ 3 ]
              Hide
              Permalink
              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.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #38062 Tue Sep 17 14:30:27 MDT 2013 jack.van.ooststroom Fixed JIRA PUSH-277 : ConcurrentModificationException thrown for pendingNotificationSet in LocalPushGroupManager
              Files Changed
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java
              Hide
              Permalink
              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.
              Jack Van Ooststroom made changes -
              Status In Progress [ 3 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 4.0 [ 11383 ]
              Fix Version/s EE-4.0.0.GA [ 11170 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                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: