ICEfaces
  1. ICEfaces
  2. ICE-2073

ICEfaces AHS: Create only one MessageServiceClient per Servlet Context

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6, 1.6.1
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Environment:
      ICEfaces Asynchronous HTTP Server

      Description

      Currently with the ICEfaces Asynchronous HTTP Server 1.6.x releases, one MessageServiceClient is created for each Session, which results in one MessageReceiver thread being created for each Session as well! We should look into trying to achieve the goal of having only one MessageServiceClient per Servlet Context that can be used by all Sessions. Fixing this will rely on the refactoring that needs to be done mentioned in ICE-1675.

        Issue Links

          Activity

          Jack Van Ooststroom created issue -
          Jack Van Ooststroom made changes -
          Field Original Value New Value
          Assignee Jack van Ooststroom [ jack.van.ooststroom ]
          Hide
          Jack Van Ooststroom added a comment -

          Depends on the refactoring to be done for ICE-1675.

          Show
          Jack Van Ooststroom added a comment - Depends on the refactoring to be done for ICE-1675 .
          Jack Van Ooststroom made changes -
          Link This issue depends on ICE-1675 [ ICE-1675 ]
          Jack Van Ooststroom made changes -
          Fix Version/s 1.7 [ 10080 ]
          Priority Major [ 3 ] Critical [ 2 ]
          Hide
          Jack Van Ooststroom added a comment -

          Currently, ICEfaces AHS includes a quick fix that ensures the same goal. However, with the ICE-1675 being fixed we should revisit this to see if we can implement a nicer fix. Leaving the Fix Version at 1.7.

          Show
          Jack Van Ooststroom added a comment - Currently, ICEfaces AHS includes a quick fix that ensures the same goal. However, with the ICE-1675 being fixed we should revisit this to see if we can implement a nicer fix. Leaving the Fix Version at 1.7.
          Ken Fyten made changes -
          Priority Critical [ 2 ] Major [ 3 ]
          Ken Fyten made changes -
          Fix Version/s 1.7.1 [ 10122 ]
          Fix Version/s 1.7 [ 10080 ]
          Hide
          Jack Van Ooststroom added a comment -

          This one doesn't seem to be in scope for 1.7.1 for the moment. I talked to Mircea and to achieve this some additional refactoring needs to be done in the core framework. One of the ideas was to move the AdaptingServlet structure in front of the MainServlet so that it would be the "front" servlet.

          Show
          Jack Van Ooststroom added a comment - This one doesn't seem to be in scope for 1.7.1 for the moment. I talked to Mircea and to achieve this some additional refactoring needs to be done in the core framework. One of the ideas was to move the AdaptingServlet structure in front of the MainServlet so that it would be the "front" servlet.
          Jack Van Ooststroom made changes -
          Fix Version/s 1.7.1 [ 10122 ]
          Hide
          Jack Van Ooststroom added a comment -

          I can't recall when this was done exactly and what the related JIRA was for it, but currently there's only one MessageServiceClient being created in the MainServlet and shared amongst the various sessions. Marking this one as FIXED.

          Show
          Jack Van Ooststroom added a comment - I can't recall when this was done exactly and what the related JIRA was for it, but currently there's only one MessageServiceClient being created in the MainServlet and shared amongst the various sessions. 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:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: