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
CarbonV2 gRPC streaming render #476
Conversation
c22e4f4
to
1accaf4
Compare
there are a few deep source check errors, can you check try resolving it or silence it with |
e2c9162
to
504d7ad
Compare
after you mark this pr as ready for review, I guess I can approve it. just a simple recap and being a paranoid in case I missed it, no backward-incompatibilities are expected, right? |
Excellent! I will mark it as ready for review as soon as go-graphite/protocol#12 is merged and the lib ref here is updated. Can you check that out as well?
That's right. The old proto messages stay the same (new messages and service are added), old API works in the same way as before alongside with the new gRPC implementation. |
I have approved the protocol pr. let's give Vladimir a few hours to see if he has opinions there. |
This PR
This is manually tested in container environment.
Notes: