Skip to content

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

Merged
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
den-crane:test/analyzer
Apr 3, 2023
Merged

test / some complex query (it fails with analyzer enabled)#48324
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
den-crane:test/analyzer

Conversation

@den-crane
Copy link
Copy Markdown
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
Copy Markdown
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
@novikd
Copy link
Copy Markdown
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.

4 participants