-
Notifications
You must be signed in to change notification settings - Fork 7
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
Implemented tcommit.Manager and tcommit.Resource as gRPC services #242
base: master
Are you sure you want to change the base?
Conversation
By the way, the year of the header should be changed |
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.
@askerka Hi! First of all could you check the test logs please? Some tests are failed.
5a28d01
to
a04aeb8
Compare
@ivanbukhtiyarov Hi! The tests are fixed. Please, have a look |
a04aeb8
to
441f408
Compare
For #238 - Implemented `tcommit.Manager` and `tcommit.Resource` as gRPC services, separated service and client packages, added `marshall` package to convert struct between the application and proto definitions, and updated `Finish` and `Begin` client's methods with `marshall` package
441f408
to
20ce599
Compare
Fixed the code style issues, added |
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.
@askerka looks good to me, thanks.
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.
@askerka this PR contains too many changes, which are not related to origin ticket. Could you please keep only relevant #238 changes there?
@g4s8 There are only two lines that are not relevant: Do you want me to delete them? Could you please guide me on what changes do you think are not relevant? |
For #238 - Implemented
tcommit.Manager
andtcommit.Resource
as gRPC services.Separated service and client packages.
Added
marshall
package to convert struct between the application and proto definitions.Updated
Finish
andBegin
client's methods withmarshall
package