Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ZOOKEEPER-4712: Fix partially shutdown of ZooKeeperServer and its processors #2154

Merged
merged 9 commits into from
Sep 20, 2024
Merged

ZOOKEEPER-4712: Fix partially shutdown of ZooKeeperServer and its processors #2154

merged 9 commits into from
Sep 20, 2024

Commits on Aug 12, 2024

  1. Avoid NPE when closing SendAckRequestProcessor after other failure

    @jonmv
    jonmv committed Aug 12, 2024
    Configuration menu
    Copy the full SHA
    c4e7e42 View commit details
    Browse the repository at this point in the history

  2. Fix shutdown method overrides, so, e.g., SyncRequestProcessor is shut… 

    … down from Learner
    @jonmv
    jonmv committed Aug 12, 2024
    Configuration menu
    Copy the full SHA
    8d055cc View commit details
    Browse the repository at this point in the history

  3. Remove shutdown(boolean) from OZKS, which was identical to that of LZKS

    @jonmv
    jonmv committed Aug 12, 2024
    Configuration menu
    Copy the full SHA
    d82213e View commit details
    Browse the repository at this point in the history

  4. Correctly propagate ZKDB.clear during shutdown of not-running ZKS chi… 

    …ldren
    @jonmv
    jonmv committed Aug 12, 2024
    Configuration menu
    Copy the full SHA
    13001bf View commit details
    Browse the repository at this point in the history

  5. Shut down sync processor before parent ZKS; it uses TXN-log during pa… 

    …rtial shutdown
    @jonmv
    jonmv committed Aug 12, 2024
    Configuration menu
    Copy the full SHA
    5d7aa33 View commit details
    Browse the repository at this point in the history

Commits on Sep 19, 2024

  1. ZOOKEEPER-4712: Add test case to assert request processors got shutdo… 

    …wn in ZooKeeperServer::shutdown
    @kezhuw
    kezhuw committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    41e6e95 View commit details
    Browse the repository at this point in the history

Commits on Sep 20, 2024

  1. Merge pull request #1 from kezhuw/ZOOKEEPER-4712-test-case 

    ZOOKEEPER-4712: Add test case to assert request processors got shutdown in ZooKeeperServer::shutdown
    @jonmv
    jonmv authored Sep 20, 2024
    Configuration menu
    Copy the full SHA
    56e959d View commit details
    Browse the repository at this point in the history

  2. Refactor shutdown to avoid requiring repeated logic in children

    @jonmv
    jonmv committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    c0682d7 View commit details
    Browse the repository at this point in the history

  3. Fix spotbugs failure

    @kezhuw
    kezhuw authored Sep 20, 2024
    Configuration menu
    Copy the full SHA
    3d6f619 View commit details
    Browse the repository at this point in the history