ICEpush
  1. ICEpush
  2. PUSH-152

BrowserDispatcher throws java.util.ConcurrentModificationException

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0-Alpha3
          • Fix Version/s: EE-3.2.0.BETA, EE-3.2.0.GA, 3.3
          • Component/s: Integrations
          • Labels:
            None
          • Environment:
            Vaadin 6.6.8, Vaadin IcePush addon 0.2.1, Tomcat 6.0.33, JRockit JRE 1.6.0_24-b07

            Description

            Occasionally IcePush throws following exceptions:

            Caused by: java.util.ConcurrentModificationException
                     at java.util.HashMap$HashIterator.nextEntry(HashMap.java:977)
                     at java.util.HashMap$ValueIterator.next(HashMap.java:1006)
                     at java.util.AbstractCollection.toArray(AbstractCollection.java:124)
                     at java.util.ArrayList.<init>(ArrayList.java:131)
                     at org.icepush.servlet.BrowserDispatcher.discardUnusedServlets(BrowserDispatcher.java:89)
                     at org.icepush.servlet.BrowserDispatcher.service(BrowserDispatcher.java:47)
                     at org.icepush.servlet.PathDispatcher.service(PathDispatcher.java:45)
                     at org.icepush.servlet.MainServlet.service(MainServlet.java:71)

            Discarding unused servlets should be thread-safe. This looks to be tightly linked to issue PUSH-151. This exception is thrown from exactly the same line as exception described in issue PUSH-151.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Sending icepush-ee/core/src/main/java/org/icepush/servlet/BrowserDispatcher.java
              Transmitting file data .
              Committed revision 32858.

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

              Synchronization is in place and iteration in both places happens similarly now with new ArrayList being created. Marking this one as FIXED.

              Show
              Jack Van Ooststroom added a comment - Synchronization is in place and iteration in both places happens similarly now with new ArrayList being created. Marking this one as FIXED.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: