Commit Graph

592 Commits

Author SHA1 Message Date
David Garcia Quintas cd21266b5f Merge branch 'master' of github.com:grpc/grpc into codegen_lib 2016-03-15 17:10:45 -07:00
David Garcia Quintas 9eef377865 clang-format 2016-03-09 11:31:02 -08:00
David Garcia Quintas 60ee8dd2fc docstrings 2016-03-08 17:21:42 -08:00
David Garcia Quintas b942640497 clang-format 2016-03-08 16:25:11 -08:00
David Garcia Quintas be25493398 codegen interface usage of gpr_inf_future 2016-03-08 16:24:35 -08:00
David Garcia Quintas ef6848fe5b reverted gpr_ time changes 2016-03-08 16:05:30 -08:00
David Garcia Quintas 723af53419 moved some files around 2016-03-08 14:26:50 -08:00
David Garcia Quintas 8c3d9943f6 compiles! 2016-03-08 00:07:14 -08:00
David Garcia Quintas e1ce31eda3 wip. cq refactored 2016-03-07 18:19:12 -08:00
David Garcia Quintas 6848c4e145 wip. prior to cq refactoring 2016-03-07 17:10:57 -08:00
Sree Kuchibhotla b0d1567e8e Revert "Revert "Properly integrate async API with server-side cancellations."" 2016-03-07 10:51:02 -08:00
David Garcia Quintas b07938fa22 added missing initializer for global proto_serializer holder to codegen lib 2016-03-06 20:07:30 -08:00
Sree Kuchibhotla edd96e4926 Revert "Properly integrate async API with server-side cancellations." 2016-03-04 14:32:50 -05:00
Vijay Pai 13ee2f2df3 Properly integrate async API with server-side cancellations.
There is a comment above IsCancelled that says when it is ok to use this.
2016-03-03 13:57:32 -08:00
David Garcia Quintas 7f7181ffc5 fix comments 2016-03-02 20:42:16 -08:00
David Garcia Quintas c4f4f7ede9 clang-format 2016-03-02 20:37:24 -08:00
David Garcia Quintas f588aeb1e2 introduced workaround for proto_utils 2016-03-02 20:33:28 -08:00
David Garcia Quintas 45add8a445 Merge branch 'master' of github.com:grpc/grpc into alarm_cpp 2016-02-22 23:15:28 -08:00
Craig Tiller c8d34d50a2 Merge pull request #5287 from yang-g/global_callback_magic
Add a way to override channel arguments for server creation
2016-02-22 22:05:32 -08:00
David Garcia Quintas 0160873273 PR comments addressed 2016-02-22 17:49:45 -08:00
David Garcia Quintas ef02fb9d6f Merge branch 'master' of github.com:grpc/grpc into alarm_cpp 2016-02-22 17:41:41 -08:00
David Garcia Quintas bcd78fe845 Added missing virtual dtor 2016-02-19 15:39:16 -08:00
David Garcia Quintas 4105505b33 Fixed refactoring of grpc_arg pointer vtable 2016-02-17 18:23:15 -08:00
David Garcia Quintas 10e269897f Merge branch 'master' of github.com:grpc/grpc into alarm_cpp 2016-02-17 17:43:42 -08:00
David Garcia Quintas 5e07d76a86 Made Alarm's constructor a template for deadline 2016-02-17 17:10:41 -08:00
Sree Kuchibhotla ac94c430d2 Merge pull request #5186 from yang-g/inherit_from_grpc_library
Provide explicit API for user to set user agent string prefix
2016-02-17 15:51:57 -08:00
yang-g eb62c94338 Add a way to override channel arguments for server creation 2016-02-17 15:37:25 -08:00
vjpai 50d7e3c4eb I merged this without checking the copyright. My mistake... 2016-02-12 23:09:04 -08:00
Craig Tiller d4f2604b53 Small tweak to make some customer code compile 2016-02-12 15:00:22 -08:00
yang-g a3c968d9c2 Merge remote-tracking branch 'upstream/master' into inherit_from_grpc_library 2016-02-12 14:54:16 -08:00
David G. Quintas 3f19f42771 Merge pull request #5052 from ctiller/compress
Fix race condition in transport API
2016-02-11 18:16:11 -08:00
Craig Tiller 1362c24108 Merge pull request #5193 from vjpai/alarm_mods
Make C++ alarm class usable and testable
2016-02-11 12:36:07 -08:00
Craig Tiller c0473cc823 Merge github.com:grpc/grpc into compress 2016-02-11 08:51:16 -08:00
Vijay Pai 810a13e2d3 Remove a small allocation, at the cost of a little uglier header file.
This commit may or may not be a great tradeoff.
2016-02-11 03:30:10 -08:00
David Garcia Quintas 26a9848416 Removed outdated mention of AsynchronousService 2016-02-11 00:45:00 -08:00
vjpai 04e992a750 Make C++ alarm class usable and testable - there were some build issues and
it was not actually even included in the library or the test suite.
2016-02-10 16:58:38 -08:00
yang-g 1945ae4285 Fix copyright 2016-02-10 12:48:32 -08:00
yang-g d59ad7ef39 Provide explicit API for user to set user agent string prefix 2016-02-10 12:42:53 -08:00
David Garcia Quintas ab5da5ef7a Added missing codegen includes 2016-02-09 20:31:37 -08:00
Craig Tiller 64914ae6b5 Merge pull request #5128 from yang-g/inherit_from_grpc_library
Add missing base class
2016-02-08 15:14:29 -08:00
Vijay Pai 30bf3ea4b9 clang-format 2016-02-08 13:21:07 -08:00
yang-g 46d22d5fa7 add missing base class 2016-02-08 12:27:10 -08:00
Vijay Pai c593ca0171 reset the shared_ptr 2016-02-08 11:38:42 -08:00
Vijay Pai bedf57fe8c Ref the collection only when it will be used (and later finalized) 2016-02-05 16:45:54 -08:00
Vijay Pai 2b5638668e Drop the ref 2016-02-05 15:11:36 -08:00
Vijay Pai 5506bea349 Undo much of the previous commits so that only CallOpSet groups
with a Sneaky member are in a collection.
2016-02-05 14:23:24 -08:00
Vijay Pai 7e3f9b0178 Forgot to include these edits in the last commit 2016-02-05 13:43:17 -08:00
Vijay Pai 38004a8e39 Wrap groups of related CallOpSet's into a ref-counted structure
whenever appropriate so as to avoid any unintentional free-before-use
problems.

Potential performance issue: this triggers an additional allocation
for each Async call initiation, along with the cost of ref-counting
shared_ptr . But this is worth it for the additional safety provided
here without any change to the exposed C++ API.
2016-02-05 13:40:22 -08:00
Craig Tiller 97cb41cfaf Merge github.com:grpc/grpc into compress 2016-02-05 10:44:56 -08:00
Craig Tiller a44cbfc11c Fix race condition in transport API
Specifically:

Receiving trailing and initial metadata had to be published in
lock-step.
=> If we wanted trailing metadata, we might not get initial metadata processed
   until messages arrived.
=> Compression code had no idea what codec to use.

To fix it, publish initial metadata as soon as it's ready (this is a
transport API change).

Requires changes to grpc_call to ensure ordering in processing initial
metadata and messages (one may be delayed).

Exposed at least some bugs in C++ where we never read initial metadata.

I expect at least one more similar bug.
2016-02-03 16:02:49 -08:00