search

uber / tchannel

network multiplexing and framing protocol for RPC
MIT License
1.15k stars 129 forks source link

Larch flush #1353

Closed jcorbin closed 9 years ago

jcorbin commented 9 years ago

r @Raynos, @rf

jcorbin commented 9 years ago

@Raynos where should we add call site to the new logtron logger.close?

rf commented 9 years ago

:+1:

The logtron backend should have a destroy method, that can close logtron. So clients.js will call Larch#destroy -> ReservoirBackend#destroy -> LogtronBackend#destroy -> Logtron#close

Raynos commented 9 years ago

lgtm.