ICEfaces
  1. ICEfaces
  2. ICE-5184

transient push failure in portal environment

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.8.2
          • Fix Version/s: 1.8.3, 1.8.2-EE-GA_P02
          • Component/s: Framework
          • Labels:
            None
          • Environment:
            ICEfaces, Liferay

            Description


            Occasionally push events will fail to be delivered. SessionRenderer.render() is known to be called but the session appears to be no longer in the group.

            This is believed to be due to the WeakReference referring to the sesionID in GroupAsyncRenderer:

                        if (!contains(object)) {
                            if (group.add(new WeakReference(object))) {


            it is not necessary to retain the sessionID String with only a WeakReference, unlike the previous implementation where a reference to the session object was maintained.

              Issue Links

              is duplicated by

              Bug - A problem which impairs or prevents the functions of the product. ICE-4434 Ajax push doesn't work anymore after a page change in liferay portal

              • Major - Major loss of function.
              • Closed

                Activity

                Repository Revision Date User Message
                ICEsoft Public SVN Repository #21174 Wed Apr 07 14:49:28 MDT 2010 deryk.sinotte ICE-5184: remove WeakReferences wrappers from session ids so that render groups members are not garbage collected inadvertently
                Files Changed
                Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/async/render/GroupAsyncRenderer.java

                  People

                  • Assignee:
                    Unassigned
                    Reporter:
                    Ted Goddard
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    0 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: