Uploaded image for project: 'JBoss Enterprise Application Platform'
  1. JBoss Enterprise Application Platform
  2. JBEAP-11835

(7.0.z) UNDERTOW-1088 SPDY: SpdyClientConnection.sendRequest() shouldn't be synchronized

    XMLWordPrintable

Details

    • EAP 7.0.8

    Description

      I believe commit https://github.com/undertow-io/undertow/commit/595322a869494ba03610773b2e8084da8d4a8372 should be reverted.

      The issue is that sendRequest() calls clientCallback.completed() which is on the caller side and can block for a long time, thus blocking other users/threads from calling sendRequest().

      So, I think that any synchronization must be done by the callers. Or it must be implemented different than just making this whole method synchronized.

      Attachments

        Issue Links

          clones

          Bug - A problem that impairs or prevents the functions of the product. UNDERTOW-1088 SPDY: SpdyClientConnection.sendRequest() shouldn't be synchronized

          • Major - To be worked after higher priority work items are addressed. When the severity is high and the effort to change it is low to moderate, this term is chosen. Issues in this category likely have an existing workaround but implementation or execution may be non-trivial.
          • Resolved
          is incorporated by

          Component Upgrade - A task tracking an update to a bundled component or revision of component upstream of the project. JBEAP-11773 [GSS](7.0.z) Upgrade undertow from 1.3.29.Final to 1.3.31.Final

          • Major - To be worked after higher priority work items are addressed. When the severity is high and the effort to change it is low to moderate, this term is chosen. Issues in this category likely have an existing workaround but implementation or execution may be non-trivial.
          • Verified

          Activity

            People

              sdouglas1@redhat.com Stuart Douglas
              thofman Tomas Hofman
              Michael Cada Michael Cada
              Michael Cada Michael Cada
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: