Add PARTITIONS_COUNT setting to yql#7602
Conversation
|
⚪ ⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
|
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
3a09324 to
a2cc61d
Compare
|
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
|
⚪ ⚪ Details
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
| bool TSqlTranslation::ValidateTableSettings(const TTableSettings& settings) { | ||
| if (settings.PartitionsCount) { | ||
| if (!settings.StoreType || to_lower(settings.StoreType->Name) != "column") { | ||
| Ctx.Error() << " PARTITION_COUNT can be used only with STORE=COLUMN"; |
There was a problem hiding this comment.
Добавь, пожалуйста, тестов, проверяющий что запросы с неправильной комбинацией параметров не будут проходить
|
⚪ ⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
2 similar comments
|
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
|
⚪ ⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
Changelog entry
...
Changelog category
Additional information
...