Don't use deprecated Futures#addCallback method (#78)

robert3005 · bulldozer-bot[bot] · commit dcbaffae3b8c · 2019-02-26T02:21:08.000Z
## Before this PR
tracing-java doesn't work with guava 27

## After this PR
tracing-java works with guava 27
diff --git a/tracing/src/main/java/com/palantir/tracing/AsyncSpanObserver.java b/tracing/src/main/java/com/palantir/tracing/AsyncSpanObserver.java
@@ -73,7 +73,7 @@ public void onFailure(Throwable error) {
                     log.trace("Failed to notify observer", error);
                     numInflights.decrementAndGet();
                 }
-            });
+            }, MoreExecutors.directExecutor());
         } else {
             log.trace("Failed to notify span observer since the maximum number of allowed concurrent observations was "
                     + "exceeded", SafeArg.of("maxInflights", maxInflights));

Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,7 @@ public void onFailure(Throwable error) {`
`73`	`73`	`log.trace("Failed to notify observer", error);`
`74`	`74`	`numInflights.decrementAndGet();`
`75`	`75`	`}`
`76`		`- });`
	`76`	`+ }, MoreExecutors.directExecutor());`
`77`	`77`	`} else {`
`78`	`78`	`log.trace("Failed to notify span observer since the maximum number of allowed concurrent observations was "`
`79`	`79`	`+ "exceeded", SafeArg.of("maxInflights", maxInflights));`