Small refactoring of sendErrorResponse
This commit is contained in:
parent
bf46d974ab
commit
a3a5fd3180
@ -131,7 +131,7 @@ public class PartitionedQueueResponseTemplate<Request extends TbQueueMsg, Respon
|
|||||||
@Override
|
@Override
|
||||||
public void onFailure(Throwable t) {
|
public void onFailure(Throwable t) {
|
||||||
log.error("[{}] Failed to send response {}", requestId, response, t);
|
log.error("[{}] Failed to send response {}", requestId, response, t);
|
||||||
sendErrorResponse(requestId, responseTopic, request, t);
|
sendErrorResponse(requestId, tpi, request, t);
|
||||||
stats.incrementFailed();
|
stats.incrementFailed();
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
@ -158,12 +158,11 @@ public class PartitionedQueueResponseTemplate<Request extends TbQueueMsg, Respon
|
|||||||
consumer.commit();
|
consumer.commit();
|
||||||
}
|
}
|
||||||
|
|
||||||
private void sendErrorResponse(UUID requestId, String responseTopic, Request request, Throwable cause) {
|
private void sendErrorResponse(UUID requestId, TopicPartitionInfo tpi, Request request, Throwable cause) {
|
||||||
Response errorResponseMsg = handler.constructErrorResponseMsg(request, cause);
|
Response errorResponseMsg = handler.constructErrorResponseMsg(request, cause);
|
||||||
|
|
||||||
if (errorResponseMsg != null) {
|
if (errorResponseMsg != null) {
|
||||||
errorResponseMsg.getHeaders().put(REQUEST_ID_HEADER, uuidToBytes(requestId));
|
errorResponseMsg.getHeaders().put(REQUEST_ID_HEADER, uuidToBytes(requestId));
|
||||||
TopicPartitionInfo tpi = TopicPartitionInfo.builder().topic(responseTopic).build();
|
|
||||||
responseProducer.send(tpi, errorResponseMsg, null);
|
responseProducer.send(tpi, errorResponseMsg, null);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user