Adding subscribe to shard request id logging to ShardConsumerSubscriber. (#705)
This commit is contained in:
parent
763d506e20
commit
7c7491c30d
1 changed files with 4 additions and 3 deletions
|
|
@ -192,8 +192,8 @@ class ShardConsumerSubscriber implements Subscriber<RecordsRetrieved> {
|
||||||
protected void logOnErrorWarning(Throwable t) {
|
protected void logOnErrorWarning(Throwable t) {
|
||||||
log.warn(
|
log.warn(
|
||||||
"{}: onError(). Cancelling subscription, and marking self as failed. KCL will "
|
"{}: onError(). Cancelling subscription, and marking self as failed. KCL will "
|
||||||
+ "recreate the subscription as neccessary to continue processing.",
|
+ "recreate the subscription as neccessary to continue processing. Last successful request details -- {}",
|
||||||
shardConsumer.shardInfo().shardId(), t);
|
shardConsumer.shardInfo().shardId(), recordsPublisher.getLastSuccessfulRequestDetails(), t);
|
||||||
}
|
}
|
||||||
|
|
||||||
protected void logOnErrorReadTimeoutWarning(Throwable t) {
|
protected void logOnErrorReadTimeoutWarning(Throwable t) {
|
||||||
|
|
@ -202,7 +202,8 @@ class ShardConsumerSubscriber implements Subscriber<RecordsRetrieved> {
|
||||||
+ "are seeing this warning frequently consider increasing the SDK timeouts "
|
+ "are seeing this warning frequently consider increasing the SDK timeouts "
|
||||||
+ "by providing an OverrideConfiguration to the kinesis client. Alternatively you"
|
+ "by providing an OverrideConfiguration to the kinesis client. Alternatively you"
|
||||||
+ "can configure LifecycleConfig.readTimeoutsToIgnoreBeforeWarning to suppress"
|
+ "can configure LifecycleConfig.readTimeoutsToIgnoreBeforeWarning to suppress"
|
||||||
+ "intermittant ReadTimeout warnings.", shardConsumer.shardInfo().shardId(), t);
|
+ "intermittant ReadTimeout warnings. Last successful request details -- {}",
|
||||||
|
shardConsumer.shardInfo().shardId(), recordsPublisher.getLastSuccessfulRequestDetails(), t);
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue