Skip to content

Conversation

gingerwizard
Copy link
Collaborator

Summary

enhances projections, fixes uk price paid, manual toc for data (for now) and removes redundant postgres comments

Checklist

@gingerwizard gingerwizard requested a review from Blargian April 11, 2025 18:01
@gingerwizard gingerwizard requested a review from a team as a code owner April 11, 2025 18:01
Copy link

vercel bot commented Apr 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
clickhouse-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 11, 2025 7:30pm
clickhouse-docs-jp 🔄 Building (Inspect) Visit Preview 💬 Add feedback Apr 11, 2025 7:30pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
clickhouse-docs-ru ⬜️ Ignored (Inspect) Visit Preview Apr 11, 2025 7:30pm
clickhouse-docs-zh ⬜️ Ignored (Inspect) Visit Preview Apr 11, 2025 7:30pm

The results should be the same, but the performance better on the latter example!


#### Query 2. Average Price Per Year in London {#average-price-london-projections}
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
#### Query 2. Average Price Per Year in London {#average-price-london-projections}
#### Query 2. Average price per year in London {#average-price-london-projections}

ORDER BY year ASC
```

#### Query 3. The Most Expensive Neighborhoods {#most-expensive-neighborhoods-projections}
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
#### Query 3. The Most Expensive Neighborhoods {#most-expensive-neighborhoods-projections}
#### Query 3. The most expensive neighborhoods {#most-expensive-neighborhoods-projections}


The following queries contrast performance with and without projections. To disable projection use we use the setting [`optimize_use_projections`](/operations/settings/settings#optimize_use_projections), which is enabled by default.

#### Query 1. Average Price Per Year {#average-price-projections}
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
#### Query 1. Average Price Per Year {#average-price-projections}
#### Query 1. Average price per year {#average-price-projections}

@gingerwizard gingerwizard merged commit d6ac65c into main Apr 11, 2025
7 of 11 checks passed
@gingerwizard gingerwizard deleted the uk_postgres branch April 13, 2025 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants