ICEpush
  1. ICEpush
  2. PUSH-206

ConcurrentModificationException in LocalPushGroupManager causing the Timer to fail

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-3.0.0.GA_P01, 3.2
          • Fix Version/s: EE-3.2.0.GA, 3.3
          • Component/s: Push Library, Push Server
          • Labels:
            None
          • Environment:
            ICEpush and ICEpush-EE (EPS)

            Description

            java.util.ConcurrentModificationException
               at java.util.HashMap$HashIterator.nextEntry(HashMap.java:793)
               at java.util.HashMap$ValueIterator.next(HashMap.java:822)
               at java.util.AbstractCollection.toArray(AbstractCollection.java:124)
               at java.util.ArrayList.<init>(ArrayList.java:131)
               at org.icepush.LocalPushGroupManager.scan(LocalPushGroupManager.java:79)
               at org.icepush.LocalPushGroupManager$Notification$1.handle(LocalPushGroupManager.java:369)
               at org.icepush.LocalNotificationBroadcaster$2.handlingConfirmed(LocalNotificationBroadcaster.java:90)
               at org.icepush.BlockingConnectionServer$5.run(BlockingConnectionServer.java:223)
               at java.util.TimerThread.mainLoop(Timer.java:512)
               at java.util.TimerThread.run(Timer.java:462)

            This Exception breaks the Timer causing a cascading number of Timer related Exceptions.

              Issue Links

              blocks

              Bug - A problem which impairs or prevents the functions of the product. PUSH-207 NegativeArraySizeException in LocalPushGroupManager

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #32918 Thu Dec 20 16:02:21 MST 2012 jack.van.ooststroom Fixed JIRA PUSH-206 : ConcurrentModificationException in LocalPushGroupManager causing the Timer to fail
                Files Changed
                Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java
                Jack Van Ooststroom created issue -
                Jack Van Ooststroom made changes -
                Field Original Value New Value
                Component/s Push Server [ 10043 ]
                Jack Van Ooststroom made changes -
                Link This issue blocks PUSH-207 [ PUSH-207 ]
                Jack Van Ooststroom made changes -
                Assignee Jack Van Ooststroom [ jack.van.ooststroom ]
                Hide
                Permalink
                Jack Van Ooststroom added a comment -

                Sending icepush-ee/core/src/main/java/org/icepush/LocalPushGroupManager.java
                Transmitting file data .
                Committed revision 32918.

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

                Both groupMap and pushIDMap are now ConcurrentHashMaps. The iteration we have now takes this into account avoiding the additional creation of ArrayLists. Marking this one as FIXED.

                Show
                Jack Van Ooststroom added a comment - Both groupMap and pushIDMap are now ConcurrentHashMaps. The iteration we have now takes this into account avoiding the additional creation of ArrayLists. Marking this one as FIXED.
                Jack Van Ooststroom made changes -
                Status Open [ 1 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                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: