Skip to content

Fix distributed IN without database on initiator#15538

Merged
4ertus2 merged 5 commits intoClickHouse:masterfrom
4ertus2:some
Oct 6, 2020
Merged

Fix distributed IN without database on initiator#15538
4ertus2 merged 5 commits intoClickHouse:masterfrom
4ertus2:some

Conversation

@4ertus2
Copy link
Copy Markdown
Contributor

@4ertus2 4ertus2 commented Oct 2, 2020

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):
Fix 'Database doesn't exist.' in queries with IN and Distributed table when there's no database on initiator.

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Oct 2, 2020
@robot-clickhouse robot-clickhouse added pr-bugfix Pull request with bugfix, not backported by default and removed pr-not-for-changelog This PR should not be mentioned in the changelog labels Oct 2, 2020
@4ertus2 4ertus2 changed the title Minor fix in NonGlobalTableVisitor Fix distributed IN without database on initiator Oct 2, 2020
@4ertus2
Copy link
Copy Markdown
Contributor Author

4ertus2 commented Oct 5, 2020

DB::Exception: Database test doesn't exist. in python live view tests.

@4ertus2
Copy link
Copy Markdown
Contributor Author

4ertus2 commented Oct 5, 2020

An issue for fuzzer error #15623

@4ertus2
Copy link
Copy Markdown
Contributor Author

4ertus2 commented Oct 6, 2020

There's helpers.uexpect.ExpectTimeoutError: Timeout 20.000s error in integration test TestLiveViewOverDistributedSuite.
There's no IN or JOIN in test queries, so PR should not affect the test.

None of perf test errors could be caused by PR.

@4ertus2 4ertus2 merged commit bad8171 into ClickHouse:master Oct 6, 2020
alesapin pushed a commit to alexelex/ClickHouse that referenced this pull request Oct 8, 2020
@alexey-milovidov
Copy link
Copy Markdown
Member

We no longer support this case. But why did we have to support it in the first place?
There are no references.

@4ertus2 4ertus2 deleted the some branch July 11, 2025 22:02
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.

3 participants