Uploaded image for project: 'Commons JCS'
  1. Commons JCS
  2. JCS-171

Multiple CacheEventQueue.QProcessor spawned for the same cache region

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • jcs-2.0
    • jcs-2.1
    • Composite Cache
    • None

    Description

      I noticed that running on new version of JCS I get multiple CacheEventQueue.QProcessor thread. They spawn from time to time.

      I've checked recent changes and changes few things in r1774925 look suspicious:
      1. In previous code we spawned a new thread in synchronized section. This got us a guarantee, that there will be no two threads trying to spawn a new thread in the same time. Maybe some locking is needed around thread creation?

      2. QProcessor uses isAlive() method. But this is defined by Thread.isAlive() while it should probably check for CacheEventQueue.this.isAlive()

      Attachments

        1. CacheEventQueue.patch
          0.8 kB
          Wiktor N
        2. jcs-perf-test.zip
          1.45 MB
          Wiktor N

        Activity

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

          People

            tv Thomas Vandahl
            wiktorn Wiktor N
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment