Skip to content
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

Parts metadata cache and failing mutation #39197

Closed
tavplubix opened this issue Jul 13, 2022 · 2 comments · Fixed by #51303
Closed

Parts metadata cache and failing mutation #39197

tavplubix opened this issue Jul 13, 2022 · 2 comments · Fixed by #51303
Assignees
Labels
fuzz Problem found by one of the fuzzers

Comments

@tavplubix
Copy link
Member

https://s3.amazonaws.com/clickhouse-test-reports/0/2079699f7542527d62cbaed828940d54b0ca00a2/stress_test__undefined__actions_.html

BC check in Stress Tests sometimes fail with

2022.07.13 10:26:18.097238 [ 1051250 ] {} <Error> MutateFromLogEntryTask: virtual bool DB::ReplicatedMergeMutateTaskBase::executeStep(): Code: 47. DB::Exception: Missing columns: 'v3' while processing query: 'v3, k, v1, v2, p', required columns: 'v3' 'k' 'v1' 'v2' 'p', maybe you meant: ['v1','k','v1','v2','p']. (UNKNOWN_IDENTIFIER), Stack trace (when copying this message, always include the lines below):
2022.07.13 10:26:18.098966 [ 1051240 ] {} <Error> MutateFromLogEntryTask: virtual bool DB::ReplicatedMergeMutateTaskBase::executeStep(): Code: 47. DB::Exception: Missing columns: 'v3' while processing query: 'v3, k, v1, v2, p', required columns: 'v3' 'k' 'v1' 'v2' 'p', maybe you meant: ['v1','k','v1','v2','p']. (UNKNOWN_IDENTIFIER), Stack trace (when copying this message, always include the lines below):
2022.07.13 10:26:18.101742 [ 1051251 ] {} <Error> MutateFromLogEntryTask: virtual bool DB::ReplicatedMergeMutateTaskBase::executeStep(): Code: 47. DB::Exception: Missing columns: 'v3' while processing query: 'v3, k, v1, v2, p', required columns: 'v3' 'k' 'v1' 'v2' 'p', maybe you meant: ['v1','k','v1','v2','p']. (UNKNOWN_IDENTIFIER), Stack trace (when copying this message, always include the lines below):

i.e. some mutation cannot be executed (slightly related to #39174). This mutation was started by the test 01233_check_table_with_metadata_cache. Logs:
by table id: https://pastila.nl/?024eb85d/65a0e50dba5aafb52d0ecec2cf2d5737
by query ids: https://pastila.nl/?002a7f2e/7505e0215864bbb1ea3da78c4a0151db

The second mutation (MODIFY TTL + implicit MATERIALIZE TTL) is trying to read from column that was dropped by the first mutation (DROP COLUMN v3). The error persists after server restart, so I suppose metadata was corrupted. Probably not metadata itself, but metadata cache.

cc: @taiyang-li, @CurtizJ

@tavplubix tavplubix added the fuzz Problem found by one of the fuzzers label Jul 13, 2022
@taiyang-li
Copy link
Contributor

@tavplubix Thanks for your issue. I'll try to fix it soon

@alexey-milovidov
Copy link
Member

We decided to remove the metadata cache instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fuzz Problem found by one of the fuzzers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants