You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "zhangstar333 (via GitHub)" <gi...@apache.org> on 2025/10/16 15:24:34 UTC
[PR] [ut](be) add some ut test case for varbinary [doris]
zhangstar333 opened a new pull request, #57070:
URL: https://github.com/apache/doris/pull/57070
### What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [ ] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg: https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR should merge into -->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3411618073
<details>
<summary>TPC-DS: <b>Total hot run time: 189324 ms</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
TPC-DS sf100 test result on commit 0870e9e259a833e67bab4a8201188bfa08ffb628, data reload: false
query1 1083 419 406 406
query2 6573 1668 1696 1668
query3 6761 227 218 218
query4 26863 23817 23082 23082
query5 5132 639 486 486
query6 353 233 211 211
query7 4658 493 299 299
query8 300 278 262 262
query9 8709 2574 2572 2572
query10 522 339 308 308
query11 15731 14925 14986 14925
query12 187 122 113 113
query13 1674 543 431 431
query14 11487 9182 9458 9182
query15 214 197 181 181
query16 7682 650 508 508
query17 1283 774 623 623
query18 2096 484 387 387
query19 279 229 195 195
query20 146 132 138 132
query21 219 140 133 133
query22 4668 4812 4500 4500
query23 34989 33137 33727 33137
query24 8602 2568 2627 2568
query25 626 553 456 456
query26 1264 281 162 162
query27 2803 510 360 360
query28 4584 2214 2216 2214
query29 815 674 519 519
query30 298 233 219 219
query31 981 849 798 798
query32 107 95 84 84
query33 970 392 325 325
query34 827 865 556 556
query35 848 848 785 785
query36 1031 1010 962 962
query37 139 120 95 95
query38 3729 3832 3650 3650
query39 1546 1495 1463 1463
query40 226 132 125 125
query41 67 75 66 66
query42 139 120 115 115
query43 500 521 477 477
query44 1396 871 845 845
query45 205 184 178 178
query46 873 1022 658 658
query47 1882 1934 1820 1820
query48 423 437 366 366
query49 846 526 441 441
query50 690 706 455 455
query51 4140 4207 4066 4066
query52 110 107 102 102
query53 240 267 200 200
query54 608 597 518 518
query55 89 88 81 81
query56 327 331 297 297
query57 1156 1184 1099 1099
query58 275 269 276 269
query59 2515 2724 2485 2485
query60 351 354 324 324
query61 152 150 154 150
query62 801 752 681 681
query63 228 196 197 196
query64 4384 1221 833 833
query65 4054 3986 3978 3978
query66 1074 427 320 320
query67 15118 15184 14949 14949
query68 9733 966 597 597
query69 514 323 284 284
query70 1411 1264 1253 1253
query71 524 358 314 314
query72 5799 4856 4847 4847
query73 831 574 355 355
query74 8957 8831 8699 8699
query75 4489 3386 2789 2789
query76 5413 1170 746 746
query77 1085 393 298 298
query78 9543 9772 8933 8933
query79 7324 849 576 576
query80 735 561 488 488
query81 483 271 232 232
query82 395 160 139 139
query83 295 278 253 253
query84 301 107 91 91
query85 868 472 423 423
query86 335 317 287 287
query87 3819 3781 3564 3564
query88 2886 2242 2219 2219
query89 446 333 286 286
query90 2109 222 226 222
query91 172 159 131 131
query92 86 70 66 66
query93 3858 1007 643 643
query94 690 427 330 330
query95 400 333 315 315
query96 494 598 281 281
query97 2904 2954 2872 2872
query98 235 219 217 217
query99 1439 1441 1305 1305
Total cold run time: 292221 ms
Total hot run time: 189324 ms
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "zhangstar333 (via GitHub)" <gi...@apache.org>.
zhangstar333 merged PR #57070:
URL: https://github.com/apache/doris/pull/57070
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3486188680
# BE UT Coverage Report
Increment line coverage ` ` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/71463a4da6494aaf8f0bd6404a8fa9edebfce387_71463a4da6494aaf8f0bd6404a8fa9edebfce387/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/71463a4da6494aaf8f0bd6404a8fa9edebfce387_71463a4da6494aaf8f0bd6404a8fa9edebfce387/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 52.74% (18085/34294) |
| Line Coverage | 38.05% (164493/432294) |
| Region Coverage | 32.36% (125317/387300) |
| Branch Coverage | 33.81% (54908/162402) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3486565799
# BE Regression && UT Coverage Report
Increment line coverage `100% (0/0)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/57070_71463a4da6494aaf8f0bd6404a8fa9edebfce387_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/57070_71463a4da6494aaf8f0bd6404a8fa9edebfce387_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 71.50% (24105/33712) |
| Line Coverage | 57.92% (250822/433038) |
| Region Coverage | 52.98% (208142/392894) |
| Branch Coverage | 54.69% (89429/163511) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "zhangstar333 (via GitHub)" <gi...@apache.org>.
zhangstar333 commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3485862503
run buildall
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3486009960
<details>
<summary>ClickBench: <b>Total hot run time: 27.27 s</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
ClickBench test result on commit 71463a4da6494aaf8f0bd6404a8fa9edebfce387, data reload: false
query1 0.06 0.05 0.05
query2 0.09 0.05 0.05
query3 0.25 0.08 0.08
query4 1.61 0.12 0.11
query5 0.27 0.26 0.26
query6 1.18 0.64 0.64
query7 0.03 0.02 0.02
query8 0.05 0.04 0.04
query9 0.59 0.51 0.51
query10 0.58 0.58 0.59
query11 0.16 0.12 0.11
query12 0.16 0.12 0.12
query13 0.61 0.60 0.61
query14 1.00 1.01 1.00
query15 0.84 0.85 0.84
query16 0.39 0.40 0.39
query17 1.02 1.02 1.06
query18 0.21 0.20 0.19
query19 1.86 1.78 1.74
query20 0.01 0.02 0.02
query21 15.43 0.18 0.12
query22 4.97 0.07 0.05
query23 15.68 0.26 0.10
query24 2.42 1.29 0.29
query25 0.07 0.06 0.06
query26 0.15 0.13 0.14
query27 0.07 0.06 0.05
query28 5.07 1.14 0.94
query29 12.55 3.99 3.23
query30 0.29 0.14 0.12
query31 2.81 0.57 0.38
query32 3.23 0.55 0.47
query33 2.97 3.03 3.01
query34 15.86 5.20 4.54
query35 4.64 4.61 4.56
query36 0.66 0.50 0.50
query37 0.10 0.07 0.07
query38 0.06 0.05 0.04
query39 0.03 0.03 0.03
query40 0.17 0.14 0.13
query41 0.08 0.03 0.04
query42 0.04 0.02 0.02
query43 0.04 0.04 0.03
Total cold run time: 98.36 s
Total hot run time: 27.27 s
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3412161708
# BE Regression && UT Coverage Report
Increment line coverage `100% (0/0)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/57070_0870e9e259a833e67bab4a8201188bfa08ffb628_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/57070_0870e9e259a833e67bab4a8201188bfa08ffb628_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 71.27% (23695/33248) |
| Line Coverage | 57.72% (247428/428701) |
| Region Coverage | 52.85% (205271/388432) |
| Branch Coverage | 54.62% (88604/162209) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "Thearas (via GitHub)" <gi...@apache.org>.
Thearas commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3411444297
Thank you for your contribution to Apache Doris.
Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR).
Please clearly describe your PR:
1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
3. What features were added. Why was this function added?
4. Which code was refactored and why was this part of the code refactored?
5. Which functions were optimized and what is the difference before and after the optimization?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3411689105
# BE UT Coverage Report
Increment line coverage ` ` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/0870e9e259a833e67bab4a8201188bfa08ffb628_0870e9e259a833e67bab4a8201188bfa08ffb628/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/0870e9e259a833e67bab4a8201188bfa08ffb628_0870e9e259a833e67bab4a8201188bfa08ffb628/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 52.55% (17821/33910) |
| Line Coverage | 37.76% (161948/428944) |
| Region Coverage | 32.22% (123563/383470) |
| Branch Coverage | 33.61% (54223/161314) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "zclllyybb (via GitHub)" <gi...@apache.org>.
zclllyybb commented on code in PR #57070:
URL: https://github.com/apache/doris/pull/57070#discussion_r2443820448
##########
be/test/vec/columns/column_varbinary_test.cpp:
##########
@@ -454,4 +456,75 @@ TEST_F(ColumnVarbinaryTest, AllocatedBytesAndHasEnoughCapacity) {
ASSERT_FALSE(dest->has_enough_capacity(*src_big));
}
+TEST_F(ColumnVarbinaryTest, InsertRangeFromOutOfBoundsThrows) {
+ auto src = ColumnVarbinary::create();
+ std::vector<std::string> vals = {make_bytes(2, 0x10), make_bytes(3, 0x20)};
+ for (auto& v : vals) {
+ src->insert_data(v.data(), v.size());
+ }
+
+ auto dst = ColumnVarbinary::create();
+ // start+length 超界应抛出 doris::Exception
Review Comment:
should use English
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3489141056
# BE Regression && UT Coverage Report
Increment line coverage `100% (0/0)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/57070_71463a4da6494aaf8f0bd6404a8fa9edebfce387_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/57070_71463a4da6494aaf8f0bd6404a8fa9edebfce387_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 71.50% (24105/33712) |
| Line Coverage | 57.92% (250808/433038) |
| Region Coverage | 52.94% (208002/392894) |
| Branch Coverage | 54.68% (89407/163511) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3489887623
PR approved by anyone and no changes requested.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3485991541
<details>
<summary>TPC-DS: <b>Total hot run time: 189426 ms</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
TPC-DS sf100 test result on commit 71463a4da6494aaf8f0bd6404a8fa9edebfce387, data reload: false
query1 1069 426 388 388
query2 6565 1691 1684 1684
query3 6755 220 222 220
query4 26324 23328 23028 23028
query5 4404 647 473 473
query6 322 238 229 229
query7 4650 502 291 291
query8 305 269 258 258
query9 8711 2566 2588 2566
query10 503 349 303 303
query11 15364 15090 14775 14775
query12 193 115 109 109
query13 1673 555 431 431
query14 11453 9066 9200 9066
query15 193 191 184 184
query16 7669 635 515 515
query17 1493 781 621 621
query18 2034 432 330 330
query19 223 216 191 191
query20 136 133 144 133
query21 223 150 127 127
query22 4777 4890 4694 4694
query23 35058 33930 33907 33907
query24 8426 2477 2491 2477
query25 600 529 498 498
query26 1245 284 168 168
query27 2836 535 379 379
query28 5189 2243 2225 2225
query29 820 613 495 495
query30 302 237 206 206
query31 987 852 762 762
query32 108 71 70 70
query33 714 406 344 344
query34 914 910 562 562
query35 797 844 784 784
query36 1012 1018 966 966
query37 149 112 87 87
query38 3630 3670 3522 3522
query39 1450 1393 1415 1393
query40 219 122 114 114
query41 62 55 57 55
query42 119 112 114 112
query43 487 479 460 460
query44 1216 729 735 729
query45 180 177 174 174
query46 894 978 641 641
query47 1751 1819 1704 1704
query48 391 425 332 332
query49 777 490 399 399
query50 629 689 411 411
query51 3918 3901 3797 3797
query52 111 107 99 99
query53 230 259 193 193
query54 297 283 279 279
query55 80 84 81 81
query56 313 334 301 301
query57 1201 1197 1130 1130
query58 294 279 309 279
query59 2533 2749 2520 2520
query60 374 356 342 342
query61 195 194 193 193
query62 801 747 688 688
query63 234 195 200 195
query64 4584 1206 843 843
query65 4045 3941 3950 3941
query66 1098 416 328 328
query67 15405 15105 15020 15020
query68 8335 862 591 591
query69 493 321 287 287
query70 1264 1322 1287 1287
query71 442 332 327 327
query72 5754 4997 4872 4872
query73 636 560 349 349
query74 8844 9195 9168 9168
query75 3503 3368 2814 2814
query76 3440 1148 731 731
query77 710 387 308 308
query78 9486 9675 8915 8915
query79 2158 821 592 592
query80 682 575 480 480
query81 509 263 234 234
query82 207 154 129 129
query83 258 259 244 244
query84 255 113 87 87
query85 870 484 447 447
query86 377 307 293 293
query87 3695 3737 3631 3631
query88 3131 2246 2233 2233
query89 382 320 285 285
query90 2048 215 215 215
query91 159 164 146 146
query92 81 66 61 61
query93 2285 977 625 625
query94 673 437 323 323
query95 389 315 316 315
query96 481 586 277 277
query97 2897 2952 2858 2858
query98 245 219 207 207
query99 1365 1384 1274 1274
Total cold run time: 277768 ms
Total hot run time: 189426 ms
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3411639045
<details>
<summary>ClickBench: <b>Total hot run time: 30.59 s</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
ClickBench test result on commit 0870e9e259a833e67bab4a8201188bfa08ffb628, data reload: false
query1 0.05 0.05 0.05
query2 0.09 0.05 0.06
query3 0.26 0.08 0.08
query4 1.61 0.12 0.12
query5 0.28 0.26 0.25
query6 1.18 0.67 0.64
query7 0.03 0.02 0.02
query8 0.06 0.05 0.04
query9 0.63 0.53 0.53
query10 0.59 0.60 0.57
query11 0.16 0.11 0.11
query12 0.15 0.12 0.12
query13 0.64 0.62 0.61
query14 1.02 1.03 1.03
query15 0.86 0.85 0.86
query16 0.39 0.39 0.39
query17 1.03 1.08 1.06
query18 0.21 0.20 0.19
query19 1.93 1.84 1.87
query20 0.02 0.01 0.02
query21 15.42 0.91 0.58
query22 0.78 1.14 0.63
query23 15.00 1.37 0.65
query24 7.05 1.86 0.85
query25 0.51 0.13 0.15
query26 0.68 0.16 0.14
query27 0.06 0.05 0.05
query28 10.19 1.33 0.93
query29 12.58 3.96 3.34
query30 0.28 0.14 0.11
query31 2.82 0.61 0.39
query32 3.24 0.57 0.47
query33 3.07 3.12 3.29
query34 16.06 5.44 4.84
query35 4.92 4.90 4.98
query36 0.68 0.52 0.51
query37 0.10 0.07 0.08
query38 0.06 0.05 0.05
query39 0.03 0.03 0.03
query40 0.16 0.15 0.15
query41 0.09 0.04 0.03
query42 0.04 0.03 0.03
query43 0.05 0.04 0.03
Total cold run time: 105.06 s
Total hot run time: 30.59 s
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3489887485
PR approved by at least one committer and no changes requested.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3415186061
# BE Regression && UT Coverage Report
Increment line coverage `100% (0/0)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/57070_0870e9e259a833e67bab4a8201188bfa08ffb628_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/57070_0870e9e259a833e67bab4a8201188bfa08ffb628_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 71.25% (23690/33248) |
| Line Coverage | 57.69% (247306/428701) |
| Region Coverage | 52.83% (205199/388432) |
| Branch Coverage | 54.60% (88566/162209) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3413668086
# BE Regression && UT Coverage Report
Increment line coverage `100% (0/0)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/57070_0870e9e259a833e67bab4a8201188bfa08ffb628_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/57070_0870e9e259a833e67bab4a8201188bfa08ffb628_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 70.99% (23603/33248) |
| Line Coverage | 57.42% (246140/428701) |
| Region Coverage | 52.46% (203783/388432) |
| Branch Coverage | 54.32% (88109/162209) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "zhangstar333 (via GitHub)" <gi...@apache.org>.
zhangstar333 commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3411444640
run buildall
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [ut](be) add some ut test case for varbinary [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #57070:
URL: https://github.com/apache/doris/pull/57070#issuecomment-3413787568
# BE Regression && UT Coverage Report
Increment line coverage `100% (0/0)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/57070_0870e9e259a833e67bab4a8201188bfa08ffb628_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/57070_0870e9e259a833e67bab4a8201188bfa08ffb628_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 70.97% (23596/33248) |
| Line Coverage | 57.38% (245990/428701) |
| Region Coverage | 52.42% (203607/388432) |
| Branch Coverage | 54.28% (88055/162209) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org