uber / tchannel-node

MIT License
203 stars 40 forks source link

Fixup benchmarks Makefile to work out of box. #336

Closed Raynos closed 8 years ago

Raynos commented 8 years ago

The Makefile is not configured to work out of the box.

There was also one bug in the benchmark bootstrapper that caused the tracing benchmark to fail

I've also turned on 100% trace reporting instead of 1% for the tracing benchmark...

So currently when benchmarking wrt to tracing:

The four modes do the following on my laptop,

We currently do not have a make take_no_trace so you have to hand edit and s/trace: true/trace: false/.

r: @rf @oibe

oibe commented 8 years ago

@Raynos Thanks a lot for the analysis. lgtm. I will use this branch as a starting point for finishing up the opentracing diff that Rooz started.

Raynos commented 8 years ago

All files effected are in the benchmarks/ folder so I'm going to go ahead and land this.