Skip to content

iproto: new field in applier -> relay ACKs #3193

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
Closed
Tracked by #3373
TarantoolBot opened this issue Oct 13, 2022 · 0 comments · Fixed by #3433
Closed
Tracked by #3373

iproto: new field in applier -> relay ACKs #3193

TarantoolBot opened this issue Oct 13, 2022 · 0 comments · Fixed by #3433
Assignees
@xuniq
Labels
2.11 2.11 release and the associated technical debt iproto Related to the iproto protocol replication [area] Related to Replication

Comments

@TarantoolBot
Copy link
Collaborator

TarantoolBot commented Oct 13, 2022 •
edited by patiencedaur
Loading

Product: Tarantool
Since: 2.11
Audience/target: dev
Root document: https://www.tarantool.io/en/doc/latest/dev_guide/internals/box_protocol/
SME: @ Gerold103

Details

Do this after #3151 is merged.
Update the Replication and Keys sections.

The applier->relay channel (from replica back to master) is used
only for sending ACKs. Replication data goes the other way
(relay->applier).

These ACKs had 2 fields: IPROTO_VCLOCK (0x26) and
IPROTO_VCLOCK_SYNC (0x5a).

Now they have a new field: IPROTO_TERM (0x53). It is a unsigned
number containing box.info.election.term of the sender node
(applier, replica).
Requested by @Gerold103 in tarantool/tarantool@5449551.
@patiencedaur patiencedaur added iproto Related to the iproto protocol 2.11 2.11 release and the associated technical debt replication [area] Related to Replication labels Oct 14, 2022
@xuniq xuniq mentioned this issue Apr 4, 2023
Closed
7 tasks
@xuniq xuniq self-assigned this Apr 10, 2023
@xuniq xuniq mentioned this issue Apr 10, 2023
Merged
@xuniq xuniq linked a pull request Apr 10, 2023 that will close this issue
iproto: add new keys to Binary protocol page #3433
Merged
xuniq added a commit that referenced this issue Apr 13, 2023
@xuniq
iproto: add new keys to Binary protocol section (#3433)
8607c8a 
* Added IPROTO_VCLOCK_SYNC key. Updated vclock keys description. Updated heartbeats' description and byte codes.
* Renamed CLUSTER_UUID key to REPLICASET_UUID.
* Added IPROTO_BALLOT_BOOTSTRAP_LEADER_UUID and IPROTO_BALLOT_REGISTERED_REPLICA_UUIDS keys. Updated IPROTO_BALLOT description.
* Updated IPROTO_TERM key, updated heartbeats' description and byte codes.

Resolves #3166
Resolves #3294
Resolves #3193
Resolves #3299
@xuniq xuniq closed this as completed Apr 13, 2023
@xuniq xuniq changed the title New iproto field in applier -> relay ACKs iproto: new field in applier -> relay ACKs Apr 28, 2023
andreyaksenov pushed a commit that referenced this issue May 23, 2023
@xuniq @andreyaksenov
iproto: add new keys to Binary protocol section (#3433)
d7dd6a2 
* Added IPROTO_VCLOCK_SYNC key. Updated vclock keys description. Updated heartbeats' description and byte codes.
* Renamed CLUSTER_UUID key to REPLICASET_UUID.
* Added IPROTO_BALLOT_BOOTSTRAP_LEADER_UUID and IPROTO_BALLOT_REGISTERED_REPLICA_UUIDS keys. Updated IPROTO_BALLOT description.
* Updated IPROTO_TERM key, updated heartbeats' description and byte codes.

Resolves #3166
Resolves #3294
Resolves #3193
Resolves #3299
andreyaksenov pushed a commit that referenced this issue May 24, 2023
@xuniq @andreyaksenov
iproto: add new keys to Binary protocol section (#3433)
66710ac 
* Added IPROTO_VCLOCK_SYNC key. Updated vclock keys description. Updated heartbeats' description and byte codes.
* Renamed CLUSTER_UUID key to REPLICASET_UUID.
* Added IPROTO_BALLOT_BOOTSTRAP_LEADER_UUID and IPROTO_BALLOT_REGISTERED_REPLICA_UUIDS keys. Updated IPROTO_BALLOT description.
* Updated IPROTO_TERM key, updated heartbeats' description and byte codes.

Resolves #3166
Resolves #3294
Resolves #3193
Resolves #3299
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xuniq xuniq

Labels
2.11 2.11 release and the associated technical debt iproto Related to the iproto protocol replication [area] Related to Replication
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

iproto: add new keys to Binary protocol page tarantool/doc
3 participants
@TarantoolBot @patiencedaur @xuniq