upgrade Guava dependency to something recent about servo HOT 6 CLOSED

ah, I see the project is now deprecated and no longer supported. never mind, closing this issue then

from servo.

Ideally servo will work fine if newer versions are pulled in. We haven't updated the dependency because it causes issues for some other uses which still require 19.

What specific errors did you see when running on a newer version?

from servo.

while older projects still use Guava 19, it may be hard to justify supporting it forever - those people would need to stay with older versions of related libraries.

Exception in thread "main" java.lang.NoSuchMethodError: com.google.common.base.Objects.toStringHelper(Ljava/lang/Object;)Lcom/google/common/base/Objects$ToStringHelper;
at com.netflix.servo.monitor.MonitorConfig.toString(MonitorConfig.java:161)
...
at com.netflix.loadbalancer.LoadBalancerBuilder.buildFixedServerListLoadBalancer(LoadBalancerBuilder.java:56)

from servo.

from servo.

while older projects still use Guava 19, it may be hard to justify supporting it forever

As @dmuino indicated, we significantly reduced our usage of Guava several years back to minimize the number of issues we hit with compatibility changes. It would have been removed all together, but it is unfortunately used in the public API in some places.

I tried updating the guava dependency locally to the latest version, 27.1-jre, and it compiles fine and the tests pass. So servo shouldn't be a blocker to adopting newer versions of guava assuming you are using a recent version of servo.

from servo.

got it, thank you guys!

from servo.