also emit stats to global client

uber-archive / hyperbahn

Service discovery and routing for large scale microservice operations

MIT License

396 stars 57 forks source link

Closed blampe closed 8 years ago

blampe commented 8 years ago

Hello hi what is this.

@prashantv @Raynos @sloppyfocus

Raynos commented 8 years ago

lgtm.

We should do this for timing() and all other methods as well.

prashantv commented 8 years ago

+1, should also add this all the other methods

Raynos commented 8 years ago

lgtm. merge at will.

blampe commented 8 years ago

Very flakey tests :(

@Raynos after I land what's the release/deploy process?

Raynos commented 8 years ago

@blampe

Just run bash publish.sh; since it requires npm publish permissions I can run it on your behalf or you can login in as the uber npm user.

blampe commented 8 years ago

@Raynos I'll probably take you up on that, let's plan on releasing/deploying Monday. I'd like to look over your shoulder while you do it.

Raynos commented 8 years ago

Running

bash ./publish.sh patch

Created hyperbahn@2.14.36