The great Hyperbahn migration - Githubissues

uber / tchannel

network multiplexing and framing protocol for RPC

MIT License

1.15k stars 129 forks source link

The great Hyperbahn migration #1334

Closed Raynos closed 8 years ago

Raynos commented 8 years ago

This PR moves a lot of code out of the TChannel package into the Hyperbahn package.

I'll summary in another comment.

r: @jcorbin @kriskowal @rf

Raynos commented 8 years ago

Files moved:

service-proxy.js ( ServiceProxy )
egress-nodes.js ( EgressNode )
handler.js ( HyperbahnHandler )
All the hyperbahn client tests
circuits.js ( Circuits ) , state_machine.js, states.js
rate_limiter.js ( RateLimiter )

Files deleted:

permission-cache.js ( PermissionCache )
test/lib/fake-egress-nodes.js
test/lib/hyperbahn-cluster.js
test/lib/relay-network.js

Files not moved:

hyperbahn-client.js ( HyperbahnClient )

Weird dependencies introduced

tchannel/lib/event_emitter
tchannel/lib/stat
tchannel/lib/statsd-clean

Raynos commented 8 years ago

TODO:

[ ] fix hyperbahn/test/circuits/circuits.js which I broke for "move fast"