-
Notifications
You must be signed in to change notification settings - Fork 217
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
Support tso service #1038
Support tso service #1038
Conversation
93a760b
to
f827317
Compare
Signed-off-by: Ryan Leung <rleungx@gmail.com>
@binshi-bing Would you like to take a look? |
@@ -0,0 +1,42 @@ | |||
syntax = "proto3"; | |||
package tsopb; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to our last sync, my understanding is that we'll provide these APIs as storage interface used by tso service internal.
option (gogoproto.unmarshaler_all) = true; | ||
option (rustproto.lite_runtime_all) = true; | ||
|
||
service Tso { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This service is to provide TSO storage. Shall we call it TsoStorage to differentiate it from the TSO service which provide GetTSs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Though we don't need this change for now, this is my first review for PD.
Signed-off-by: Ryan Leung rleungx@gmail.com