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

test / some complex query (it fails with analyzer enabled) #48324

Merged
merged 1 commit into from
Apr 3, 2023

Conversation

den-crane
Copy link
Contributor

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-not-for-changelog This PR should not be mentioned in the changelog label Apr 2, 2023
@den-crane den-crane changed the title some complex query (it fails with analyzer enabled) test / some complex query (it fails with analyzer enabled) Apr 2, 2023
Copy link
Member

@alexey-milovidov alexey-milovidov left a comment

Choose a reason for hiding this comment

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

LGTM. This test will help us when we are going to enable Analyzer.

@alexey-milovidov alexey-milovidov self-assigned this Apr 3, 2023
@alexey-milovidov alexey-milovidov merged commit 9ad744f into ClickHouse:master Apr 3, 2023
2 checks passed
@novikd
Copy link
Member

novikd commented Apr 24, 2023

I'm not sure if it makes sense to support such queries. The exception says:

using identifier 'account_id' cannot be resolved from left table expression.

The JOIN expression is: srv_account_parts LEFT JOIN (SELECT ...). The table actually doesn't contain account_id column, it has account_ids Array(Int64) which can be used by an alias arrayJoin(account_ids) AS account_id defined in the SELECT.

This is a very strange behavior that it worked in the old infrastructure. In my opinion, we should not support it. @kitaisreal what do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

None yet

4 participants