Fixed backward compatibility in fq-proxy result_writer_actor #5111
Conversation
|
⚪
|
|
⚪
|
| @@ -96,6 +96,13 @@ class TResultSetWriterActor : public TBaseComputeActor<TResultSetWriterActor> { | |||
|
|
|||
| void Handle(const TEvYdbCompute::TEvFetchScriptResultResponse::TPtr& ev) { | |||
| const auto& response = *ev.Get()->Get(); | |||
| if (response.Status == NYdb::EStatus::BAD_REQUEST && FetchRowsLimit == 0) { | |||
There was a problem hiding this comment.
А точно bad requst только в этом случае вылетает? Не можем мы тут откатиться по другой ошибке?
There was a problem hiding this comment.
Можем и кажется это даже может быть хорошо, например если в результатах какие-то проблемы (например слишком длинные строчки, более 20MB) и новый вычитыватель не справился:
ydb/ydb/core/kqp/proxy_service/kqp_script_executions.cpp
Lines 2170 to 2179 in 1b4620e
There was a problem hiding this comment.
Но такого же быть не должно. Это баг получается в этом случае. На сенсорах будем видеть такие затупы, а то можем не заметить деградаций?
There was a problem hiding this comment.
Сенсоров на такие затупы нету, но эта проблема должна пофикситься когда GetResult будет переписан на стримовый запрос
Changelog entry
Fixed backward compatibility in fq-proxy result_writer_actor
Changelog category
Additional information