Skip to content

BP-41: Separate BookieId from Bookie Network Address #2396

Closed

Description

@eolivelli

BP
This is the master ticket for tracking BP-41: Separate BookieId from Bookie Network Address

We want to separate the concepts of BookieId from BookieSocketAddress.
Currently (up to 4.11.x) there is a too strict coupling from the ID of a Bookie and its location:

  • We are storing BookieSocketAddresses (simply a hostname:port or ip:port pair) into Ledger Metadata
  • The Identity of a bookie is bound to the network address of the primary endpoint of the Bookie ('bookie-rpc' endpoint in term of BP-38)

This fact is very awkward, because:

  • you cannot change the network address of a Bookie
  • you cannot use a custom Id for the bookie, something that would be more useful in the context of the deployment of the bookie

The proposal will run thru this problems and it will propose to separate the concept of BookieId from BookieSocketAddress.

The implementation will be divided in two separate works:

  • client side changes: drop BookieSocketAddress from LedgerMetadata and use BookieId in EnsemblePlacemenetPolicies
  • bookie side changes: allow to configure a store a BookieId that is independent from the current BookieSocketAddress

Proposal PR #2397

Patches:

Activity

  1. self-assigned this
    on Aug 8, 2020
  2. sijie commented on Aug 10, 2020

    @sijie
    Member

    @eolivelli this is a great initiative!

  3. added a commit that references this issue on Sep 19, 2020
  4. added a commit that references this issue on Oct 20, 2020
  5. jiazhai commented on Nov 16, 2020

    @jiazhai
    Member

    Would like to close this issue. since all related PRs are merged

  6. added this to the 4.12.0 milestone on Nov 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions