amery
/
jpictl
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
Labels Milestones

cluster: use typed IDs, introduce Region.ID, and pre-compute primary region #49

Merged
karasz merged 7 commits from pr-amery-regions into main 1 month ago
Conversation 1 Commits 7 Files Changed 13
amery commented 1 month ago
Owner
There is no content yet.
amery added 7 commits 1 month ago
d98c879409 build-sys: use revive 1.3.7 instead of master
f4b837f3e0 rings: add String() to RegionID, ZoneID, NodeID
bb40f395d9 rings: fix NodeZeroMax/NodeMax to not accept .255
7f849415ca cluster: use rings.ZoneID and rings.NodeID types
410a9f8e30 cluster: run initRegions() before scanZoneIDs()
c84318448c cluster: pre-compute Zone's primary region
amery added 7 commits 1 month ago
83921c1e13 build-sys: use revive 1.3.7 instead of master
ff8f2c6ea1 rings: add String() to RegionID, ZoneID, NodeID
6d89e0ea3c rings: fix NodeZeroMax/NodeMax to not accept .255
99dece1e43 cluster: use rings.ZoneID and rings.NodeID types
cede95e119 cluster: run initRegions() before scanZoneIDs()
982e16581c cluster: pre-compute Zone's primary region
karasz approved these changes 1 month ago
karasz left a comment
Owner

LGTM

LGTM
karasz merged commit 932a41a3ac into main 1 month ago
karasz deleted branch pr-amery-regions 1 month ago
amery commented 1 month ago
Poster
Owner

v0.8.2 tagged

`v0.8.2` tagged

Reviewers

karasz approved these changes 1 month ago
The pull request has been merged as 932a41a3ac.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
karasz
Labels
Apply labels
Clear labels
No items
No Label
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'pr-amery-regions'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes