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.

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

Akka.Cluster: need to log send time in verbose heartbeat messages #6496

Closed
Aaronontheweb opened this issue Mar 7, 2023 · 1 comment · Fixed by #6548 or #6551
Closed

Akka.Cluster: need to log send time in verbose heartbeat messages #6496

Aaronontheweb opened this issue Mar 7, 2023 · 1 comment · Fixed by #6548 or #6551

Comments

@Aaronontheweb
Copy link
Member

// TODO: log response time and validate sequence nrs once serialisation of sendTime is released

@Aaronontheweb
Copy link
Member Author

The serializer for these values was introduced in v1.4.19 and this serializer was enabled by default in Akka.NET 1.5.0

/**
* Prior to version 1.4.19
* Heartbeat
* Sends an Address
* Version 1.4.19 can deserialize this message but does not send it
*/
message Heartbeat {
Akka.Remote.Serialization.Proto.Msg.AddressData from = 1;
int64 sequenceNr = 2;
sint64 creationTime = 3;
}
/**
* Prior to version 1.4.19
* HeartbeatRsp
* Sends an UniqueAddress
* Version 1.4.19 can deserialize this message but does not send it
*/
message HeartBeatResponse {
UniqueAddress from = 1;
int64 sequenceNr = 2;
int64 creationTime = 3;
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant