Coder Social home page Coder Social logo

Comments (13)

sitano avatar sitano commented on June 18, 2024 2

@andream16 you can make it work with pinning the thrift version in your deps (glide or whatever) to something working like:

- package: github.com/apache/thrift
  version: a533ea1615e9df935e77b99440cf264c6b417e3e

from zipkin-go-opentracing.

jcchavezs avatar jcchavezs commented on June 18, 2024 1

@andream16 you can go to your $GOPATH/src/github.com/apache/thrift and there do git checkout a533ea1615e9df935e77b99440cf264c6b417e3e. Another solution is this one which is waiting to be merged.

from zipkin-go-opentracing.

cc5092 avatar cc5092 commented on June 18, 2024

Add another picture by go get github.com/openzipkin/zipkin-go-opentracing

image

from zipkin-go-opentracing.

kujtimiihoxha avatar kujtimiihoxha commented on June 18, 2024

I can confirm ^ also

from zipkin-go-opentracing.

codefromthecrypt avatar codefromthecrypt commented on June 18, 2024

I just kicked off the build again, which indeed breaks
https://travis-ci.org/openzipkin/zipkin-go-opentracing/builds/290589542

Can someone follow-up with this? for example raise a PR?

from zipkin-go-opentracing.

codefromthecrypt avatar codefromthecrypt commented on June 18, 2024

Also a bit curious if folks are actually using the scribe transport or not, or if this is just a problem building the project. For example, if you aren't conflicting because you use thrift, perhaps there's a way to not use the trunk version of thrift.

(I have no idea what dependency hygiene looks like in go world, so forgive me in advance if this isn't a relevant suggestion)

from zipkin-go-opentracing.

kujtimiihoxha avatar kujtimiihoxha commented on June 18, 2024

I am not using thrift at all, it is just a problem while building. (I am using go-kit, it uses this library)

from zipkin-go-opentracing.

kujtimiihoxha avatar kujtimiihoxha commented on June 18, 2024

@adriancole here you go #87

from zipkin-go-opentracing.

jcchavezs avatar jcchavezs commented on June 18, 2024

This PR solves the actual problem #88

@adriancole @basvanbeek

from zipkin-go-opentracing.

sitano avatar sitano commented on June 18, 2024

it's a shame it depends on some middle version in between thrift 0.11.0 and the latest, and it does not compatible to either one. apache thrift broke backward compatibility at 4f77ab8e296d64c57e6ea1c6e3f0f152bc7d6a3a.

from zipkin-go-opentracing.

andream16 avatar andream16 commented on June 18, 2024

When will it be merged? I really need this issue to get fixed. I cannot run any go-kit example.

from zipkin-go-opentracing.

andream16 avatar andream16 commented on June 18, 2024

Solved, thanks guys.

from zipkin-go-opentracing.

basvanbeek avatar basvanbeek commented on June 18, 2024

updated the generated thrift code in #90 and tests pass again

from zipkin-go-opentracing.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.