Skip to content

Commit eb92a69

Browse files
author
Stephane Landelle
committed
Rename ReaperFuture into FutureReaper
1 parent 44fb968 commit eb92a69

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

providers/netty4/src/main/java/org/asynchttpclient/providers/netty4/ReaperFuture.java renamed to providers/netty4/src/main/java/org/asynchttpclient/providers/netty4/FutureReaper.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,15 +14,15 @@
1414
* Because some implementation of the ThreadSchedulingService do not clean up cancel task until they try to run them, we wrap the task with the future so the when the NettyResponseFuture cancel the reaper future this wrapper will release the references to the channel and the
1515
* nettyResponseFuture immediately. Otherwise, the memory referenced this way will only be released after the request timeout period which can be arbitrary long.
1616
*/
17-
public final class ReaperFuture implements Runnable {
17+
public final class FutureReaper implements Runnable {
1818

1919
private final AtomicBoolean isClose;
2020
private final Channels channels;
2121
private Future<?> scheduledFuture;
2222
private NettyResponseFuture<?> nettyResponseFuture;
2323
private AsyncHttpClientConfig config;
2424

25-
public ReaperFuture(NettyResponseFuture<?> nettyResponseFuture, AsyncHttpClientConfig config, AtomicBoolean isClose, Channels channels) {
25+
public FutureReaper(NettyResponseFuture<?> nettyResponseFuture, AsyncHttpClientConfig config, AtomicBoolean isClose, Channels channels) {
2626
this.nettyResponseFuture = nettyResponseFuture;
2727
this.channels = channels;
2828
this.config = config;

providers/netty4/src/main/java/org/asynchttpclient/providers/netty4/NettyRequestSender.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -403,7 +403,7 @@ private void scheduleReaper(NettyResponseFuture<?> future) {
403403
: requestTimeout) : config.getIdleConnectionTimeoutInMs();
404404

405405
if (schedulePeriod != -1 && !future.isDone() && !future.isCancelled()) {
406-
ReaperFuture reaperFuture = new ReaperFuture(future, config, isClose, channels);
406+
FutureReaper reaperFuture = new FutureReaper(future, config, isClose, channels);
407407
Future<?> scheduledFuture = config.reaper().scheduleAtFixedRate(reaperFuture, 0, schedulePeriod, TimeUnit.MILLISECONDS);
408408
reaperFuture.setScheduledFuture(scheduledFuture);
409409
future.setReaperFuture(reaperFuture);

providers/netty4/src/main/java/org/asynchttpclient/providers/netty4/NettyResponseFuture.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ public enum STATE {
6969
private HttpResponse httpResponse;
7070
private final AtomicReference<ExecutionException> exEx = new AtomicReference<ExecutionException>();
7171
private final AtomicInteger redirectCount = new AtomicInteger();
72-
private volatile ReaperFuture reaperFuture;
72+
private volatile FutureReaper reaperFuture;
7373
private final AtomicBoolean inAuth = new AtomicBoolean(false);
7474
private final AtomicBoolean statusReceived = new AtomicBoolean(false);
7575
private final AtomicLong touch = new AtomicLong(millisTime());
@@ -358,7 +358,7 @@ public int incrementAndGetCurrentRedirectCount() {
358358
return redirectCount.incrementAndGet();
359359
}
360360

361-
public void setReaperFuture(ReaperFuture reaperFuture) {
361+
public void setReaperFuture(FutureReaper reaperFuture) {
362362
cancelReaper();
363363
this.reaperFuture = reaperFuture;
364364
}

0 commit comments

Comments
 (0)