added columnshard tests for bool #26561
Conversation
|
🟢 |
.github/config/muted_ya.txt
Outdated
| ydb/core/kqp/ut/olap KqpOlapJson.BrokenJsonWriting[2,true,1,10,1000000,0] | ||
| ydb/core/kqp/ut/olap KqpOlapOptimizer.OptimizationAfterDeletion | ||
| ydb/core/kqp/ut/olap KqpOlapSysView.StatsSysViewColumns | ||
| ydb/core/kqp/ut/olap KqpBoolColumnShard.TestSimpleQueries+UseScanQuery |
There was a problem hiding this comment.
Предлагаю их не в mute добавлять а через return отключить чтобы в ci чеках не тратить время пока функциональность не готова
|
⚪
🟢 |
ydb/core/kqp/ut/olap/bool_ut.cpp
Outdated
| check(tester); | ||
| } | ||
|
|
||
| Y_UNIT_TEST_DUO(TestFilterNulls, UseScanQuery) { |
There was a problem hiding this comment.
Давай строчные и колоночные вместе протестируем здесь
There was a problem hiding this comment.
Мне кажется у даташардов есть свои тесты для этого
There was a problem hiding this comment.
Можем тогда объединиться с их тестами? Точно ли нам нужны одинаковые тесты с двух сторон?
There was a problem hiding this comment.
Я не уверен что они одинаковые
There was a problem hiding this comment.
Просто у них есть какие-то свои тесты для bool
| }; | ||
|
|
||
| Y_UNIT_TEST_SUITE(KqpBoolColumnShard) { | ||
| class TBoolTestCase { |
There was a problem hiding this comment.
Обычно TBoolTestCase выносят вне suite
There was a problem hiding this comment.
Это не так. Я делал по аналогии с decimal
| TestHelper.ExecuteQuery(query); | ||
| } | ||
|
|
||
| void PrepareTable1() { |
There was a problem hiding this comment.
Не хватает разных форматов, нужен CSV, YdbValue, Arrow
There was a problem hiding this comment.
Это другие тесты, в других местах, в этом пр я только добавляю bool_ut в ydb/core/kqp/ut/olap
|
⚪
🟢 |
|
⚪
🟢 |
|
Договорились что здесь останется технический долг и тесты нужно будет переписать по аналогии с https://a.yandex-team.ru/arcadia/contrib/ydb/services/ydb/backup_ut/ydb_backup_ut.cpp?rev=r17704661#L142 |
|
⚪ ⚪ Ya make output | Test bloat | Test bloat
🟢 |
|
⚪
🟢 |
Changelog entry
...
Changelog category
Description for reviewers
...