Closed turbolent closed 6 years ago
Hey @turbolent – thanks for reporting this. I haven't had a chance to look properly yet, but this does sound like a problem with Finagle versions and compatibility. I'll try and take a look this evening, and try and get it working with the latest version of Finagle.
Great, thank you! I'll also have a look tonight, but I don't have much experience with Finagle's metrics
Hey @turbolent, sorry for the delay, it's been a busy week. I just raised PR #17 to bump the Finagle version to 18.2.0. The tests passed locally for me, and I'll keep an eye on the Travis CI builds.
Would you be able to provide me some example code that reproduces the issue? A snippet would work, but you might get a faster reply if it's either a project I can check out and run. You might also be able to fork this repo and add a failing test. From the stack trace, it looks like you're doing something Thrift-related – is that correct?
Great! No worries, same here.
Trying to produce a minimal working example, it seems like just having finagle-prometheus in the classpath causes the crash. When investigating further, I also found https://github.com/finagle/finagle-postgres/issues/63. It seems like Finagle's API changed a while ago, and having different versions of it and its dependencies causes the crash. The upgrade of the library to 18.2.0 should probably fix the issue for users who use a recent version of Finagle.
@turbolent That would make sense – I just released v0.0.5 on bintray, and it's just in the process of syncing to Maven Central. Let me know if it helps!
Great, that seems to do the trick -- Thanks for publishing an updated release and the great library!
Great news :+1:
I tried integrating this library like shown in the example, but get the following error when my server starts up:
Could the issue be that I'm using Finagle 17.12.0?