Skip to content

Fix rename Distributed table#25667

Merged
tavplubix merged 2 commits intomasterfrom
fix_rename_distributed2
Jun 24, 2021
Merged

Fix rename Distributed table#25667
tavplubix merged 2 commits intomasterfrom
fix_rename_distributed2

Conversation

@tavplubix
Copy link
Copy Markdown
Member

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fixed No such file or directory error on moving Distributed table between databases. Fixes #24971

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Jun 24, 2021

INSERT INTO test_01155_atomic.src VALUES ('after moving tables');
SELECT 2, substr(_table, 1, 10), s FROM merge('test_01155_atomic', '') ORDER BY _table, s; -- { serverError 81 }
SELECT materialize(2), substr(_table, 1, 10), s FROM merge('test_01155_atomic', '') ORDER BY _table, s; -- { serverError 81 }
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

How materialization is connected to the issue of this PR?

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.

Test has found another bug (Cannot convert column 1 because it is non constant in source stream but must be constant in result), materialize() is a workaround

@nikitamikhaylov nikitamikhaylov self-assigned this Jun 24, 2021
@tavplubix tavplubix merged commit a14724c into master Jun 24, 2021
@tavplubix tavplubix deleted the fix_rename_distributed2 branch June 24, 2021 21:05
tavplubix added a commit that referenced this pull request Jun 25, 2021
Backport #25667 to 21.5: Fix rename Distributed table
tavplubix added a commit that referenced this pull request Jun 25, 2021
Backport #25667 to 21.6: Fix rename Distributed table
tavplubix added a commit that referenced this pull request Jun 28, 2021
Backport #25667 to 20.8: Fix rename Distributed table
tavplubix added a commit that referenced this pull request Jun 28, 2021
Backport #25667 to 21.3: Fix rename Distributed table
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.

Unable to move Distributed table from Ordinary database to Atomic

3 participants