/zipkin

Change StatsService to ServiceStatsFilter #168

Closed
wants to merge 2 commits into
from

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@franklinhu @johanoskarsson
@franklinhu
Contributor

franklinhu commented Sep 27, 2012

It makes more sense for the stats to be a Filter rather than a Service
@johanoskarsson

This comment has been minimized.

Show comment Hide comment
@johanoskarsson

johanoskarsson Sep 27, 2012

Contributor

+1

Contributor

johanoskarsson commented Sep 27, 2012

+1

@franklinhu franklinhu closed this in 282ab23 Sep 27, 2012

adriancole added a commit that referenced this pull request Jun 3, 2016

@adriancole
Merge pull request #168 from openzipkin/brave-3.6 
Updates to Brave 3.6 and makes self-tracing optional
89f2c62
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment