@VisibleForTesting
static final long NO_PROGRESS_TIMEOUT_NANOS = TimeUnit.MINUTES.toNanos(15);
+ // Emit a debug entry if we sleep for more that this amount
+ private static final long DEBUG_DELAY_NANOS = TimeUnit.MILLISECONDS.toNanos(100);
+
private final Lock lock = new ReentrantLock();
private final ClientActorContext context;
@GuardedBy("lock")
final long now = currentTime();
final long delay = enqueueEntry(new ConnectionEntry(request, callback, now), now);
try {
+ if (delay >= DEBUG_DELAY_NANOS && LOG.isDebugEnabled()) {
+ LOG.debug("Sleeping for {}ms", TimeUnit.NANOSECONDS.toMillis(delay));
+ }
TimeUnit.NANOSECONDS.sleep(delay);
} catch (InterruptedException e) {
Thread.currentThread().interrupt();
sendRequest(abortRequest(), resp -> {
LOG.debug("Transaction {} abort completed with {}", getIdentifier(), resp);
- sendPurge();
+ enqueuePurge();
});
}
// This is a terminal request, hence we do not need to record it
LOG.debug("Transaction {} abort completed", this);
- sendPurge();
+ enqueuePurge();
});
}
// This is a terminal request, hence we do not need to record it
LOG.debug("Transaction {} directCommit completed", this);
- sendPurge();
+ enqueuePurge();
});
return ret;
}
LOG.debug("Transaction {} doCommit completed", this);
- sendPurge();
+ enqueuePurge();
});
}
- private void sendPurge() {
- sendPurge(null);
+ private void enqueuePurge() {
+ enqueuePurge(null);
}
- final void sendPurge(final Consumer<Response<?, ?>> callback) {
- sendRequest(purgeRequest(), resp -> completePurge(resp, callback));
+ final void enqueuePurge(final Consumer<Response<?, ?>> callback) {
+ // Purge request are dispatched internally, hence should not wait
+ enqueuePurge(callback, parent.currentTime());
}
final void enqueuePurge(final Consumer<Response<?, ?>> callback, final long enqueuedTicks) {
- enqueueRequest(purgeRequest(), resp -> completePurge(resp, callback), enqueuedTicks);
+ enqueueRequest(purgeRequest(), resp -> {
+ LOG.debug("Transaction {} purge completed", this);
+ parent.completeTransaction(this);
+ if (callback != null) {
+ callback.accept(resp);
+ }
+ }, enqueuedTicks);
}
private TransactionPurgeRequest purgeRequest() {
return new TransactionPurgeRequest(getIdentifier(), nextSequence(), localActor());
}
- private void completePurge(final Response<?, ?> resp, final Consumer<Response<?, ?>> callback) {
- LOG.debug("Transaction {} purge completed", this);
- parent.completeTransaction(this);
- if (callback != null) {
- callback.accept(resp);
- }
- }
-
// Called with the connection unlocked
final synchronized void startReconnect() {
// At this point canCommit/directCommit are blocked, we assert a new successor state, retrieving the previous
} else if (handleReadRequest(request, callback)) {
// No-op
} else if (request instanceof TransactionPurgeRequest) {
- sendPurge(callback);
+ enqueuePurge(callback);
} else {
throw new IllegalArgumentException("Unhandled request " + request);
}
successor.abort();
} else if (request instanceof TransactionPurgeRequest) {
LOG.debug("Forwarding purge {} to successor {}", request, successor);
- successor.sendPurge(callback);
+ successor.enqueuePurge(callback);
} else {
throw new IllegalArgumentException("Unhandled request" + request);
}
if (request instanceof AbortLocalTransactionRequest) {
successor.sendAbort(request, callback);
} else if (request instanceof TransactionPurgeRequest) {
- successor.sendPurge(callback);
+ successor.enqueuePurge(callback);
} else {
throw new IllegalArgumentException("Unhandled request" + request);
}
if (identifier.equals(req.getTarget())) {
Verify.verify(req instanceof LocalHistoryRequest);
if (req instanceof CreateLocalHistoryRequest) {
- successor.connection.sendRequest(req, e.getCallback());
+ successor.connection.enqueueRequest(req, e.getCallback(), e.getEnqueuedTicks());
it.remove();
break;
}
if (identifier.equals(req.getTarget())) {
Verify.verify(req instanceof LocalHistoryRequest);
if (req instanceof DestroyLocalHistoryRequest) {
- successor.connection.sendRequest(req, e.getCallback());
+ successor.connection.enqueueRequest(req, e.getCallback(), e.getEnqueuedTicks());
it.remove();
break;
}