-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[improvement](meta) Switch meta serialization to gson 1 #36125
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
fe/fe-core/src/main/java/org/apache/doris/analysis/AlterDatabaseQuotaStmt.java
Outdated
Show resolved
Hide resolved
run buildall |
432d403
to
9036c57
Compare
run buildall |
2 similar comments
run buildall |
run buildall |
run buildall |
TPC-H: Total hot run time: 40402 ms
|
run buildall |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
TPC-H: Total hot run time: 39793 ms
|
TPC-DS: Total hot run time: 172961 ms
|
ClickBench: Total hot run time: 30.66 s
|
run buildall |
1 similar comment
run buildall |
TPC-H: Total hot run time: 40389 ms
|
TPC-DS: Total hot run time: 172023 ms
|
ClickBench: Total hot run time: 30.04 s
|
run buildall |
run buildall |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR approved by at least one committer and no changes requested. |
TPC-H: Total hot run time: 39770 ms
|
run buildall |
TPC-H: Total hot run time: 39808 ms
|
run buildall |
TPC-H: Total hot run time: 39988 ms
|
TPC-DS: Total hot run time: 170101 ms
|
ClickBench: Total hot run time: 30.58 s
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FsBroker is wrong.
run buildall |
run buildall |
TPC-H: Total hot run time: 39845 ms
|
TPC-DS: Total hot run time: 170148 ms
|
ClickBench: Total hot run time: 31.23 s
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR approved by at least one committer and no changes requested. |
## Proposed changes Issue Number: Switch meta serialization to gson Contains the following classes: ~~JournalEntity~~ DatabaseInfo ~~PartitionPersistInfo~~ ConsistencyCheckInfo Timestamp BrokerMgr.ModifyBrokerInfo LoadErrorHub.Param TableInfo --------- Co-authored-by: zhangpeicheng <zhangpeicheng@meituan.com>
Proposed changes
Issue Number:
Switch meta serialization to gson
Contains the following classes:
JournalEntityDatabaseInfo
PartitionPersistInfoConsistencyCheckInfo
Timestamp
BrokerMgr.ModifyBrokerInfo
LoadErrorHub.Param
TableInfo