-
Notifications
You must be signed in to change notification settings - Fork 164
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
undefined: zipkingoopentracing.NewHTTPCollector #45
Comments
@blockerr current zipkin not compatible with this old code, i get it worked by manually copying old github.com/openzipkin/zipkin-go-opentracing to current pkg. |
I have solved this problem in the PR #43 : |
I can't visit "github.com/openzipkin/zipkin-go-opentracing" ,and I followed your instructions replacing the import package after my "go get ...", it still indicates unresolved reference |
You should visit "https://github.com/openzipkin-contrib/zipkin-go-opentracing" |
I am getting this error when trying to work through the example: @GrantZheng - is it possible to use your PR right away? I see you have https://github.com/GrantZheng/kit but it's not clear how to use your version over this one? If I try to |
Thank you very much for pointing out the problem. When I run "go install github.com/GrantZheng/kit", it installs the older package "github.com/kujtimiihoxha/kit", I haven't found a better way to solve this problem. And I update the README to introduce the installing way: git clone https://github.com/GrantZheng/kit.git
cd kit
go install |
I follow the example but get this error when run : go run hello/cmd/main.go
hello/cmd/service/service.go:56:21: undefined: zipkintracer.NewHTTPCollector
hello/cmd/service/service.go:62:15: undefined: zipkintracer.NewRecorder
hello/cmd/service/service.go:63:17: undefined: zipkintracer.NewTracer
The text was updated successfully, but these errors were encountered: