-
Notifications
You must be signed in to change notification settings - Fork 646
Integrate stable-24-3-9-cs-2 into stable-24-3 #9675
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Integrate stable-24-3-9-cs-2 into stable-24-3 #9675
Conversation
1aae6f4
to
45a2b8f
Compare
⚪
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
⚪ |
⚪ |
Co-authored-by: Semyon <[email protected]> Co-authored-by: ivanmorozov333 <[email protected]> Co-authored-by: Ivan <[email protected]> Co-authored-by: Artem Alekseev <[email protected]> Co-authored-by: Alexander Avdonkin <[email protected]> Co-authored-by: Nikita Vasilev <[email protected]> Co-authored-by: Vladislav Gogov <[email protected]>
Co-authored-by: Nikita Vasilev <[email protected]>
Co-authored-by: ivanmorozov333 <[email protected]>
Co-authored-by: Pisarenko Grigoriy <[email protected]> Co-authored-by: ivanmorozov333 <[email protected]>
45a2b8f
to
a5915f7
Compare
⚪
*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 |
.github/CODEOWNERS
Outdated
@@ -1 +1 @@ | |||
* @ydb-platform/ReleaseApprovers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please, revert this change
fixes #9670 |
@@ -285,7 +285,7 @@ message TTableServiceConfig { | |||
optional bool EnableOlapSink = 55 [default = false]; | |||
|
|||
optional bool EnablePerStatementQueryExecution = 56 [default = false]; | |||
optional bool EnableCreateTableAs = 57 [default = true]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nikvas0, check this change, please
⚪
🔴
*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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kqp looks ok
Co-authored-by: zverevgeny <[email protected]> Co-authored-by: Semyon <[email protected]> Co-authored-by: ivanmorozov333 <[email protected]> Co-authored-by: Ivan <[email protected]> Co-authored-by: Alexander Avdonkin <[email protected]> Co-authored-by: Nikita Vasilev <[email protected]> Co-authored-by: Vladislav Gogov <[email protected]> Co-authored-by: Pisarenko Grigoriy <[email protected]>
Changelog entry
Integrate all columnshard changes from stable-24-3-9-cs2 to stable-24-3
Basically, just rebasing stable-24-3.9-cs2(602dfb0) to stable-24.3
fixes #9670
Changelog category
Additional information
...