Start untangling Exception.h and Field.h#76299
Merged
Algunenano merged 14 commits intoClickHouse:masterfrom Feb 20, 2025
Merged
Start untangling Exception.h and Field.h#76299Algunenano merged 14 commits intoClickHouse:masterfrom
Algunenano merged 14 commits intoClickHouse:masterfrom
Conversation
Contributor
Member
Author
|
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 |
Merged
via the queue into
ClickHouse:master
with commit Feb 20, 2025
2eded0e
121 of 126 checks passed
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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changelog category (leave one):
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