rings: Prefix and Address factories #48

Merged
karasz merged 6 commits from pr-amery-rings into main 7 months ago
amery commented 7 months ago
Owner
  • ring 0 (backbone connecting all zones), 16 bits prefix.
  • ring 1 (per-zone lan), 20 bits prefix.
  • ring 2 (global services), 20 bits prefix.
  • ring 3 (pods of a region cluster), 12 bits prefix.
* ring 0 (backbone connecting all zones), 16 bits prefix. * ring 1 (per-zone lan), 20 bits prefix. * ring 2 (global services), 20 bits prefix. * ring 3 (pods of a region cluster), 12 bits prefix.
amery added 6 commits 7 months ago
378bab2f96 rings: introduce RegionID, ZoneID and NodeID
52e1195139 rings: introduce generic ErrOutOfRange() factory
859e307ac4 rings: RingOnePrefix()/RingOneAddress()
127a5830cc rings: RingZeroPrefix()/RingZeroAddress()
1d7b37c3e7 rings: RingThreePrefix()
16f6f1141c rings: RingTwoPrefix()
amery requested review from karasz 7 months ago
amery added 1 commit 7 months ago
349b5a7df1 rings: DecodeRingZeroAddress() and DecodeRingOneAddress()
amery force-pushed pr-amery-rings from 349b5a7df1 to 16f6f1141c 7 months ago
amery added 4 commits 7 months ago
394a84c3ab rings: RingOnePrefix()/RingOneAddress()
0f177acf57 rings: RingZeroPrefix()/RingZeroAddress()
6142d0f7f0 rings: RingThreePrefix()
72a2468a10 rings: RingTwoPrefix()
Poster
Owner

moved RingFooPrefix() and RingFooAddress() to encode.go in preparation for decode.go @karasz

moved `RingFooPrefix()` and `RingFooAddress()` to `encode.go` in preparation for `decode.go` @karasz
karasz approved these changes 7 months ago
karasz left a comment
Owner

LGTM

LGTM
karasz merged commit e1186975a6 into main 7 months ago
karasz deleted branch pr-amery-rings 7 months ago

Reviewers

karasz approved these changes 7 months ago
The pull request has been merged as e1186975a6.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.