News for hackers. Stuff that matters.
Re:Chiselstore: an embeddable, distributed SQLite for Rust by malkia on Wednesday December 15, 2021 @07:23PM (#29573314)
I've looked at the https://github.com/chiselstrike/chiselstore/blob/main/proto/... - and calling the service "RPC" is not great if this is going to be bundled with other gRPC services - why not call it "chiselStore" or something more concrete?

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")

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.
Re:Chiselstore: an embeddable, distributed SQLite for Rust by malkia on Thursday December 16, 2021 @03:07PM (#29583203)
Didn't know that! (Good to know, and thanks!)