Uploaded image for project: 'Spring Web Services'
  1. Spring Web Services
  2. SWS-564

CommonsHttpMessageSender no longer properly shuts down MultiThreadedHttpConnectionManager

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.5.9
    • Component/s: None
    • Labels:
      None

      Description

      As of revision 1414 of CommonsHttpMessageSender, the MultiThreadedHttpConnectionManager never gets it's shutdown method invoked.

      It appears that the prior revision of CommonsHttpMessageSender implemented DisposableBean, and invoked this properly.
      SWS-482 removed the destroy method to cope with a problem report, but the result is worse.

      If you use MultiThreadedHttpConnectionManager, you have to call it's shutdown method, or it will block application exit.
      For example, I have a class with a main method that uses WebServiceTemplate along with CommonsHttpMessageSender that is invoked via an ant task. The main method will complete, but the JVM cannot exit.

      Lets revisit SWS-482 and come up with a means for resolving the issue appropriately.
      The reporter of that issue mentions that:
      "it happens it is always after we redeploy our application (weblogic 9.2MP2)".

        Issue Links

          Activity

          nblair Nicholas Blair created issue -
          arjen.poutsma Arjen Poutsma made changes -
          Field Original Value New Value
          Fix Version/s 1.5.9 [ 11296 ]
          arjen.poutsma Arjen Poutsma made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          arjen.poutsma Arjen Poutsma made changes -
          Link This issue is related to SWS-482 [ SWS-482 ]
          arjen.poutsma Arjen Poutsma made changes -
          Time Spent 4h 5m [ 14700 ]
          Remaining Estimate 0d [ 0 ]
          arjen.poutsma Arjen Poutsma made changes -
          Status In Progress [ 3 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          arjen.poutsma Arjen Poutsma made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              arjen.poutsma Arjen Poutsma
              Reporter:
              nblair Nicholas Blair
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0d
                0d
                Logged:
                Time Spent - 4h 5m
                4h 5m