Re:Chiselstore: an embeddable, distributed SQLite for Rust
by returnfalse on Thursday December 16, 2021 @05:02AM
(#29576732)
They can replace Void with AppendEntriesResponse anytime they want and that won't break anything since the change is wire compatible.
on a 2nd note, each method should have it's own Request Response, and ideally they should be suffixed same way. Returning same proto for multiple methods might break future compatibility - e.g. what if AppendEntries needs to return somehing else than Void? - ideally you make it from the start to return an empty AppendEntriesResponse, and then you can extend that proto.
https://developers.google.com/protocol-buffers/docs/proto3#u...
(edit again, also the package should not be called just "proto")