-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
Completion queue binding for new requests API change #1493
Completion queue binding for new requests API change #1493
Conversation
Move completion queue binding for new requests to the new request request time, not server instantiation time.
…-the-stream Conflicts: src/cpp/server/server.cc
…thub.com:ctiller/grpc into churn-churn-churn-the-api-gently-down-the-stream
Green - we should probably announce before merging, but could you take a quick look? |
@nathanielmanistaatgoogle, @soltanmm - please look at Python changes @yang-g - please look at everything |
As with that other C core API change in the last week or two: LGTM but @soltanmm is actively working in the area right now and he must also review and approve. |
LGTM |
1 similar comment
LGTM |
…-the-stream Conflicts: test/cpp/end2end/async_end2end_test.cc
LGTM |
@soltanmm - thoughts? If possible I'd like to have this in Monday. |
Apologies for the delay - I've been in my own little Python world for a bit. LGTM. |
Such green. Wow. |
…y-down-the-stream Completion queue binding for new requests API change
Move completion queue binding for new requests to the new request
request time, not server instantiation time.