Skip to content

Storage read query plan (Continuation)#14936

Merged
KochetovNicolai merged 41 commits intomasterfrom
storage-read-query-plan
Nov 10, 2020
Merged

Storage read query plan (Continuation)#14936
KochetovNicolai merged 41 commits intomasterfrom
storage-read-query-plan

Conversation

@KochetovNicolai
Copy link
Copy Markdown
Member

@KochetovNicolai KochetovNicolai commented Sep 17, 2020 •
edited
Loading

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

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Add VIEW subquery description to EXPLAIN. Limit push down optimisation for VIEW. Add local replicas of Distributed to query plan.

@robot-clickhouse robot-clickhouse added the pr-improvement Pull request with some product improvements label Sep 17, 2020
@KochetovNicolai KochetovNicolai added the do not test disable testing on pull request label Sep 18, 2020
@KochetovNicolai KochetovNicolai removed the do not test disable testing on pull request label Sep 25, 2020
@KochetovNicolai KochetovNicolai marked this pull request as ready for review September 28, 2020 17:27
@KochetovNicolai
Copy link
Copy Markdown
Member Author

KochetovNicolai commented Sep 28, 2020 

EXPLAIN
SELECT sum(number)
FROM remote('127.0.0.{1,2}', numbers(10))
FORMAT TSV

Expression (Projection)
  Expression (Before ORDER BY and SELECT)
    MergingAggregated
      SettingQuotaAndLimits (Set limits and quota after reading from storage)
        Union
          Converting (Convert block structure for query from local replica)
            Aggregating
              Expression (Before GROUP BY)
                SettingQuotaAndLimits (Set limits and quota after reading from storage)
                  ReadFromStorage (SystemNumbers)
          ReadFromPreparedSource (Read from remote replica)

@nikitamikhaylov nikitamikhaylov self-assigned this Oct 6, 2020
@KochetovNicolai
Copy link
Copy Markdown
Member Author

KochetovNicolai commented Oct 22, 2020

-34.994x select number from numbers_view limit 100

@KochetovNicolai KochetovNicolai merged commit 15d3fc1 into master Nov 10, 2020
@KochetovNicolai KochetovNicolai deleted the storage-read-query-plan branch November 10, 2020 16:57
azat added a commit to azat/ClickHouse that referenced this pull request Dec 6, 2020
@azat
Fix max_distributed_connections (one more time)
c3a4e5e 
With prefer_localhost_replica=1 max_distributed_connections uses
max_threads before this patch.

Fixes: ClickHouse#14936
Previous fix: ClickHouse#9673
@azat azat mentioned this pull request Dec 6, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nikitamikhaylov nikitamikhaylov nikitamikhaylov approved these changes

Assignees

@nikitamikhaylov nikitamikhaylov

Labels

pr-improvement Pull request with some product improvements

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@KochetovNicolai @nikitamikhaylov @akuzm @robot-clickhouse