-
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
[ut](datatype) Add BE UT for some basic data types and serializers #48788
base: master
Are you sure you want to change the base?
Conversation
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
6cb0295
to
f8fe567
Compare
run buildall |
f25334b
to
b9b981b
Compare
run buildall |
TPC-H: Total hot run time: 32381 ms
|
TPC-DS: Total hot run time: 184848 ms
|
ClickBench: Total hot run time: 31.06 s
|
5ec7ffc
to
5365220
Compare
run buildall |
TPC-H: Total hot run time: 32049 ms
|
TPC-DS: Total hot run time: 191794 ms
|
ClickBench: Total hot run time: 31.25 s
|
0a68e98
to
92e0daa
Compare
run buildall |
TPC-H: Total hot run time: 32985 ms
|
TPC-DS: Total hot run time: 192504 ms
|
ClickBench: Total hot run time: 31.05 s
|
5bbe8aa
to
5693f00
Compare
run buildall |
TPC-H: Total hot run time: 32820 ms
|
TPC-DS: Total hot run time: 191877 ms
|
ClickBench: Total hot run time: 31.38 s
|
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
b46a1ee
to
31f8cb7
Compare
run buildall |
TPC-H: Total hot run time: 32865 ms
|
TPC-DS: Total hot run time: 192012 ms
|
ClickBench: Total hot run time: 30.71 s
|
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
… and ColumnDictionary
DataTypeNumber, DataTypeDecimal, DataTypeString, DataTypeDateTimeV2
31f8cb7
to
3331ed4
Compare
run buildall |
TPC-H: Total hot run time: 32973 ms
|
TPC-DS: Total hot run time: 191991 ms
|
What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
DataTypeNumber, DataTypeDecimal, DataTypeString, DataTypeDateTimeV2
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)