ydb icon indicating copy to clipboard operation
ydb copied to clipboard

PR broke ydb/tests/functional/clickbench/test.py.test_run_benchmark[generic-column]

Open Tony-Romanov opened this issue 9 months ago • 2 comments

Вот этот PR ломает 4 YDB теста: https://github.com/ydb-platform/ydb/pull/4096

В том числе и вот этот большой тест со множеством кейсов: ydb/tests/functional/clickbench/test.py.test_run_benchmark[generic-column]

В двух тестах проблема была в выходном типе COUNT(*) и я это легко поправил.

Но в этом тесте проблема в типе Timestamp который в CS хранится как arrow::timestamp а YQL ожидает arrow::uint64.

Во-первых, это неправильно сознательно комитить то, что ломает работающие тесты.

Во-вторых, решить это можно либо runtime преобразованием колонки, чего мы в CS очень хотели избежать, либо просто убрав эту debug-only проверку, без которой всё работет. То есть по факту откатить это PR.

Tony-Romanov avatar May 07 '24 11:05 Tony-Romanov

@vitstn @iddqdex

Почему замерджен PR с явно сломанными тестами?

spuchin avatar May 07 '24 12:05 spuchin

@gridnevvvit @ivanmorozov333 @fomichev3000 @vitstn @nsofya

Tony-Romanov avatar May 07 '24 12:05 Tony-Romanov