Skip to content

Commit 48e5ccf

Browse files
Merge 0e535b2 into 951ad3b
2 parents 951ad3b + 0e535b2 commit 48e5ccf

File tree

1 file changed

+17
-7
lines changed

1 file changed

+17
-7
lines changed

ydb/core/kqp/executer_actor/kqp_executer_impl.h

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -589,14 +589,24 @@ class TKqpExecuterBase : public TActorBootstrapped<TDerived> {
589589
auto reason = ev->Get()->Reason;
590590
switch (eventType) {
591591
case TEvKqpNode::TEvStartKqpTasksRequest::EventType: {
592-
if (reason == TEvents::TEvUndelivered::EReason::ReasonActorUnknown) {
593-
LOG_D("Schedule a retry by ActorUnknown reason, nodeId:" << ev->Sender.NodeId() << " requestId: " << ev->Cookie);
594-
this->Schedule(TDuration::MilliSeconds(Planner->GetCurrentRetryDelay(ev->Cookie)), new typename TEvPrivate::TEvRetry(ev->Cookie, ev->Sender));
595-
return;
592+
switch (reason) {
593+
case TEvents::TEvUndelivered::EReason::ReasonActorUnknown: {
594+
LOG_D("Schedule a retry by ActorUnknown reason, nodeId:" << ev->Sender.NodeId() << " requestId: " << ev->Cookie);
595+
this->Schedule(TDuration::MilliSeconds(Planner->GetCurrentRetryDelay(ev->Cookie)), new typename TEvPrivate::TEvRetry(ev->Cookie, ev->Sender));
596+
return;
597+
}
598+
case TEvents::TEvUndelivered::EReason::Disconnected: {
599+
InvalidateNode(ev->Sender.NodeId());
600+
ReplyUnavailable(TStringBuilder()
601+
<< "Failed to send EvStartKqpTasksRequest because node is unavailable: " << ev->Sender.NodeId());
602+
return;
603+
}
604+
case TEvents::TEvUndelivered::EReason::ReasonUnknown: {
605+
InvalidateNode(ev->Sender.NodeId());
606+
InternalError(TStringBuilder() << "TEvKqpNode::TEvStartKqpTasksRequest lost: " << reason);
607+
return;
608+
}
596609
}
597-
InvalidateNode(ev->Sender.NodeId());
598-
return InternalError(TStringBuilder()
599-
<< "TEvKqpNode::TEvStartKqpTasksRequest lost: " << reason);
600610
}
601611
default: {
602612
LOG_E("Event lost, type: " << eventType << ", reason: " << reason);

0 commit comments

Comments
 (0)