Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

Do not use deprecated APIs internally #416

Merged

Conversation

pavolloffay
Copy link
Member

Signed-off-by: Pavol Loffay ploffay@redhat.com

Signed-off-by: Pavol Loffay <ploffay@redhat.com>
@ghost ghost assigned pavolloffay May 16, 2018
@ghost ghost added the review label May 16, 2018
@codecov
Copy link

codecov bot commented May 16, 2018

Codecov Report

Merging #416 into master will decrease coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             master     #416     +/-   ##
===========================================
- Coverage     82.24%   82.14%   -0.1%     
+ Complexity      487      486      -1     
===========================================
  Files            66       66             
  Lines          2033     2033             
  Branches        247      247             
===========================================
- Hits           1672     1670      -2     
- Misses          280      281      +1     
- Partials         81       82      +1
Impacted Files Coverage Δ Complexity Δ
...aegertracing/samplers/RemoteControlledSampler.java 77.77% <0%> (-1.02%) 17% <0%> (-1%)
...er-core/src/main/java/io/jaegertracing/Tracer.java 86.74% <0%> (-0.41%) 23% <0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6f75dfe...b7f6dda. Read the comment docs.

@pavolloffay
Copy link
Member Author

I will merge this, There is no API change or in the code. It just swaps invocation of deprecated APIs to supported.

@@ -103,7 +104,7 @@ public void shutdown() throws ExecutionException, InterruptedException {
Iterator<NetworkListener> iterator = server.getListeners().iterator();
while (iterator.hasNext()) {
NetworkListener next = iterator.next();
next.shutdown();
GrizzlyFuture<NetworkListener> shutdown = next.shutdown();
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This removes warning message from errorprone

/home/ploffay/projects/jaeger/jaeger-client-java/jaeger-crossdock/src/main/java/io/jaegertracing/crossdock/JerseyServer.java:106: warning: [FutureReturnValueIgnored] Return value of methods returning Future must be checked. Ignoring returned Futures suppresses exceptions thrown from the code that completes the Future.
      next.shutdown();
                   ^
    (see http://errorprone.info/bugpattern/FutureReturnValueIgnored)
  Did you mean to remove this line?
Note: /home/ploffay/projects/jaeger/jaeger-client-java/jaeger-crossdock/src/main/java/io/jaegertracing/crossdock/resources/behavior/EndToEndBehavior.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
1 warning

@pavolloffay pavolloffay merged commit de63e40 into jaegertracing:master May 16, 2018
@ghost ghost removed the review label May 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant