Uploaded image for project: 'Shiro'
  1. Shiro
  2. SHIRO-514

ExecutorServiceSessionValidationScheduler should create threads with a configurable name

Attach filesAttach ScreenshotVotersWatch issueWatchersLinkUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    Description

      While looking through thread dumps I noticed a bunch of 'Thread-[number]' threads. From the heap dump I took I identified them as coming from the ExecutorServiceSessionValidationScheduler, which creates new threads without setting their name.

      This is confusing, so attached patch makes the thread name (or better: a prefix of it) configurable, and sets a reasonable default.

      Attachments

        Issue Links

        is related to

        Bug - A problem which impairs or prevents the functions of the product. SHIRO-515 ExecutorServiceSessionValidationScheduler leaks resources due to improper synchronization

        • Critical - Crashes, loss of data, severe memory leak.
        • Resolved

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

          People

            Unassigned Unassigned
            ankon Andreas Kohn
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment