Skip to content

Start untangling Exception.h and Field.h#76299

Merged
Algunenano merged 14 commits intoClickHouse:masterfrom
Algunenano:exception_compilation
Feb 20, 2025
Merged

Start untangling Exception.h and Field.h#76299
Algunenano merged 14 commits intoClickHouse:masterfrom
Algunenano:exception_compilation

Conversation

@Algunenano
Copy link
Copy Markdown
Member

@Algunenano Algunenano commented Feb 17, 2025

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Start untangling Exception.h and Field.h

Technically, this PR does little to nothing to improve compilation times. The idea is to start hiding Exception.h when possible, but the problem is that it's used in ~100 different headers so when you remove it from one you still get it from another one. Since the PR already contains ~250 changed files and touches some really common / used files, which could affect compilation and performance, let's do a quick check and merge this and continue later.

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Feb 17, 2025

Workflow [PR], commit [e8b8541]

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Feb 17, 2025
@Algunenano Algunenano assigned Algunenano and unassigned Algunenano Feb 18, 2025
@Algunenano
Copy link
Copy Markdown
Member Author

Algunenano commented Feb 19, 2025

Differences in perf tests might be related to the changes in Field::safeGet<>. Need to check

Checked. No noticeable differences locally. Same QPS at ~31

@Algunenano Algunenano added the 🚀 Fast CI 🥒 A fast database deserves a fast CI label Feb 20, 2025
@Algunenano Algunenano enabled auto-merge February 20, 2025 18:58
@Algunenano Algunenano added this pull request to the merge queue Feb 20, 2025
Merged via the queue into ClickHouse:master with commit 2eded0e Feb 20, 2025
121 of 126 checks passed
@Algunenano Algunenano deleted the exception_compilation branch February 20, 2025 19:27
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Feb 20, 2025
baibaichen pushed a commit to Kyligence/ClickHouse that referenced this pull request Feb 21, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
baibaichen added a commit to Kyligence/gluten that referenced this pull request Feb 21, 2025
baibaichen pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 18, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 18, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
baibaichen pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 19, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 19, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
baibaichen pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 20, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 20, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 21, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 22, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 23, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 24, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
baibaichen pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 26, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 26, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 27, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 28, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 29, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 30, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Mar 31, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Apr 1, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Apr 2, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Apr 3, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Apr 4, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Apr 5, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Apr 6, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Apr 7, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
baibaichen pushed a commit to Kyligence/ClickHouse that referenced this pull request Apr 9, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Apr 9, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Apr 10, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Apr 11, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Apr 12, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
kyligence-git pushed a commit to Kyligence/ClickHouse that referenced this pull request Apr 13, 2025
(cherry picked from commit aa13a91)

- 20250221 Fix Build due to ClickHouse#76299
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚀 Fast CI 🥒 A fast database deserves a fast CI pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants