Details

    • Sub-task
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.99.0, 0.98.7, 0.94.24
    • None
    • None
    • Reviewed

    Description

      apurtell pointed out a potential race condition in RecoverableZookeeper.checkZk() that I introduced in parent.

      If multiple threads would enter that method at the same time without a valid Zookeeper reference, we could leak Zookeeper objects.

      Since this is not a on a hot code path we should just synchronize the two involved methods.

      Attachments

        1. 11923-trunk.txt
          2 kB
          Lars Hofhansl
        2. hbase-11923-0.94.patch
          1 kB
          Enis Soztutar

        Activity

          People

            larsh Lars Hofhansl
            larsh Lars Hofhansl
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: