Skip to content

Cherry pick #19449 to 21.1: Mark distributed batch as broken in case of empty data block in one of files#19502

Merged
vitlibar merged 4 commits intobackport/21.1/19449from
cherrypick/21.1/7f059e376559784097e8e308e902dc53be0fec7e
Feb 1, 2021
Merged

Cherry pick #19449 to 21.1: Mark distributed batch as broken in case of empty data block in one of files#19502
vitlibar merged 4 commits intobackport/21.1/19449from
cherrypick/21.1/7f059e376559784097e8e308e902dc53be0fec7e

Conversation

@robot-clickhouse
Copy link
Copy Markdown
Member

Original pull-request #19449

This pull-request is a first step of an automated backporting.
It contains changes like after calling a local command git cherry-pick.
If you intend to continue backporting this changes, then resolve all conflicts if any.
Otherwise, if you do not want to backport them, then just close this pull-request.

The check results does not matter at this step - you can safely ignore them.
Also this pull-request will be merged automatically as it reaches the mergeable state, but you always can merge it manually.

azat and others added 2 commits January 22, 2021 21:29
…tributed

It is possible to get corruption (even though it is very unlikely, and
initially it wasn't corruption) just before the data block goes in the
file on disk, and in case of batching, it will break the packets, since
it will write the packet type but will not write any data after.
Mark distributed batch as broken in case of empty data block in one of files
@robot-clickhouse robot-clickhouse added do not test disable testing on pull request pr-cherrypick Cherry-pick of merge-commit before backporting. Do not use manually - automated use only! labels Jan 23, 2021
@vitlibar vitlibar merged commit ecadfef into backport/21.1/19449 Feb 1, 2021
@vitlibar vitlibar deleted the cherrypick/21.1/7f059e376559784097e8e308e902dc53be0fec7e branch February 1, 2021 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not test disable testing on pull request pr-cherrypick Cherry-pick of merge-commit before backporting. Do not use manually - automated use only!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants