Uploaded image for project: 'ActiveMQ .Net'
  1. ActiveMQ .Net
  2. AMQNET-649

bugs in PooledTaskRunner.cs

Attach filesAttach ScreenshotAdd voteVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • 1.7.2
    • None
    • ActiveMQ
    • None

    Description

      Although the 1.7.2 release of the AMQ .net client uses the DedicatedTaskRunner runner by default. We compile from source and use PooledTaskRunner.cs for performance reasons. However this code has some bugs. Namely the Shutdown() method will deadlock on a call to Thread.Sleep with infinite timespan. If you look at the Java source for PooledTaskRunner, it looks like the C# port is missing the thread synchronization logic that the Java implementation has. I've attached a version which corrects this and makes it in-line with the Java implementation and corrects the logic in Shutdown which needs to be a synchronized wait (e.g., Monitor.Wait())

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            kilroytrout JEFF ANDERSON

            Dates

              Created:
              Updated:

              Issue deployment