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

Add persistable bookie status #265

Closed
sijie opened this issue Jul 20, 2017 · 0 comments
Closed

Add persistable bookie status #265

sijie opened this issue Jul 20, 2017 · 0 comments
Assignees
@sijie
Labels
area/bookie release/4.6.0 type/feature
Milestone
4.6.0

Comments

@sijie
Copy link
Member

sijie commented Jul 20, 2017

FEATURE REQUEST

  1. Please describe the feature you are requesting.

Currently bookie status is transient. It is a bit hard for management tooling. This feature is to add persistable bookie status.

Summary of this change:

- Add support for persisting bookie status
- Add configuration to enable/disable this feature
- Add test cases
- Improve bookie status stat
  1. Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have). Are you currently using any workarounds to address this issue?

blocker

  1. Provide any additional detail on your proposed use case for this feature.

This change is contributed by @yzhang and ported from Twitter branch.
@sijie sijie added this to the 4.5.0 milestone Jul 20, 2017
@sijie sijie modified the milestones: 4.6.0, 4.5.0 Aug 1, 2017
@sijie sijie self-assigned this Oct 27, 2017
@sijie sijie closed this as completed in 3a20fe0 Oct 27, 2017
test-things-on-github pushed a commit to test-things-on-github/bookkeeper that referenced this issue Nov 2, 2017
@sijie
ISSUE apache#265: Add persistable bookie status
3c49322 
Descriptions of the changes in this PR:

- Add support for persisting bookie status
- Add configuration to enable/disable this feature
- Add test cases
- Improve bookie status stat

It also includes changes for BOOKKEEPER-754 (flush ledger storage after replaying journal)

Author: Sijie Guo <sijie@apache.org>
Author: Yiming Zang <yzang@twitter.com>

Reviewers: Yiming Zang <yzang2016@gmail.com>, Enrico Olivelli <eolivelli@gmail.com>

This closes apache#266 from sijie/server_side_crc32, closes apache#265
athanatos pushed a commit to athanatos/bookkeeper that referenced this issue Jan 25, 2019
@hanm
ZOOKEEPER-2796: Fix broken test testCreateNodeWithoutData caused by Z…
ce0aa2b 
…K-2757.

ZK-2757 introduces a new MalformedPathException which intercepts and wraps the IllegalArgumentException that this test was expecting.

Author: Michael Han <hanm@apache.org>

Closes apache#265 from hanm/ZOOKEEPER-2796
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sijie sijie

Labels
area/bookie release/4.6.0 type/feature
Projects
None yet
Milestone
4.6.0
Development

No branches or pull requests
1 participant
@sijie