Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-16064 Tackle random Thread().start() usage
  3. SOLR-16065

Run checkIfIamStillLeader outside of a Thread in the overseer

    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Open
    • Major
    • Resolution: Unresolved
    • None
    • None
    • None
    • None

    Description

      Currently at the end of Overseer.run(), a thread is created to continuously check if the overseer is still the leader.

      //do this in a separate thread because any wait is interrupted in this main thread
      new Thread(this::checkIfIamStillLeader, "OverseerExitThread").start();

      I'm currently unsure as to why the check needs to ignore any interruptions, but this is causing failing tests due to this thread being leaked.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              houston Houston Putman
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated: