-
Notifications
You must be signed in to change notification settings - Fork 103
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
WIP Thread sk, rk, and rd through reqmeta to transports #324
Conversation
@kriskowal, thanks for your PR! By analyzing the annotation information on this pull request, we identified @abhinav, @breerly and @bombela to be potential reviewers |
Blocking on integration for the routing key property in tchannel-go. |
@@ -17,7 +17,7 @@ import: | |||
subpackages: | |||
- gomock | |||
- package: github.com/uber/tchannel-go | |||
version: ^1.1.0 | |||
version: ^1.2.0 |
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 can be just ^1.2
, no reason to specify the patch.
* Context is no longer a reqmeta property * Thread shard key, routing key, and routing delegate * Uprev tchannel
|
||
GetProcedure() string | ||
GetHeaders() Headers | ||
GetShardKey() string | ||
GetRoutingKey() string | ||
GetRoutingDelegate() string |
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.
Do servers have no need of accessing this information?
* master: (31 commits) Prepare v0.2.0 (#339) crossdock: Add an apachethrift crossdock behavior (#335) Use thriftrw-go >= 0.2, < 0.3 (#336) Thread sk, rk, and rd through reqmeta to transports (#324) version-check: Fix for OSX (#334) thrift/outbound: Support TMultiplexedProtocol (#333) transports: merge common error handling. (#322) Adding version.go (#331) Fix go docs link travis: Split crossdock and linter out (#323) Catch (recover) panics from user handlers. (#315) thrift: Suggest plugins over --yarpc (#320) Support open tracing in HTTP and TChannel (#307) thrift: Implement ThriftRW plugin (#317) Reformat changelog (#316) Rename some behaviors with more obvious names. (#311) Changelog for v0.1.1 Go back to regular github.com/yarpc/yarpc-go import path (#305) changelog (#303) deps - Pin to thriftrw ~0.1 (#301) ...
This adds support for three RPC request properties: shard key, routing key, and routing delegate, writing them to the wire in the way appropriate per transport.
r @prashantv @breerly