Skip to content

Fix possible "No column to rollback" logical error during INSERT into Buffer#43336

Merged
davenger merged 3 commits intoClickHouse:masterfrom
azat:buffer/fix-rollback
Nov 23, 2022
Merged

Fix possible "No column to rollback" logical error during INSERT into Buffer#43336
davenger merged 3 commits intoClickHouse:masterfrom
azat:buffer/fix-rollback

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Nov 17, 2022

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in official stable or prestable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Fix possible (very unlikely) "No column to rollback" logical error during INSERT into Buffer

In case of concurrent INSERT and SELECT, and memory pressure it is
possible to get this error, if you are very lucky.

More details you will find in the comments in the code and tests

Fixes: #42740 (cc @tavplubix @alexey-milovidov )

azat added 2 commits November 17, 2022 18:10
…hmark

Using clickhouse-benchmark is better since it can send way more queries,
especially if you have debug/sanitizers build.

Signed-off-by: Azat Khuzhin <[email protected]>
@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Nov 17, 2022
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Submitted it separately - #43337

… Buffer

In case of concurrent INSERT and SELECT, and memory pressure it is
possible to get this error, if you are very lucky.

Signed-off-by: Azat Khuzhin <[email protected]>
@azat azat force-pushed the buffer/fix-rollback branch from a9773f9 to aa77bea Compare November 17, 2022 18:13
@davenger davenger self-assigned this Nov 21, 2022
@davenger
Copy link
Copy Markdown
Member

AST Fuzzer (asan) failure: #43202

@davenger davenger merged commit 1dec791 into ClickHouse:master Nov 23, 2022
@azat azat deleted the buffer/fix-rollback branch November 23, 2022 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Logical error: No column to rollback

3 participants