diff options
| author | nsofya <nsofya@yandex-team.com> | 2023-03-15 12:55:05 +0300 |
|---|---|---|
| committer | nsofya <nsofya@yandex-team.com> | 2023-03-15 12:55:05 +0300 |
| commit | 720ab09aac0cd54c0b888ecdb0c53899f11daaee (patch) | |
| tree | c903a6746ebfa90f6515da8681c5d7f3c79bdcf7 /tools/black_linter | |
| parent | 7a343b35aea0c6cda407a7f0f11a2a494c8125b8 (diff) | |
| download | ydb-720ab09aac0cd54c0b888ecdb0c53899f11daaee.tar.gz | |
Show errors for not implemented alter requests
Что сделала:
1) Тест на обновление схемы колонок, который проверяет, что вернем ошибку
2) Перегруппировала текущий код, чтобы вся валидация входящего запроса была в одном классе
3) Максимально дополнила конвертер из TTableDescription в TAlterColumnTable
4) Сделала поля для Alter и Drop одного типа, аналогично даташарду.
Упал тест: https://a.yandex-team.ru/arcadia/ydb/core/tx/schemeshard/ut_olap.cpp?rev=b100a39763180844768c40940c1e2863fd140ee3#L568
Но насколько я видела из кода - это синтетический тест, по факту эти операции нельзя было произвести, верно? Ну и все равно, кажется, что переименование тут не критично, но поправьте, если я чего-то не поняла.
Diffstat (limited to 'tools/black_linter')
0 files changed, 0 insertions, 0 deletions