Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace ServiceLoader code by TracerResolver call. #12

Merged

Conversation

sjoerdtalsma
Copy link
Collaborator

This fixes #11.

@sjoerdtalsma
Copy link
Collaborator Author

Tests (including the tracer lookup) still pass. I'll probably merge this later today if there are no objections.

Since this requires OT-api 0.22.0:
Merge remote-tracking branch 'remotes/upstream/master' into tracerresolver

# Conflicts:
#	pom.xml
Since this requires OT-api 0.22.0:
Merge remote-tracking branch 'remotes/upstream/master' into tracerresolver

# Conflicts:
#	pom.xml
@sjoerdtalsma sjoerdtalsma merged commit 9cd8a16 into opentracing-contrib:master May 5, 2017
@sjoerdtalsma sjoerdtalsma deleted the tracerresolver branch May 5, 2017 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace Tracer resolution by TracerResolver
1 participant