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

AddDefaultDatabaseVisitor support dictGet #29650

Conversation

kitaisreal
Copy link
Collaborator

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Closes #29559.

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Oct 1, 2021
@Avogar Avogar self-assigned this Oct 4, 2021
@Avogar
Copy link
Member

Avogar commented Oct 6, 2021

@kitaisreal can you fix build?

@kitaisreal
Copy link
Collaborator Author

@Avogar first need to refactor context.

@kitaisreal kitaisreal force-pushed the add-default-database-visitor-support-dict-get branch from 459952b to c1f0445 Compare October 14, 2021 13:31
@kitaisreal
Copy link
Collaborator Author

@Avogar lets review it now.

@kitaisreal kitaisreal merged commit 27fdeef into ClickHouse:master Oct 15, 2021
@baltazorbest
Copy link

Could some one compile deb package with the last ClickHouse stable version and this PR please? We can't migrate to a new cluster because of this bug. Thanks.

@den-crane
Copy link
Contributor

I think it worth to backport it to 21.10, 21.9, 21.8

@kitaisreal kitaisreal added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Oct 27, 2021
kitaisreal added a commit that referenced this pull request Oct 29, 2021
Backport #29650 to 21.10: AddDefaultDatabaseVisitor support dictGet
kitaisreal added a commit that referenced this pull request Oct 29, 2021
Backport #29650 to 21.8: AddDefaultDatabaseVisitor support dictGet
kitaisreal added a commit that referenced this pull request Oct 29, 2021
Backport #29650 to 21.9: AddDefaultDatabaseVisitor support dictGet
@Felixoid Felixoid added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-must-backport Pull request should be backported intentionally. Use this label with great care! pr-not-for-changelog This PR should not be mentioned in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot restart server: <Error> Application: DB::Exception: Dictionary (...) not found
6 participants