Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly merged PR #61029: URL: https://github.com/apache/doris/pull/61029 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
github-actions[bot] commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4072162517 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4066200951 # BE Regression && UT Coverage Report Increment line coverage `100% (0/0)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 57.15% (20951/36660) | | Line Coverage | 40.20% (203719/506729) | | Region Coverage | 36.72% (162805/443321) | | Branch Coverage | 37.55% (69734/185709) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4065688038 run external -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4065424484 # BE Regression && UT Coverage Report Increment line coverage `100% (0/0)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 56.88% (20852/36660) | | Line Coverage | 39.97% (202536/506729) | | Region Coverage | 36.49% (161775/443321) | | Branch Coverage | 37.28% (69226/185709) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4065393663 # BE UT Coverage Report Increment line coverage ` ` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207/report/index.html) | Category | Coverage | |---|| | Function Coverage | 52.62% (19708/37454) | | Line Coverage | 36.20% (184040/508442) | | Region Coverage | 32.34% (142055/439272) | | Branch Coverage | 33.53% (62104/185227) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4065324185 # BE Regression && UT Coverage Report Increment line coverage `100% (0/0)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 55.84% (20472/36660) | | Line Coverage | 38.65% (195842/506739) | | Region Coverage | 35.36% (156774/443321) | | Branch Coverage | 35.92% (66714/185709) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4065246859 # BE Regression && UT Coverage Report Increment line coverage `100% (0/0)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 55.77% (20447/36660) | | Line Coverage | 38.57% (195434/506739) | | Region Coverage | 35.46% (157194/443321) | | Branch Coverage | 35.99% (66831/185709) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4065113929 # BE Regression && UT Coverage Report Increment line coverage `100% (0/0)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 55.77% (20447/36660) | | Line Coverage | 38.57% (195434/506739) | | Region Coverage | 35.46% (157194/443321) | | Branch Coverage | 35.99% (66831/185709) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4064901882 # FE UT Coverage Report Increment line coverage ` 84.62% (11/13)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207/fe_increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207/fe_report/index.html) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4064798107 TPC-DS: Total hot run time: 169905 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS sf100 test result on commit e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207, data reload: false query5 4333649 521 521 query6 331 230 204 204 query7 4226471 262 262 query8 337 254 237 237 query9 8699275927512751 query10 518 421 348 348 query11 7014515349434943 query12 190 132 125 125 query13 1291465 358 358 query14 5658382635563556 query14_12926290228642864 query15 209 200 175 175 query16 981 461 434 434 query17 895 733 640 640 query18 2448461 353 353 query19 219 212 189 189 query20 142 128 131 128 query21 220 139 117 117 query22 13484 14283 14587 14283 query23 16240 16015 15659 15659 query23_115934 15855 15719 15719 query24 7239163512371237 query24_11239124912491249 query25 573 532 404 404 query26 1239269 154 154 query27 2767486 306 306 query28 4470187318771873 query29 835 566 487 487 query30 298 230 193 193 query31 996 962 861 861 query32 83 71 70 70 query33 526 331 293 293 query34 897 895 528 528 query35 638 695 620 620 query36 10961127975 975 query37 130 98 87 87 query38 3048303529252925 query39 871 842 810 810 query39_1798 802 798 798 query40 228 154 139 139 query41 63 63 57 57 query42 262 260 260 260 query43 247 251 219 219 query44 query45 199 199 184 184 query46 876 984 610 610 query47 2091209120622062 query48 305 322 225 225 query49 634 459 389 389 query50 685 280 228 228 query51 4100414041024102 query52 268 270 259 259 query53 291 340 284 284 query54 318 280 290 280 query55 90 90 82 82 query56 324 334 312 312 query57 1950178215761576 query58 285 281 283 281 query59 2830297527622762 query60 343 350 337 337 query61 154 148 147 147 query62 630 593 537 537 query63 321 290 282 282 query64 50641300995 995 query65 query66 1451453 363 363 query67 24445 24469 24517 24469 query68 query69 406 320 297 297 query70 982 995 952 952 query71 347 316 305 305 query72 2788268024862486 query73 546 558 325 325 query74 9641962794389438 query75 2846275724992499 query76 22981047706 706 query77 409 398 316 316 query78 11016 11158 10517 10517 query79 2274813 589 589 query80 1757643 552 552 query81 557 265 230 230 query82 1018162 135 135 query83 341 265 257 257 query84 295 115 107 107 query85 887 489 453 453 query86 418 309 293 293 query87 3158321229862986 query88 3594265526472647 query89 435 376 362 362 query90 2026192 191 191 query91 170 162 135 135 query92 78 79 75 75 query93 1117897 513 513 query94 629 304 300 300 query95 604 358 383 358 query96 641 530 236 236 query97 2477252124022402 query98 243 223 224 223 query99 1000999 893 893 Total cold run time: 252005 ms Total hot run time: 169905 ms ``` -- 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: [email protected].
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4064763243 TPC-H: Total hot run time: 26809 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207, data reload: false -- Round 1 -- orders Doris NULLNULL0 0 0 NULL0 NULLNULL2023-12-26 18:27:23 2023-12-26 18:42:55 NULLutf-8 NULLNULL q1 17633 443342994299 q2 q3 10648 795 535 535 q4 4675353 251 251 q5 7548119810441044 q6 176 172 149 149 q7 771 844 665 665 q8 9289146113511351 q9 4918476946954695 q10 6289193316691669 q11 458 241 243 241 q12 743 575 458 458 q13 18031 294721772177 q14 230 230 214 214 q15 q16 720 740 659 659 q17 713 852 425 425 q18 5935534553045304 q19 1121971 612 612 q20 541 492 380 380 q21 4401182613911391 q22 337 290 401 290 Total cold run time: 95177 ms Total hot run time: 26809 ms - Round 2, with runtime_filter_mode=off - orders Doris NULLNULL15000 42 6422171781 NULL22778155NULLNULL2023-12-26 18:27:23 2023-12-26 18:42:55 NULLutf-8 NULLNULL q1 4807468646624662 q2 q3 3904435938963896 q4 880 1200768 768 q5 4023436543854365 q6 194 181 150 150 q7 1823174715671567 q8 2494266725552555 q9 7510749973487348 q10 3778395336383638 q11 517 488 419 419 q12 491 600 436 436 q13 2778327424452445 q14 310 331 300 300 q15 q16 772 776 710 710 q17 1174143313621362 q18 7138682966466646 q19 849 874 918 874 q20 2093214320212021 q21 3934344733253325 q22 488 428 372 372 Total cold run time: 49957 ms Total hot run time: 47859 ms ``` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4064681441 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4038497118 # BE Regression && UT Coverage Report Increment line coverage `100.00% (1/1)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 57.13% (20913/36606) | | Line Coverage | 40.20% (203063/505109) | | Region Coverage | 36.87% (163079/442262) | | Branch Coverage | 37.62% (69635/185086) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4037852562 # BE UT Coverage Report Increment line coverage `100.00% (1/1)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207_e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207/report/index.html) | Category | Coverage | |---|| | Function Coverage | 52.56% (19648/37384) | | Line Coverage | 36.16% (183221/506650) | | Region Coverage | 32.29% (141466/438114) | | Branch Coverage | 33.45% (61725/184510) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4037481583 TPC-DS: Total hot run time: 153027 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS sf100 test result on commit e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207, data reload: false query5 4361648 530 530 query6 337 231 208 208 query7 4234469 268 268 query8 370 256 233 233 query9 8720274627512746 query10 535 398 354 354 query11 7348588256405640 query12 210 126 129 126 query13 1269452 351 351 query14 5902387035513551 query14_12816280328202803 query15 202 194 182 182 query16 1041479 457 457 query17 1109719 619 619 query18 2481445 352 352 query19 219 217 185 185 query20 142 132 130 130 query21 222 143 122 122 query22 4714490746404640 query23 15925 15570 15377 15377 query23_115460 16336 15992 15992 query24 7460166212641264 query24_11279126913071269 query25 577 518 440 440 query26 1484297 176 176 query27 3267503 304 304 query28 4556192519231923 query29 886 607 486 486 query30 309 250 217 217 query31 1367128012271227 query32 84 74 73 73 query33 497 332 277 277 query34 918 907 560 560 query35 645 667 599 599 query36 10841129985 985 query37 133 101 84 84 query38 2929293129002900 query39 899 857 845 845 query39_1826 848 827 827 query40 233 150 134 134 query41 62 58 59 58 query42 299 324 296 296 query43 239 254 220 220 query44 query45 203 192 180 180 query46 873 977 604 604 query47 2113212420812081 query48 313 311 228 228 query49 629 442 381 381 query50 670 273 214 214 query51 4214406640624062 query52 292 289 285 285 query53 288 332 286 286 query54 302 273 271 271 query55 97 88 86 86 query56 310 323 310 310 query57 1368134113011301 query58 289 271 278 271 query59 1333146412701270 query60 335 339 321 321 query61 148 143 152 143 query62 619 570 534 534 query63 313 279 282 279 query64 51131294981 981 query65 query66 1453462 356 356 query67 16518 16365 16270 16270 query68 query69 382 306 280 280 query70 1012979 985 979 query71 339 310 323 310 query72 3043284426402640 query73 532 546 319 319 query74 9954989597609760 query75 2860274924892489 query76 23381036663 663 query77 354 389 301 301 query78 11236 11457 10670 10670 query79 1148762 622 622 query80 1298618 535 535 query81 534 279 245 245 query82 1370157 124 124 query83 349 269 258 258 query84 250 122 96 96 query85 1162498 431 431 query86 407 318 315 315 query87 3170312230033003 query88 3510265826022602 query89 420 364 353 353 query90 1796178 183 178 query91 172 207 136 136 query92 79 73 71 71 query93 896 860 505 505 query94 542 319 302 302 query95 573 397 313 313 query96 645 516 225 225 query97 2483248624062406 query98 241 233 225 225 query99 1029991 902 902 Total cold run time: 233850 ms Total hot run time: 153027 ms ``` -- 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: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4037422910 TPC-H: Total hot run time: 27828 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit e6366c4e3da410dcacdee4a9ff39e4a7f8f9f207, data reload: false -- Round 1 -- q1 17602 453143334333 q2 q3 10652 814 517 517 q4 4674376 275 275 q5 7561119910271027 q6 173 184 146 146 q7 791 851 667 667 q8 9295145313431343 q9 4810466846734668 q10 6252188116441644 q11 450 261 240 240 q12 705 564 462 462 q13 18027 301122202220 q14 233 231 219 219 q15 931 808 823 808 q16 721 736 690 690 q17 689 834 444 444 q18 5884529753155297 q19 12671004599 599 q20 510 498 391 391 q21 4703199915551555 q22 368 350 283 283 Total cold run time: 96298 ms Total hot run time: 27828 ms - Round 2, with runtime_filter_mode=off - q1 4814453445554534 q2 q3 3871434538753875 q4 896 1200778 778 q5 4088435743154315 q6 180 180 142 142 q7 1803163715341534 q8 2473267628002676 q9 7443739074257390 q10 3833402836123612 q11 521 448 437 437 q12 509 592 451 451 q13 2676310123642364 q14 297 306 276 276 q15 895 813 824 813 q16 723 776 732 732 q17 1155133813721338 q18 7164682664806480 q19 933 950 938 938 q20 2163214420072007 q21 4078349933353335 q22 511 451 368 368 Total cold run time: 51026 ms Total hot run time: 48395 ms ``` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4037299030 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4037087863 # FE UT Coverage Report Increment line coverage ` 84.62% (11/13)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_d1bd34b360c37d91f41ceea20576734384d609a1/fe_increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_d1bd34b360c37d91f41ceea20576734384d609a1/fe_report/index.html) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4036774043 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4036755281 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4035587856 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4021482968 # FE UT Coverage Report Increment line coverage ` 84.62% (11/13)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_04c56e7f2af358a89910535c73ab7a6e12ab2a72/fe_increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_04c56e7f2af358a89910535c73ab7a6e12ab2a72/fe_report/index.html) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
github-actions[bot] commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4021463247 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4021334617 TPC-DS: Total hot run time: 153068 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS sf100 test result on commit 04c56e7f2af358a89910535c73ab7a6e12ab2a72, data reload: false query5 4369641 529 529 query6 335 227 208 208 query7 4231463 275 275 query8 338 255 234 234 query9 8723270827142708 query10 520 399 372 372 query11 7280591256705670 query12 180 133 135 133 query13 1267466 361 361 query14 5775379235913591 query14_12787279528322795 query15 210 188 174 174 query16 978 485 447 447 query17 935 723 624 624 query18 2439455 345 345 query19 214 212 182 182 query20 135 132 129 129 query21 227 145 121 121 query22 5097517350585058 query23 16531 15898 15859 15859 query23_115853 15759 15734 15734 query24 7429167312951295 query24_11259127313091273 query25 591 487 393 393 query26 1241270 149 149 query27 2812475 294 294 query28 4479182418301824 query29 851 554 470 470 query30 312 249 209 209 query31 1350128812081208 query32 80 74 71 71 query33 504 322 290 290 query34 900 935 572 572 query35 636 668 592 592 query36 11141132988 988 query37 135 98 85 85 query38 2932292028402840 query39 879 851 827 827 query39_1825 821 811 811 query40 242 150 134 134 query41 63 63 58 58 query42 291 300 296 296 query43 234 247 227 227 query44 query45 203 188 181 181 query46 871 969 604 604 query47 2133212920492049 query48 314 333 232 232 query49 614 460 375 375 query50 682 275 212 212 query51 4044409940154015 query52 279 290 287 287 query53 287 332 289 289 query54 301 275 259 259 query55 95 82 80 80 query56 303 317 309 309 query57 1370133412851285 query58 284 278 280 278 query59 1313142813081308 query60 333 336 315 315 query61 144 147 142 142 query62 643 582 541 541 query63 305 274 280 274 query64 50611269998 998 query65 query66 1454475 351 351 query67 16272 16543 16327 16327 query68 query69 393 310 299 299 query70 998 932 978 932 query71 350 314 300 300 query72 2849261323772377 query73 533 543 315 315 query74 9934988397099709 query75 2820273624792479 query76 22741018689 689 query77 355 383 307 307 query78 11089 11198 10623 10623 query79 2518777 613 613 query80 1715623 537 537 query81 554 283 239 239 query82 1022151 127 127 query83 322 267 244 244 query84 290 121 98 98 query85 909 505 440 440 query86 414 305 323 305 query87 3127311629932993 query88 3557266126212621 query89 425 372 343 343 query90 2041176 174 174 query91 160 154 135 135 query92 76 73 68 68 query93 1115839 500 500 query94 661 322 291 291 query95 581 339 375 339 query96 656 521 227 227 query97 2477249124142414 query98 230 209 220 209 query99 982 976 922 922 Total cold run time: 234146 ms Total hot run time: 153068 ms ``` -- 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: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4021304971 TPC-H: Total hot run time: 27560 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit 04c56e7f2af358a89910535c73ab7a6e12ab2a72, data reload: false -- Round 1 -- q1 17645 447742834283 q2 q3 10641 771 506 506 q4 4675355 253 253 q5 75561208997 997 q6 171 172 144 144 q7 789 840 688 688 q8 9685145913281328 q9 5061481147354735 q10 6319191116601660 q11 455 264 245 245 q12 747 567 478 478 q13 18039 290821772177 q14 240 226 218 218 q15 949 797 789 789 q16 732 723 660 660 q17 701 861 418 418 q18 5923536952325232 q19 1437984 604 604 q20 506 497 400 400 q21 4492215714781478 q22 435 340 267 267 Total cold run time: 97198 ms Total hot run time: 27560 ms - Round 2, with runtime_filter_mode=off - q1 4734458446154584 q2 q3 3871435238373837 q4 875 1180770 770 q5 4048433343734333 q6 187 172 150 150 q7 1807163515421542 q8 2495271225382538 q9 7668731474807314 q10 3850395436173617 q11 503 426 420 420 q12 470 603 425 425 q13 2714336323762376 q14 381 307 269 269 q15 899 810 792 792 q16 717 765 720 720 q17 1163153113571357 q18 7015681568456815 q19 908 934 882 882 q20 2060216919981998 q21 3958340533953395 q22 509 441 379 379 Total cold run time: 50832 ms Total hot run time: 48513 ms ``` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4021187195 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4020952810 TPC-DS: Total hot run time: 153327 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS sf100 test result on commit 04c56e7f2af358a89910535c73ab7a6e12ab2a72, data reload: false query5 4351650 523 523 query6 345 245 219 219 query7 4231483 267 267 query8 363 252 237 237 query9 8718276727452745 query10 491 389 346 346 query11 7468588656765676 query12 198 129 127 127 query13 1262451 362 362 query14 5681383235703570 query14_12818281228732812 query15 206 193 180 180 query16 1025478 468 468 query17 1118733 631 631 query18 2461457 355 355 query19 218 213 189 189 query20 138 133 133 133 query21 230 150 133 133 query22 4837506758515067 query23 16357 15924 15743 15743 query23_115883 15980 15768 15768 query24 7296161512281228 query24_11236123412511234 query25 570 501 467 467 query26 1247263 144 144 query27 2760470 282 282 query28 4486183718451837 query29 863 567 475 475 query30 306 244 214 214 query31 1350128712131213 query32 82 72 73 72 query33 513 321 277 277 query34 914 929 584 584 query35 649 697 599 599 query36 11121124973 973 query37 135 98 87 87 query38 2965297429332933 query39 902 881 833 833 query39_1842 832 842 832 query40 226 155 135 135 query41 62 63 59 59 query42 298 299 300 299 query43 244 263 224 224 query44 query45 196 189 190 189 query46 884 1006622 622 query47 2115213621082108 query48 327 331 227 227 query49 633 474 384 384 query50 700 277 221 221 query51 4120408840374037 query52 287 295 280 280 query53 295 333 287 287 query54 298 271 263 263 query55 87 93 83 83 query56 324 351 308 308 query57 1373134412631263 query58 293 285 279 279 query59 1369148212541254 query60 343 346 323 323 query61 159 149 158 149 query62 630 601 512 512 query63 308 282 275 275 query64 5083127310001000 query65 query66 1466487 380 380 query67 16371 16408 16144 16144 query68 query69 396 312 295 295 query70 1035100810041004 query71 352 316 304 304 query72 2873266224352435 query73 558 564 322 322 query74 10029 999298049804 query75 2854277924772477 query76 22901043679 679 query77 364 383 300 300 query78 11216 11392 10705 10705 query79 1150820 600 600 query80 1509624 570 570 query81 573 279 243 243 query82 1023147 126 126 query83 358 273 250 250 query84 250 123 101 101 query85 1109483 438 438 query86 425 341 295 295 query87 3149310630063006 query88 3582268726462646 query89 431 373 345 345 query90 1928189 176 176 query91 163 164 140 140 query92 80 74 72 72 query93 985 827 496 496 query94 644 339 286 286 query95 600 348 382 348 query96 662 539 226 226 query97 2502247223782378 query98 232 240 228 228 query99 10141011911 911 Total cold run time: 232715 ms Total hot run time: 153327 ms ``` -- 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: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4020920199 TPC-H: Total hot run time: 27863 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit 04c56e7f2af358a89910535c73ab7a6e12ab2a72, data reload: false -- Round 1 -- q1 17627 446543824382 q2 q3 10648 764 515 515 q4 4683372 257 257 q5 7559121610321032 q6 172 173 146 146 q7 775 841 670 670 q8 9329146512961296 q9 4766472447104710 q10 6264192016551655 q11 482 255 254 254 q12 703 573 478 478 q13 18024 298121912191 q14 228 237 222 222 q15 904 798 811 798 q16 757 719 676 676 q17 720 873 428 428 q18 5921532553135313 q19 1130996 607 607 q20 503 511 406 406 q21 4349197115251525 q22 390 340 302 302 Total cold run time: 95934 ms Total hot run time: 27863 ms - Round 2, with runtime_filter_mode=off - q1 4759455145304530 q2 q3 3887434437843784 q4 898 1224773 773 q5 4070440143314331 q6 179 175 138 138 q7 1797166915341534 q8 2436267426792674 q9 7508737773367336 q10 3858403536763676 q11 515 457 437 437 q12 501 604 455 455 q13 2768313422902290 q14 279 292 281 281 q15 857 810 789 789 q16 708 772 718 718 q17 1151139113191319 q18 7061671866616661 q19 911 879 906 879 q20 2085228220912091 q21 4008347933303330 q22 449 428 381 381 Total cold run time: 50685 ms Total hot run time: 48407 ms ``` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4020862810 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4003183592 # FE Regression Coverage Report Increment line coverage ` 100.00% (13/13)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_merge_fe/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_merge_fe/report/index.html) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4003100686 # BE Regression && UT Coverage Report Increment line coverage `100.00% (1/1)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 73.30% (26830/36601) | | Line Coverage | 56.61% (286095/505414) | | Region Coverage | 54.09% (239212/442225) | | Branch Coverage | 55.63% (103026/185212) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4003020396 # FE Regression Coverage Report Increment line coverage ` 100.00% (13/13)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_merge_fe/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_merge_fe/report/index.html) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4002975314 # BE Regression && UT Coverage Report Increment line coverage `100.00% (1/1)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 73.30% (26827/36601) | | Line Coverage | 56.60% (286057/505414) | | Region Coverage | 54.10% (239251/442225) | | Branch Coverage | 55.62% (103011/185212) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4002437796 run external -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4002437206 run p0 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4002281778 # FE Regression Coverage Report Increment line coverage ` 100.00% (13/13)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_merge_fe/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_merge_fe/report/index.html) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4002227806 # BE Regression && UT Coverage Report Increment line coverage `100.00% (1/1)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 73.28% (26823/36601) | | Line Coverage | 56.59% (286028/505414) | | Region Coverage | 54.10% (239252/442225) | | Branch Coverage | 55.61% (103002/185212) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4001865349 # BE UT Coverage Report Increment line coverage `100.00% (1/1)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/1b91fcbf0c7a6b98ddd1f18dc60f37c055563129_1b91fcbf0c7a6b98ddd1f18dc60f37c055563129/report/index.html) | Category | Coverage | |---|| | Function Coverage | 52.58% (19654/37379) | | Line Coverage | 36.22% (183646/506967) | | Region Coverage | 32.52% (142487/438088) | | Branch Coverage | 33.46% (61780/184648) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
github-actions[bot] commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4001787624 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4001764643 TPC-DS: Total hot run time: 184006 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS sf100 test result on commit 1b91fcbf0c7a6b98ddd1f18dc60f37c055563129, data reload: false query5 5244630 500 500 query6 334 224 201 201 query7 4218468 276 276 query8 347 259 245 245 query9 8749278127702770 query10 613 392 335 335 query11 16974 17497 17290 17290 query12 202 137 122 122 query13 1302517 374 374 query14 7152341630723072 query14_13019289228702870 query15 210 205 179 179 query16 1006495 471 471 query17 1607821 623 623 query18 2809436 335 335 query19 218 211 178 178 query20 135 126 132 126 query21 216 130 108 108 query22 4849505147584758 query23 17201 16781 16564 16564 query23_116676 16866 16671 16671 query24 7148162012291229 query24_11234120612331206 query25 575 468 402 402 query26 1232261 145 145 query27 2773487 295 295 query28 4509188718961887 query29 789 563 464 464 query30 310 243 209 209 query31 853 728 646 646 query32 82 70 68 68 query33 512 320 284 284 query34 926 917 567 567 query35 641 688 620 620 query36 10431092935 935 query37 136 97 85 85 query38 2935291928902890 query39 881 882 813 813 query39_1828 827 832 827 query40 227 152 138 138 query41 62 60 57 57 query42 109 108 108 108 query43 379 391 358 358 query44 query45 191 190 184 184 query46 904 996 613 613 query47 2085210820512051 query48 343 321 234 234 query49 626 464 375 375 query50 697 278 212 212 query51 4085411240274027 query52 106 108 97 97 query53 289 336 288 288 query54 289 277 262 262 query55 83 85 83 83 query56 301 324 316 316 query57 1359133712781278 query58 300 276 278 276 query59 2582273325892589 query60 336 334 330 330 query61 162 153 148 148 query62 621 584 543 543 query63 315 281 280 280 query64 4875127810081008 query65 query66 1380455 352 352 query67 16315 16383 16590 16383 query68 query69 392 303 282 282 query70 1015981 940 940 query71 352 305 300 300 query72 2998285625942594 query73 547 561 335 335 query74 9987996897869786 query75 2883278724912491 query76 23071067716 716 query77 375 406 323 323 query78 11180 11341 10682 10682 query79 1177804 609 609 query80 1439630 533 533 query81 564 280 253 253 query82 1022149 113 113 query83 342 264 247 247 query84 252 117 102 102 query85 908 489 437 437 query86 424 303 301 301 query87 3133307329732973 query88 3641269327002693 query89 427 391 346 346 query90 1962180 176 176 query91 162 155 135 135 query92 71 81 71 71 query93 963 883 503 503 query94 651 339 294 294 query95 572 403 309 309 query96 634 532 230 230 query97 2443253324252425 query98 233 225 214 214 query99 1022995 917 917 Total cold run time: 254874 ms Total hot run time: 184006 ms ``` -- 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: [email protected].
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4001728544 TPC-H: Total hot run time: 29251 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit 1b91fcbf0c7a6b98ddd1f18dc60f37c055563129, data reload: false -- Round 1 -- q1 17593 453543264326 q2 q3 10649 788 524 524 q4 4676355 253 253 q5 7545120710291029 q6 181 178 149 149 q7 791 838 678 678 q8 9289152414441444 q9 4826480847804780 q10 6797187216591659 q11 478 265 246 246 q12 689 590 500 500 q13 17793 422634103410 q14 231 230 208 208 q15 941 815 783 783 q16 729 736 668 668 q17 701 892 448 448 q18 5981543152815281 q19 1258981 664 664 q20 524 507 395 395 q21 5062202815131513 q22 379 316 293 293 Total cold run time: 97113 ms Total hot run time: 29251 ms - Round 2, with runtime_filter_mode=off - q1 4655458345944583 q2 q3 1839226817601760 q4 955 1196810 810 q5 4058441043134313 q6 202 190 140 140 q7 1792163015781578 q8 2517288125902590 q9 7632753173907390 q10 2679286124212421 q11 533 467 438 438 q12 528 589 455 455 q13 4098447935943594 q14 288 296 284 284 q15 823 793 773 773 q16 721 772 715 715 q17 1157150812611261 q18 7146696067186718 q19 1013958 968 958 q20 2127220621782178 q21 4068350435083504 q22 467 441 388 388 Total cold run time: 49298 ms Total hot run time: 46851 ms ``` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4001631888 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
github-actions[bot] commented on code in PR #61029:
URL: https://github.com/apache/doris/pull/61029#discussion_r2887130821
##
fe/fe-core/src/main/java/org/apache/doris/planner/MaterializationNode.java:
##
@@ -123,7 +167,7 @@ public String getNodeExplainString(String detailPrefix,
TExplainLevel detailLeve
}
output.append(detailPrefix).append("column_descs_lists").append(lazyColumns).append("\n");
output.append(detailPrefix).append("locations:
").append(locations).append("\n");
-output.append(detailPrefix).append("table_idxs:
").append(idxs).append("\n");
+output.append(detailPrefix).append("column_idxs_Lists:
").append(columnIdxsLists).append("\n");
output.append(detailPrefix).append("row_ids:
").append(rowIds).append("\n");
Review Comment:
Nit: Inconsistent casing — `"column_idxs_Lists: "` has a capital 'L' in
`Lists`, but the surrounding explain keys use lowercase snake_case (e.g.,
`"column_descs_lists"`, `"locations: "`, `"row_ids: "`). Should this be
`"column_idxs_lists: "` for consistency?
##
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalLazyMaterialize.java:
##
@@ -61,11 +61,44 @@ public class PhysicalLazyMaterialize extends PhysicalUn
private final List materializeInput;
private final List materializeOutput;
-// used for BE
+/**
+ * The following four fields are used by BE to perform the actual lazy
fetch.
+ * They are indexed by relation: index i corresponds to relations.get(i).
+ *
+ * Example:
+ * SQL: SELECT t1.a, t1.b, t2.c, t2.d FROM t1 JOIN t2 ON ... WHERE t1.a
> 5
+ * Assume t1.b and t2.d are lazily materialized (fetched after
filtering).
+ *
+ * materializedSlots (non-lazy, computed eagerly) = [t1.a, t2.c]
+ * Output slot order = [t1.a(0), t2.c(1), t1.b(2), t2.d(3)]
+ *
+ * rowIdList = [row_id_t1, row_id_t2]
+ * The row-id slots passed to BE to locate the
original rows.
+ *
+ * relations = [t1, t2]
+ *
+ * lazyColumns = [[Column(b)], [Column(d)]]
+ * For each relation, the Column objects to be
lazily fetched.
+ *
+* lazyBaseColumnIndices = [[colIdxOf(b) in t1], [colIdxOf(d) in t2]]
+ * For each relation, the physical column index
inside the table
Review Comment:
Nit: Missing a leading space before `*` — this line has `"*"` (4 spaces)
while all other lines in this Javadoc block use `" *"` (5 spaces). Same
issue in `MaterializationNode.java` line 107.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4001545728 /review -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
englefly commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-4001545193 run external -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-3996917251 # FE Regression Coverage Report Increment line coverage ` 100.00% (13/13)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_2a0187ed1dfe17fa9a7ecec6ab0802cdc6593fde_merge_fe/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_2a0187ed1dfe17fa9a7ecec6ab0802cdc6593fde_merge_fe/report/index.html) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-3996852437 # BE Regression && UT Coverage Report Increment line coverage `100.00% (1/1)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_2a0187ed1dfe17fa9a7ecec6ab0802cdc6593fde_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_2a0187ed1dfe17fa9a7ecec6ab0802cdc6593fde_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 71.56% (26178/36580) | | Line Coverage | 54.34% (274476/505107) | | Region Coverage | 51.68% (228404/441997) | | Branch Coverage | 53.04% (98158/185051) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
github-actions[bot] commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-3996789996 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-3996445408 # BE UT Coverage Report Increment line coverage `100.00% (1/1)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/2a0187ed1dfe17fa9a7ecec6ab0802cdc6593fde_2a0187ed1dfe17fa9a7ecec6ab0802cdc6593fde/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/2a0187ed1dfe17fa9a7ecec6ab0802cdc6593fde_2a0187ed1dfe17fa9a7ecec6ab0802cdc6593fde/report/index.html) | Category | Coverage | |---|| | Function Coverage | 52.56% (19637/37358) | | Line Coverage | 36.18% (183325/506660) | | Region Coverage | 32.49% (142257/437860) | | Branch Coverage | 33.44% (61700/184487) | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hello-stephen commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-3996391124 # FE UT Coverage Report Increment line coverage ` 84.62% (11/13)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/61029_2a0187ed1dfe17fa9a7ecec6ab0802cdc6593fde/fe_increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/61029_2a0187ed1dfe17fa9a7ecec6ab0802cdc6593fde/fe_report/index.html) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-3996162466 TPC-DS: Total hot run time: 183335 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS sf100 test result on commit 2a0187ed1dfe17fa9a7ecec6ab0802cdc6593fde, data reload: false query5 5416648 521 521 query6 323 232 199 199 query7 4221456 274 274 query8 332 241 226 226 query9 8730275827612758 query10 572 367 343 343 query11 16903 17461 17095 17095 query12 197 137 122 122 query13 1252463 346 346 query14 7872332530383038 query14_12952285328362836 query15 211 194 180 180 query16 1058497 468 468 query17 1542761 634 634 query18 3291485 355 355 query19 209 195 175 175 query20 131 125 129 125 query21 213 127 111 111 query22 4804493046744674 query23 17210 16852 16465 16465 query23_116727 16681 16641 16641 query24 7002160512171217 query24_11222124112141214 query25 523 456 451 451 query26 1246251 147 147 query27 2779467 287 287 query28 4527189318761876 query29 783 550 467 467 query30 304 250 209 209 query31 865 727 640 640 query32 80 77 70 70 query33 518 322 282 282 query34 889 901 562 562 query35 634 668 582 582 query36 10911117981 981 query37 145 95 82 82 query38 2877291428402840 query39 893 898 853 853 query39_1815 830 813 813 query40 229 147 135 135 query41 61 59 59 59 query42 110 105 110 105 query43 376 391 345 345 query44 query45 191 185 182 182 query46 885 993 597 597 query47 2120214420692069 query48 311 317 228 228 query49 621 457 371 371 query50 672 273 213 213 query51 4088407540074007 query52 109 105 96 96 query53 296 332 282 282 query54 303 268 264 264 query55 90 85 82 82 query56 311 319 328 319 query57 1371134012651265 query58 302 289 286 286 query59 2646270125732573 query60 345 372 325 325 query61 153 147 139 139 query62 628 589 525 525 query63 312 285 268 268 query64 47751256971 971 query65 query66 1357449 372 372 query67 16271 16367 16386 16367 query68 query69 384 298 272 272 query70 976 971 955 955 query71 350 306 294 294 query72 2810276325352535 query73 529 549 327 327 query74 9981991997959795 query75 2841275524812481 query76 22971039683 683 query77 372 367 320 320 query78 11238 11410 10730 10730 query79 1124799 616 616 query80 1645632 569 569 query81 570 284 252 252 query82 1034150 117 117 query83 353 271 258 258 query84 262 127 101 101 query85 1396551 501 501 query86 441 314 293 293 query87 3182312929782978 query88 3548267326722672 query89 418 366 352 352 query90 1874180 174 174 query91 162 150 132 132 query92 82 73 74 73 query93 1126884 505 505 query94 659 329 291 291 query95 573 339 375 339 query96 632 509 226 226 query97 2475249424162416 query98 230 214 214 214 query99 1031989 912 912 Total cold run time: 256230 ms Total hot run time: 183335 ms ``` -- 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: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
doris-robot commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-3996108920 TPC-H: Total hot run time: 28696 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit 2a0187ed1dfe17fa9a7ecec6ab0802cdc6593fde, data reload: false -- Round 1 -- q1 17584 451743014301 q2 q3 10650 772 517 517 q4 4677358 260 260 q5 7536118010211021 q6 175 179 147 147 q7 775 855 662 662 q8 9294146012821282 q9 4754470547114705 q10 6766186516361636 q11 441 270 242 242 q12 687 565 461 461 q13 17769 422533873387 q14 224 242 218 218 q15 923 789 791 789 q16 731 726 677 677 q17 711 855 407 407 q18 5910552652495249 q19 1250958 592 592 q20 509 488 390 390 q21 5199199614751475 q22 374 304 278 278 Total cold run time: 96939 ms Total hot run time: 28696 ms - Round 2, with runtime_filter_mode=off - q1 5791453145244524 q2 q3 1809223718031803 q4 862 1177792 792 q5 4008435444974354 q6 179 169 136 136 q7 1772161714971497 q8 2558268025082508 q9 7440736175177361 q10 2600280923592359 q11 491 428 403 403 q12 472 612 477 477 q13 4015461735543554 q14 281 301 271 271 q15 904 841 827 827 q16 716 752 740 740 q17 1407149213121312 q18 6963678365546554 q19 849 837 882 837 q20 2256219520072007 q21 4031349334343434 q22 429 418 489 418 Total cold run time: 49833 ms Total hot run time: 46168 ms ``` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [fix](lazy materialize) set uniqueId for lazy materialized slots [doris]
hubgeter commented on PR #61029: URL: https://github.com/apache/doris/pull/61029#issuecomment-3996013563 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
