Re: [PR] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4940213596 run check_coverage -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
github-actions[bot] commented on code in PR #60323:
URL: https://github.com/apache/doris/pull/60323#discussion_r3556182859
##
regression-test/suites/external_table_p0/tvf/test_brokers_tvf.groovy:
##
@@ -0,0 +1,196 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership. The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License. You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied. See the License for the
+// specific language governing permissions and limitations
+// under the License.
+
+// This suit test the `brokers` tvf
+suite("test_brokers_tvf","p0") {
+def brokerClusterA = "brokers_tvf_cluster_a"
Review Comment:
This new TVF now relies on an ADMIN/OPERATOR privilege check, but the
regression suite only exercises `brokers()` through the default privileged
session. That leaves the fixed SQL path unprotected: the unit test mocks
`BrokersTableValuedFunction` construction, but it does not cover
parser/planner/session privilege propagation for a real user. Please add a
regression case like the existing `test_show_broker_auth.groovy` /
`test_backends_auth.groovy` patterns that connects as a non-admin user,
verifies `select * from brokers()` is denied, then grants the appropriate
node/operator privilege and verifies the TVF succeeds.
--
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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
yx-keith commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4931623590 /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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4927670159 ClickBench: Total hot run time: 25.24 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools ClickBench test result on commit 11a21138b12a2ed06378c01098ac106df9a385b9, data reload: false query1 0.000.000.00 query2 0.090.050.05 query3 0.260.140.14 query4 1.610.140.14 query5 0.240.250.23 query6 1.261.041.08 query7 0.040.010.01 query8 0.050.040.04 query9 0.400.310.31 query10 0.560.540.54 query11 0.190.140.13 query12 0.180.140.14 query13 0.480.490.47 query14 1.031.021.00 query15 0.620.610.60 query16 0.310.340.33 query17 1.121.131.12 query18 0.250.220.21 query19 2.111.991.93 query20 0.020.010.01 query21 15.43 0.220.14 query22 4.850.060.05 query23 16.11 0.290.14 query24 2.930.450.32 query25 0.110.050.04 query26 0.730.220.16 query27 0.030.030.04 query28 3.560.990.56 query29 12.55 4.093.22 query30 0.270.160.15 query31 2.760.600.32 query32 3.220.600.49 query33 3.163.303.36 query34 15.69 4.253.54 query35 3.563.533.55 query36 0.550.450.43 query37 0.090.070.06 query38 0.050.050.04 query39 0.040.030.04 query40 0.180.160.16 query41 0.080.040.03 query42 0.040.030.03 query43 0.050.030.04 Total cold run time: 96.86 s Total hot run time: 25.24 s ``` -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4927629138 TPC-DS: Total hot run time: 180991 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 11a21138b12a2ed06378c01098ac106df9a385b9, data reload: false query5 4318644 489 489 query6 460 215 195 195 query7 4843613 352 352 query8 337 186 172 172 query9 8763408541014085 query10 462 381 314 314 query11 5949234921322132 query12 165 104 102 102 query13 1293643 474 474 query14 6589538250265026 query14_14347432943124312 query15 226 209 183 183 query16 1051479 488 479 query17 1152774 598 598 query18 2713478 364 364 query19 226 194 152 152 query20 111 111 106 106 query21 236 159 135 135 query22 13682 13635 13385 13385 query23 17516 16614 16267 16267 query23_116281 16357 16356 16356 query24 7712177613251325 query24_11364135113041304 query25 563 458 397 397 query26 1336361 216 216 query27 2553597 396 396 query28 4401203320302030 query29 1065635 511 511 query30 340 269 231 231 query31 1136109210081008 query32 110 65 64 64 query33 538 332 272 272 query34 11531150635 635 query35 779 800 679 679 query36 1408140611891189 query37 158 112 102 102 query38 1910167716511651 query39 935 925 891 891 query39_1876 897 884 884 query40 243 161 139 139 query41 71 67 64 64 query42 91 90 91 90 query43 335 342 288 288 query44 1423769 780 769 query45 204 185 176 176 query46 10901188762 762 query47 2305232821832183 query48 419 437 298 298 query49 577 431 323 323 query50 1083435 335 335 query51 10770 10814 10546 10546 query52 87 87 76 76 query53 272 276 200 200 query54 277 234 233 233 query55 75 72 71 71 query56 285 290 293 290 query57 1409137113191319 query58 284 265 264 264 query59 1603164314581458 query60 307 272 257 257 query61 155 153 146 146 query62 710 634 567 567 query63 252 207 213 207 query64 28131043914 914 query65 4872481948224819 query66 1743521 394 394 query67 29609 29684 29393 29393 query68 33581520975 975 query69 418 322 285 285 query70 1138996 982 982 query71 381 313 310 310 query72 3153270824602460 query73 837 765 435 435 query74 5127495447634763 query75 2622258622342234 query76 23531200844 844 query77 383 393 282 282 query78 12450 12359 11730 11730 query79 14121233793 793 query80 1242565 470 470 query81 506 325 277 277 query82 572 159 124 124 query83 387 321 300 300 query84 276 160 130 130 query85 964 604 531 531 query86 409 310 279 279 query87 1840182817451745 query88 3738287128052805 query89 459 397 361 361 query90 1828203 201 201 query91 204 195 164 164 query92 63 68 58 58 query93 15291499989 989 query94 654 360 342 342 query95 798 587 477 477 query96 1041787 379 379 query97 2694266525532553 query98 212 205 195 195 query99 1149116610131013 Total cold run time: 267097 ms Total hot run time: 180991 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
Re: [PR] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4927536407 TPC-H: Total hot run time: 29510 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit 11a21138b12a2ed06378c01098ac106df9a385b9, data reload: false -- Round 1 -- q1 17596 401340114011 q2 2020326 206 206 q3 10379 1463866 866 q4 4682477 340 340 q5 7484861 578 578 q6 186 178 137 137 q7 763 838 627 627 q8 9383168915741574 q9 5633443643804380 q10 6747178215291529 q11 504 342 311 311 q12 701 549 439 439 q13 18087 345427402740 q14 273 271 247 247 q15 q16 790 769 708 708 q17 1037974 1043974 q18 6951586955605560 q19 1225121310111011 q20 796 633 532 532 q21 5660265624382438 q22 442 360 302 302 Total cold run time: 101339 ms Total hot run time: 29510 ms - Round 2, with runtime_filter_mode=off - q1 4323425642844256 q2 282 316 214 214 q3 4600496843694369 q4 2073219413761376 q5 4422427643424276 q6 230 173 128 128 q7 1789184319381843 q8 2579218621552155 q9 8051827276847684 q10 4785474542514251 q11 562 404 401 401 q12 737 777 538 538 q13 3278354230113011 q14 308 299 277 277 q15 q16 701 739 636 636 q17 1344130813071307 q18 7963736174517361 q19 1142113710871087 q20 2202221219371937 q21 5228462244314431 q22 514 489 407 407 Total cold run time: 57113 ms Total hot run time: 51945 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4927467159 # FE Regression Coverage Report Increment line coverage ` 85.19% (69/81)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_11a21138b12a2ed06378c01098ac106df9a385b9_merge_fe/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_11a21138b12a2ed06378c01098ac106df9a385b9_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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4927362180 # BE Regression && UT Coverage Report Increment line coverage `6.25% (1/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_11a21138b12a2ed06378c01098ac106df9a385b9_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_11a21138b12a2ed06378c01098ac106df9a385b9_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 74.77% (29967/40081) | | Line Coverage | 58.83% (328689/558693) | | Region Coverage | 55.51% (275274/495931) | | Branch Coverage | 56.85% (121328/213419) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4925888977 # Cloud UT Coverage Report Increment line coverage ` ` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/11a21138b12a2ed06378c01098ac106df9a385b9_11a21138b12a2ed06378c01098ac106df9a385b9_cloud/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/11a21138b12a2ed06378c01098ac106df9a385b9_11a21138b12a2ed06378c01098ac106df9a385b9_cloud/report/index.html) | Category | Coverage | |---|| | Function Coverage | 77.39% (1896/2450) | | Line Coverage | 64.44% (34063/52862) | | Region Coverage | 64.86% (17533/27032) | | Branch Coverage | 54.03% (9396/17390) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4925463854 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4925174293 # BE UT Coverage Report Increment line coverage `0.00% (0/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/4eaf6e1332c28df4b75382e3a5d701db3bf2b3d9_4eaf6e1332c28df4b75382e3a5d701db3bf2b3d9/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/4eaf6e1332c28df4b75382e3a5d701db3bf2b3d9_4eaf6e1332c28df4b75382e3a5d701db3bf2b3d9/report/index.html) | Category | Coverage | |---|| | Function Coverage | 56.60% (23238/41059) | | Line Coverage | 40.15% (225561/561784) | | Region Coverage | 36.05% (178012/493775) | | Branch Coverage | 37.13% (79135/213138) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4924510475 # FE UT Coverage Report Increment line coverage ` 87.65% (71/81)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_4eaf6e1332c28df4b75382e3a5d701db3bf2b3d9/fe_increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_4eaf6e1332c28df4b75382e3a5d701db3bf2b3d9/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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4924122772 # Cloud UT Coverage Report Increment line coverage ` ` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/4eaf6e1332c28df4b75382e3a5d701db3bf2b3d9_4eaf6e1332c28df4b75382e3a5d701db3bf2b3d9_cloud/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/4eaf6e1332c28df4b75382e3a5d701db3bf2b3d9_4eaf6e1332c28df4b75382e3a5d701db3bf2b3d9_cloud/report/index.html) | Category | Coverage | |---|| | Function Coverage | 77.39% (1896/2450) | | Line Coverage | 64.46% (34073/52862) | | Region Coverage | 64.86% (17532/27032) | | Branch Coverage | 54.05% (9400/17390) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4923731577 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4920545118 # Cloud UT Coverage Report Increment line coverage ` ` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/a6b4855b6c43e8ef153fc00a03c73aa202df665c_a6b4855b6c43e8ef153fc00a03c73aa202df665c_cloud/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/a6b4855b6c43e8ef153fc00a03c73aa202df665c_a6b4855b6c43e8ef153fc00a03c73aa202df665c_cloud/report/index.html) | Category | Coverage | |---|| | Function Coverage | 77.39% (1896/2450) | | Line Coverage | 64.48% (34085/52862) | | Region Coverage | 64.87% (17537/27032) | | Branch Coverage | 54.04% (9397/17390) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4920392786 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4919634347 # BE Regression && UT Coverage Report Increment line coverage `6.25% (1/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_b1f5fffe1827469a4e7872749e908319b3863ee2_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_b1f5fffe1827469a4e7872749e908319b3863ee2_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 74.68% (29924/40072) | | Line Coverage | 58.77% (328201/558487) | | Region Coverage | 55.75% (276282/495575) | | Branch Coverage | 56.92% (121434/213351) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4918858887 # FE Regression Coverage Report Increment line coverage ` 85.19% (69/81)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_b1f5fffe1827469a4e7872749e908319b3863ee2_merge_fe/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_b1f5fffe1827469a4e7872749e908319b3863ee2_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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4917969115 ClickBench: Total hot run time: 25.11 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools ClickBench test result on commit b1f5fffe1827469a4e7872749e908319b3863ee2, data reload: false query1 0.010.000.01 query2 0.100.040.06 query3 0.260.140.14 query4 1.610.140.16 query5 0.250.230.24 query6 1.261.101.07 query7 0.040.010.00 query8 0.050.040.04 query9 0.390.310.33 query10 0.560.600.58 query11 0.200.140.14 query12 0.180.150.15 query13 0.480.480.48 query14 1.031.010.99 query15 0.620.590.60 query16 0.310.330.33 query17 1.101.171.13 query18 0.240.210.21 query19 2.111.971.99 query20 0.010.010.01 query21 15.45 0.220.14 query22 4.850.050.05 query23 16.17 0.330.11 query24 2.960.430.32 query25 0.110.060.04 query26 0.730.200.15 query27 0.040.040.04 query28 3.540.870.52 query29 12.48 4.173.23 query30 0.270.160.16 query31 2.770.580.32 query32 3.250.600.49 query33 3.113.353.17 query34 15.57 4.253.55 query35 3.553.523.54 query36 0.550.440.40 query37 0.090.060.06 query38 0.050.040.03 query39 0.040.030.02 query40 0.180.160.16 query41 0.090.040.03 query42 0.040.020.02 query43 0.040.030.04 Total cold run time: 96.74 s Total hot run time: 25.11 s ``` -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4917904470 TPC-DS: Total hot run time: 180169 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 b1f5fffe1827469a4e7872749e908319b3863ee2, data reload: false query5 4342628 488 488 query6 461 220 213 213 query7 4842602 319 319 query8 346 200 184 184 query9 8805411640944094 query10 476 351 313 313 query11 5919234721652165 query12 159 107 107 107 query13 1247594 428 428 query14 6276534949894989 query14_14346432643104310 query15 219 205 188 188 query16 1047482 451 451 query17 1143740 618 618 query18 2796477 358 358 query19 220 196 156 156 query20 112 111 109 109 query21 229 159 134 134 query22 13598 13652 13332 13332 query23 17431 16565 16134 16134 query23_116401 16259 16392 16259 query24 7647179813251325 query24_11323134213131313 query25 574 474 409 409 query26 1326355 209 209 query27 2614616 390 390 query28 4448202420422024 query29 1088622 473 473 query30 320 254 227 227 query31 11311099985 985 query32 103 60 61 60 query33 529 322 251 251 query34 11601124624 624 query35 760 798 667 667 query36 1369139912021202 query37 148 109 92 92 query38 1897170516571657 query39 932 917 894 894 query39_1877 875 882 875 query40 244 159 142 142 query41 66 62 63 62 query42 94 97 95 95 query43 321 320 281 281 query44 1428781 773 773 query45 202 183 186 183 query46 11171233755 755 query47 2335234722192219 query48 408 417 285 285 query49 580 424 320 320 query50 1049417 332 332 query51 10812 10843 10791 10791 query52 87 88 73 73 query53 264 284 198 198 query54 288 247 214 214 query55 82 75 68 68 query56 297 306 273 273 query57 1438141913441344 query58 286 291 236 236 query59 1563162014251425 query60 329 270 257 257 query61 152 151 145 145 query62 693 657 586 586 query63 244 203 211 203 query64 28061057914 914 query65 4820477047884770 query66 1806526 381 381 query67 29499 29508 29267 29267 query68 3177159110481048 query69 405 303 273 273 query70 1058971 955 955 query71 365 328 310 310 query72 3121271524152415 query73 844 768 442 442 query74 5119494047554755 query75 2643260122212221 query76 23371169757 757 query77 349 378 287 287 query78 12316 12403 11703 11703 query79 13621132774 774 query80 658 549 478 478 query81 462 321 280 280 query82 592 158 127 127 query83 386 329 294 294 query84 284 160 132 132 query85 928 617 513 513 query86 368 305 289 289 query87 1826182617751775 query88 3731279327602760 query89 449 399 353 353 query90 1903190 193 190 query91 196 191 158 158 query92 68 63 58 58 query93 16041530968 968 query94 558 383 345 345 query95 781 597 471 471 query96 1010831 345 345 query97 2693271525792579 query98 211 209 212 209 query99 1140119010261026 Total cold run time: 265491 ms Total hot run time: 180169 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
Re: [PR] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4917770260 TPC-H: Total hot run time: 29676 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit b1f5fffe1827469a4e7872749e908319b3863ee2, data reload: false -- Round 1 -- q1 17622 408140694069 q2 2028348 210 210 q3 10345 1421807 807 q4 4760477 342 342 q5 8272861 600 600 q6 325 170 145 145 q7 846 868 628 628 q8 10647 172016211621 q9 6096448043844384 q10 6880178215151515 q11 518 355 316 316 q12 715 553 428 428 q13 18126 338927812781 q14 267 262 247 247 q15 q16 797 793 719 719 q17 940 948 1037948 q18 6704586856435643 q19 11321209999 999 q20 760 673 578 578 q21 5585268023952395 q22 428 359 301 301 Total cold run time: 103793 ms Total hot run time: 29676 ms - Round 2, with runtime_filter_mode=off - q1 4400430043104300 q2 297 329 212 212 q3 4592501444144414 q4 2052218413841384 q5 4469430543304305 q6 229 175 131 131 q7 2533196817201720 q8 2593224722372237 q9 81767869 q10 4794474442944294 q11 574 443 436 436 q12 769 766 560 560 q13 3275357029652965 q14 292 300 283 283 q15 q16 713 742 635 635 q17 1342144713251325 q18 7995731074087310 q19 1089110110951095 q20 2210221919731973 q21 5279463443964396 q22 540 463 446 446 Total cold run time: 58213 ms Total hot run time: 52198 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4917382371 # FE UT Coverage Report Increment line coverage ` 54.32% (44/81)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_b1f5fffe1827469a4e7872749e908319b3863ee2/fe_increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_b1f5fffe1827469a4e7872749e908319b3863ee2/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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4916523445 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4910946213 # FE Regression Coverage Report Increment line coverage ` 11.04% (69/625)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_fbef370b0ff418e478187c2269a5190bebc19741_merge_fe/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_fbef370b0ff418e478187c2269a5190bebc19741_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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4910930227 # BE Regression && UT Coverage Report Increment line coverage `6.25% (1/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_fbef370b0ff418e478187c2269a5190bebc19741_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_fbef370b0ff418e478187c2269a5190bebc19741_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 74.78% (29958/40063) | | Line Coverage | 58.78% (328317/558524) | | Region Coverage | 55.59% (275395/495422) | | Branch Coverage | 56.88% (121306/213281) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4910523246 ClickBench: Total hot run time: 25.66 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools ClickBench test result on commit fbef370b0ff418e478187c2269a5190bebc19741, data reload: false query1 0.010.010.01 query2 0.140.090.09 query3 0.360.250.25 query4 1.600.230.24 query5 0.330.310.32 query6 1.170.670.68 query7 0.040.010.01 query8 0.090.080.07 query9 0.500.380.38 query10 0.580.590.56 query11 0.310.180.18 query12 0.310.180.18 query13 0.530.520.52 query14 0.930.930.94 query15 0.680.590.59 query16 0.380.390.38 query17 0.991.001.02 query18 0.310.290.30 query19 1.911.811.76 query20 0.020.020.01 query21 15.40 0.380.31 query22 4.660.130.14 query23 15.85 0.490.30 query24 2.250.660.45 query25 0.150.100.09 query26 0.760.270.21 query27 0.100.090.11 query28 3.440.940.53 query29 12.45 4.183.22 query30 0.380.260.26 query31 2.770.610.32 query32 3.230.600.47 query33 2.943.073.11 query34 15.75 4.113.40 query35 3.333.283.29 query36 0.630.540.51 query37 0.120.100.09 query38 0.070.060.06 query39 0.080.070.06 query40 0.200.180.17 query41 0.130.090.09 query42 0.080.060.07 query43 0.080.070.07 Total cold run time: 96.04 s Total hot run time: 25.66 s ``` -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4910502374 TPC-DS: Total hot run time: 178654 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 fbef370b0ff418e478187c2269a5190bebc19741, data reload: false query5 4309633 495 495 query6 449 215 201 201 query7 4863586 322 322 query8 352 192 170 170 query9 8773400940434009 query10 472 347 303 303 query11 5847235121722172 query12 163 107 102 102 query13 1281624 447 447 query14 6681528049644964 query14_14285427542584258 query15 231 198 180 180 query16 1031473 453 453 query17 1141734 581 581 query18 2707478 355 355 query19 216 193 150 150 query20 113 109 112 109 query21 239 156 137 137 query22 13731 13544 13456 13456 query23 17255 16610 16133 16133 query23_116229 16280 16211 16211 query24 7611175412891289 query24_11313131113181311 query25 579 469 401 401 query26 1349340 210 210 query27 2543620 381 381 query28 4399201719771977 query29 1103635 509 509 query30 331 258 230 230 query31 11211096990 990 query32 110 65 62 62 query33 550 324 264 264 query34 11431131671 671 query35 759 795 682 682 query36 1368138711991199 query37 155 107 99 99 query38 1870172116431643 query39 922 919 889 889 query39_1883 901 867 867 query40 301 154 141 141 query41 64 62 62 62 query42 92 91 90 90 query43 313 317 277 277 query44 1410787 773 773 query45 200 184 178 178 query46 10711178757 757 query47 2360234021952195 query48 400 395 295 295 query49 575 416 321 321 query50 1017418 349 349 query51 10669 10716 10826 10716 query52 85 85 73 73 query53 251 275 202 202 query54 305 229 224 224 query55 75 72 67 67 query56 305 301 272 272 query57 1453141913061306 query58 290 244 247 244 query59 1552160114321432 query60 295 275 248 248 query61 149 154 146 146 query62 699 649 581 581 query63 233 205 206 205 query64 2503779 616 616 query65 4872475147364736 query66 1797535 388 388 query67 29476 29327 28739 28739 query68 31041615912 912 query69 451 304 271 271 query70 1065981 976 976 query71 356 306 304 304 query72 2873259925162516 query73 812 766 431 431 query74 5109493747564756 query75 2620265022062206 query76 23221213805 805 query77 353 396 289 289 query78 12156 12355 11698 11698 query79 14951138747 747 query80 1301541 472 472 query81 529 332 286 286 query82 601 158 124 124 query83 350 309 289 289 query84 282 161 130 130 query85 938 597 523 523 query86 424 298 290 290 query87 1829181617221722 query88 3688280827702770 query89 453 406 354 354 query90 1934192 196 192 query91 198 188 168 168 query92 65 60 54 54 query93 16731543968 968 query94 716 356 328 328 query95 782 495 559 495 query96 1099771 359 359 query97 2716267525282528 query98 216 210 197 197 query99 1174114510631063 Total cold run time: 265110 ms Total hot run time: 178654 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 g
Re: [PR] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4910450318 TPC-H: Total hot run time: 29022 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit fbef370b0ff418e478187c2269a5190bebc19741, data reload: false -- Round 1 -- q1 17922 406340194019 q2 2009319 194 194 q3 10230 1401800 800 q4 4688465 337 337 q5 7487839 558 558 q6 179 164 140 140 q7 755 818 606 606 q8 9332155415841554 q9 5982434342904290 q10 6780177915281528 q11 518 340 315 315 q12 699 529 421 421 q13 18112 337127692769 q14 276 261 236 236 q15 q16 789 775 710 710 q17 22391240678 678 q18 6717573855455545 q19 1520120010091009 q20 727 655 600 600 q21 5641257024182418 q22 432 356 295 295 Total cold run time: 103034 ms Total hot run time: 29022 ms - Round 2, with runtime_filter_mode=off - q1 4348427242764272 q2 280 309 213 213 q3 4546493643694369 q4 2038213413441344 q5 4430428043134280 q6 219 172 127 127 q7 1732192517311731 q8 2436218920682068 q9 7816780277287728 q10 4759467342414241 q11 555 411 371 371 q12 741 882 588 588 q13 3246366130343034 q14 332 327 277 277 q15 q16 711 743 675 675 q17 1337130613081306 q18 8024721469596959 q19 1091105210471047 q20 2207218319281928 q21 5170450343564356 q22 510 465 425 425 Total cold run time: 56528 ms Total hot run time: 51339 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4910129635 # BE UT Coverage Report Increment line coverage `0.00% (0/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/fbef370b0ff418e478187c2269a5190bebc19741_fbef370b0ff418e478187c2269a5190bebc19741/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/fbef370b0ff418e478187c2269a5190bebc19741_fbef370b0ff418e478187c2269a5190bebc19741/report/index.html) | Category | Coverage | |---|| | Function Coverage | 56.52% (23196/41037) | | Line Coverage | 40.08% (225120/561729) | | Region Coverage | 36.05% (177838/493330) | | Branch Coverage | 37.08% (79001/213046) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4909926233 # Cloud UT Coverage Report Increment line coverage ` ` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/fbef370b0ff418e478187c2269a5190bebc19741_fbef370b0ff418e478187c2269a5190bebc19741_cloud/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/fbef370b0ff418e478187c2269a5190bebc19741_fbef370b0ff418e478187c2269a5190bebc19741_cloud/report/index.html) | Category | Coverage | |---|| | Function Coverage | 77.39% (1896/2450) | | Line Coverage | 64.52% (34104/52862) | | Region Coverage | 64.93% (17552/27032) | | Branch Coverage | 54.10% (9408/17390) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4909699748 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4907050522 ClickBench: Total hot run time: 24.86 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools ClickBench test result on commit fbef370b0ff418e478187c2269a5190bebc19741, data reload: false query1 0.000.010.00 query2 0.100.050.04 query3 0.260.130.13 query4 1.600.140.14 query5 0.240.220.22 query6 1.281.091.05 query7 0.040.040.00 query8 0.060.040.04 query9 0.380.310.31 query10 0.540.550.54 query11 0.210.140.14 query12 0.180.150.14 query13 0.480.480.49 query14 1.021.011.02 query15 0.610.590.60 query16 0.330.320.32 query17 1.101.101.06 query18 0.220.210.21 query19 1.981.912.02 query20 0.010.020.01 query21 15.44 0.230.14 query22 4.800.060.05 query23 16.13 0.320.12 query24 2.880.410.29 query25 0.120.050.06 query26 0.730.210.14 query27 0.030.040.04 query28 3.460.960.52 query29 12.60 4.033.19 query30 0.280.140.17 query31 2.780.600.31 query32 3.220.590.48 query33 3.243.153.18 query34 15.69 4.273.56 query35 3.583.563.55 query36 0.540.440.42 query37 0.090.070.06 query38 0.050.040.04 query39 0.040.040.03 query40 0.180.160.14 query41 0.080.030.03 query42 0.040.030.03 query43 0.050.040.04 Total cold run time: 96.69 s Total hot run time: 24.86 s ``` -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4907013429 TPC-DS: Total hot run time: 180167 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 fbef370b0ff418e478187c2269a5190bebc19741, data reload: false query5 4337661 498 498 query6 467 233 223 223 query7 4854635 356 356 query8 355 198 184 184 query9 8779411841284118 query10 653 346 314 314 query11 5896238821952195 query12 172 118 104 104 query13 1297652 439 439 query14 6703533350055005 query14_14337431442894289 query15 219 208 184 184 query16 1022532 487 487 query17 1153765 606 606 query18 2743497 380 380 query19 222 199 166 166 query20 115 111 108 108 query21 238 161 137 137 query22 13626 13600 13427 13427 query23 17550 16573 16187 16187 query23_116270 16393 16326 16326 query24 7424182313141314 query24_11303132013201320 query25 578 469 392 392 query26 1323371 212 212 query27 2524648 359 359 query28 4426203520122012 query29 649 506 506 query30 343 265 235 235 query31 1129109910031003 query32 113 66 63 63 query33 536 345 271 271 query34 12021212622 622 query35 777 792 666 666 query36 1385135811891189 query37 152 108 93 93 query38 1863170416331633 query39 927 915 904 904 query39_1886 888 888 888 query40 238 161 141 141 query41 67 63 63 63 query42 93 93 90 90 query43 323 328 283 283 query44 1516792 772 772 query45 207 187 176 176 query46 11121255733 733 query47 2391237222002200 query48 375 430 307 307 query49 592 421 334 334 query50 1074436 326 326 query51 10831 10711 10387 10387 query52 87 89 75 75 query53 265 292 204 204 query54 278 256 226 226 query55 78 72 66 66 query56 283 282 310 282 query57 1450140213191319 query58 280 250 257 250 query59 1592162614911491 query60 298 278 250 250 query61 149 176 150 150 query62 712 652 583 583 query63 249 215 216 215 query64 2500768 607 607 query65 4908476447754764 query66 1827520 386 386 query67 29565 29553 29431 29431 query68 3170152510361036 query69 410 309 279 279 query70 1109972 965 965 query71 360 311 302 302 query72 2963264023472347 query73 850 817 456 456 query74 5138492547664766 query75 2683260922472247 query76 23291285789 789 query77 371 401 282 282 query78 12440 12457 11757 11757 query79 14101225788 788 query80 1285559 459 459 query81 526 331 283 283 query82 584 166 122 122 query83 363 335 294 294 query84 286 160 132 132 query85 1002632 501 501 query86 444 292 283 283 query87 1827182918051805 query88 3783280427712771 query89 461 408 364 364 query90 1945205 197 197 query91 200 192 162 162 query92 67 61 58 58 query93 17031531947 947 query94 724 341 323 323 query95 786 583 483 483 query96 1094820 336 336 query97 2692270725972597 query98 219 210 200 200 query99 1170116410111011 Total cold run time: 267201 ms Total hot run time: 180167 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
Re: [PR] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4906929267 TPC-H: Total hot run time: 29418 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit fbef370b0ff418e478187c2269a5190bebc19741, data reload: false -- Round 1 -- q1 17889 413040394039 q2 2032326 207 207 q3 10358 1459855 855 q4 4760477 338 338 q5 8206870 587 587 q6 328 171 138 138 q7 857 866 634 634 q8 10572 163216181618 q9 5796439944124399 q10 6815177615331533 q11 504 345 320 320 q12 707 561 449 449 q13 18126 335727762776 q14 269 266 249 249 q15 q16 797 782 715 715 q17 954 844 1015844 q18 6886577155395539 q19 11581261967 967 q20 752 692 564 564 q21 5430262723462346 q22 442 360 301 301 Total cold run time: 103638 ms Total hot run time: 29418 ms - Round 2, with runtime_filter_mode=off - q1 4584433043114311 q2 286 322 218 218 q3 4575499843984398 q4 2077216113611361 q5 4452427943314279 q6 227 183 132 132 q7 2178200717001700 q8 2467223022342230 q9 8196781579267815 q10 4757471742874287 q11 574 436 382 382 q12 789 813 545 545 q13 3302355128952895 q14 298 285 276 276 q15 q16 727 736 645 645 q17 1417132414461324 q18 8027726272197219 q19 1113104310871043 q20 2216218319361936 q21 5237456644094409 q22 513 455 415 415 Total cold run time: 58012 ms Total hot run time: 51820 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4905244489 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
yx-keith commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4905208480 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
yx-keith commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4905209336 /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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4905195313 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
github-actions[bot] commented on code in PR #60323: URL: https://github.com/apache/doris/pull/60323#discussion_r3536513693 ## contrib/apache-orc: ## @@ -1 +0,0 @@ -Subproject commit be0f1b73a7aeb78824a03e0dcb692c50a176d513 Review Comment: This PR deletes the gitlink for `contrib/apache-orc` and also deletes `contrib/datasketches-cpp`, but `.gitmodules` still declares both submodules. That leaves checkouts inconsistent: `git submodule update --init --recursive` will not populate these dependency directories from this commit even though the repository still advertises them, and BE builds/tests that expect the ORC or DataSketches sources can fail. These removals are unrelated to the brokers TVF feature, so please restore the gitlinks or move the submodule cleanup into a dedicated change that also updates all build references and `.gitmodules`. -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
yx-keith commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4903715800 /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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4903439259 # Cloud UT Coverage Report Increment line coverage ` ` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/fa083a31363cb2614b0cb54f84f7203f2a4af2f3_fa083a31363cb2614b0cb54f84f7203f2a4af2f3_cloud/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/fa083a31363cb2614b0cb54f84f7203f2a4af2f3_fa083a31363cb2614b0cb54f84f7203f2a4af2f3_cloud/report/index.html) | Category | Coverage | |---|| | Function Coverage | 77.39% (1896/2450) | | Line Coverage | 64.48% (34086/52862) | | Region Coverage | 64.90% (17543/27032) | | Branch Coverage | 54.09% (9407/17390) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4903175971 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4900703108 ClickBench: Total hot run time: 25.68 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools ClickBench test result on commit 7fe7a59f6e37c3c5fc84513302f18ffaf7bb1aa8, data reload: false query1 0.010.010.00 query2 0.140.080.08 query3 0.390.250.25 query4 1.610.240.27 query5 0.320.320.32 query6 1.160.670.67 query7 0.040.010.01 query8 0.090.070.08 query9 0.530.380.38 query10 0.570.590.59 query11 0.330.190.18 query12 0.310.180.18 query13 0.530.520.52 query14 0.930.950.94 query15 0.680.600.59 query16 0.390.400.39 query17 1.021.031.03 query18 0.310.300.30 query19 1.931.841.79 query20 0.020.010.02 query21 15.40 0.390.31 query22 4.850.130.14 query23 15.84 0.500.29 query24 2.480.610.44 query25 0.160.110.11 query26 0.770.270.22 query27 0.110.100.09 query28 3.480.890.52 query29 12.51 4.193.27 query30 0.380.260.27 query31 2.760.630.33 query32 3.220.590.47 query33 2.923.072.92 query34 15.80 4.073.37 query35 3.353.273.28 query36 0.670.530.52 query37 0.130.100.10 query38 0.080.070.07 query39 0.070.060.07 query40 0.200.180.17 query41 0.130.090.09 query42 0.090.060.06 query43 0.080.070.07 Total cold run time: 96.79 s Total hot run time: 25.68 s ``` -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4900589375 TPC-H: Total hot run time: 35684 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit 7fe7a59f6e37c3c5fc84513302f18ffaf7bb1aa8, data reload: false -- Round 1 -- q1 17613 395339273927 q2 1998358 196 196 q3 10283 1470865 865 q4 4688546 412 412 q5 7425796 481 481 q6 194 189 145 145 q7 810 855 573 573 q8 10112 204918351835 q9 7656640664106406 q10 8845419337123712 q11 501 372 338 338 q12 753 621 464 464 q13 17853 440237073707 q14 412 401 379 379 q15 q16 511 546 426 426 q17 10761110719 719 q18 6990573155845584 q19 1204131311031103 q20 1078913 791 791 q21 6127354333123312 q22 463 357 309 309 Total cold run time: 106592 ms Total hot run time: 35684 ms - Round 2, with runtime_filter_mode=off - q1 4317416341674163 q2 351 385 262 262 q3 2251276422072207 q4 2067217913731373 q5 4543447444704470 q6 232 183 132 132 q7 7706727271037103 q8 2579245124662451 q9 28661 28392 28472 28392 q10 4903507447194719 q11 1145892 854 854 q12 719 786 555 555 q13 4059444137013701 q14 382 395 349 349 q15 q16 490 512 437 437 q17 2846282227842784 q18 7687700470567004 q19 1100105810811058 q20 2279229020022002 q21 11255 10458 10351 10351 q22 527 525 431 431 Total cold run time: 90099 ms Total hot run time: 84798 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4900152020 # BE UT Coverage Report Increment line coverage `0.00% (0/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/7fe7a59f6e37c3c5fc84513302f18ffaf7bb1aa8_7fe7a59f6e37c3c5fc84513302f18ffaf7bb1aa8/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/7fe7a59f6e37c3c5fc84513302f18ffaf7bb1aa8_7fe7a59f6e37c3c5fc84513302f18ffaf7bb1aa8/report/index.html) | Category | Coverage | |---|| | Function Coverage | 56.52% (23160/40974) | | Line Coverage | 40.08% (224588/560327) | | Region Coverage | 36.07% (177415/491858) | | Branch Coverage | 37.11% (78844/212479) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
github-actions[bot] commented on code in PR #60323:
URL: https://github.com/apache/doris/pull/60323#discussion_r3533100701
##
regression-test/suites/external_table_p0/tvf/test_brokers_tvf.groovy:
##
@@ -0,0 +1,156 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership. The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License. You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied. See the License for the
+// specific language governing permissions and limitations
+// under the License.
+
+// This suit test the `brokers` tvf
+suite("test_brokers_tvf","p0") {
+def brokerClusterName = "brokers_tvf_test_cluster"
+try {
+def address1 = "172.18.0.1"
+def address2 = "172.18.0.2"
+def address3 = "172.18.0.3"
+def port = 18310
+def columns = new ArrayList()
+def columnMeta = sql """describe function brokers();"""
+def size = columnMeta.size()
+for (i in 0..
Re: [PR] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
yx-keith commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4899113618 /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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
yx-keith commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4899112859 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4899088578 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4898393488 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on code in PR #60323:
URL: https://github.com/apache/doris/pull/60323#discussion_r3532430375
##
fe/fe-core/src/main/java/org/apache/doris/tablefunction/BrokersTableValuedFunction.java:
##
@@ -0,0 +1,127 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership. The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License. You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied. See the License for the
+// specific language governing permissions and limitations
+// under the License.
+
+package org.apache.doris.tablefunction;
+
+import org.apache.doris.catalog.Column;
+import org.apache.doris.catalog.Env;
+import org.apache.doris.catalog.InfoSchemaDb;
+import org.apache.doris.catalog.ScalarType;
+import org.apache.doris.common.AnalysisException;
+import org.apache.doris.common.ErrorCode;
+import org.apache.doris.datasource.InternalCatalog;
+import org.apache.doris.mysql.privilege.PrivPredicate;
+import org.apache.doris.qe.ConnectContext;
+import org.apache.doris.thrift.TBrokersMetadataParams;
+import org.apache.doris.thrift.TMetaScanRange;
+import org.apache.doris.thrift.TMetadataType;
+
+import com.google.common.collect.ImmutableList;
+import com.google.common.collect.ImmutableMap;
+import org.apache.commons.lang3.StringUtils;
+
+import java.util.List;
+import java.util.Map;
+
+/**
+ * The Implement of table valued function
+ * brokers().
+ */
+public class BrokersTableValuedFunction extends MetadataTableValuedFunction {
+
+public static final String NAME = "brokers";
+private static final String CLUSTER_NAME = "cluster_name";
+private static final ImmutableList SCHEMA = ImmutableList.of(
+new Column("Name", ScalarType.createStringType()),
+new Column("Host", ScalarType.createStringType()),
+new Column("Port", ScalarType.createStringType()),
+new Column("Alive", ScalarType.createStringType()),
+new Column("LastStartTime", ScalarType.createStringType()),
+new Column("LastUpdateTime", ScalarType.createStringType()),
+new Column("ErrMsg", ScalarType.createStringType())
+);
+private static final ImmutableMap COLUMN_TO_INDEX;
+
+static {
+ImmutableMap.Builder builder = new
ImmutableMap.Builder();
+for (int i = 0; i < SCHEMA.size(); i++) {
+builder.put(SCHEMA.get(i).getName().toLowerCase(), i);
+}
+COLUMN_TO_INDEX = builder.build();
+}
+
+private final String clusterName;
+
+/**
+ * constructor for BrokersTableValuedFunction
+ */
+public BrokersTableValuedFunction(Map params) throws
AnalysisException {
+if (params == null) {
+throw new AnalysisException("params cannot be null");
+}
+String originalClusterName = params.get(CLUSTER_NAME);
Review Comment:
I fixed it already.
--
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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on code in PR #60323:
URL: https://github.com/apache/doris/pull/60323#discussion_r3532410354
##
gensrc/thrift/Types.thrift:
##
@@ -764,6 +764,7 @@ enum TMetadataType {
PAIMON = 12,
PARQUET = 13,
STREAMS = 14,
+ BROKERS = 15,
Review Comment:
Thanks for the suggestion. However, currently none of the other TVFs handle
upgrade compatibility either. To keep consistency with existing
implementations, we won’t address it separately in this PR for now. If there is
a unified upgrade compatibility solution in the future, we can support it then.
--
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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on code in PR #60323:
URL: https://github.com/apache/doris/pull/60323#discussion_r3532404460
##
fe/fe-core/src/main/java/org/apache/doris/tablefunction/BrokersTableValuedFunction.java:
##
@@ -0,0 +1,127 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership. The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License. You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied. See the License for the
+// specific language governing permissions and limitations
+// under the License.
+
+package org.apache.doris.tablefunction;
+
+import org.apache.doris.catalog.Column;
+import org.apache.doris.catalog.Env;
+import org.apache.doris.catalog.InfoSchemaDb;
+import org.apache.doris.catalog.ScalarType;
+import org.apache.doris.common.AnalysisException;
+import org.apache.doris.common.ErrorCode;
+import org.apache.doris.datasource.InternalCatalog;
+import org.apache.doris.mysql.privilege.PrivPredicate;
+import org.apache.doris.qe.ConnectContext;
+import org.apache.doris.thrift.TBrokersMetadataParams;
+import org.apache.doris.thrift.TMetaScanRange;
+import org.apache.doris.thrift.TMetadataType;
+
+import com.google.common.collect.ImmutableList;
+import com.google.common.collect.ImmutableMap;
+import org.apache.commons.lang3.StringUtils;
+
+import java.util.List;
+import java.util.Map;
+
+/**
+ * The Implement of table valued function
+ * brokers().
+ */
+public class BrokersTableValuedFunction extends MetadataTableValuedFunction {
+
+public static final String NAME = "brokers";
+private static final String CLUSTER_NAME = "cluster_name";
+private static final ImmutableList SCHEMA = ImmutableList.of(
+new Column("Name", ScalarType.createStringType()),
+new Column("Host", ScalarType.createStringType()),
+new Column("Port", ScalarType.createStringType()),
+new Column("Alive", ScalarType.createStringType()),
+new Column("LastStartTime", ScalarType.createStringType()),
+new Column("LastUpdateTime", ScalarType.createStringType()),
+new Column("ErrMsg", ScalarType.createStringType())
+);
+private static final ImmutableMap COLUMN_TO_INDEX;
+
+static {
+ImmutableMap.Builder builder = new
ImmutableMap.Builder();
+for (int i = 0; i < SCHEMA.size(); i++) {
+builder.put(SCHEMA.get(i).getName().toLowerCase(), i);
+}
+COLUMN_TO_INDEX = builder.build();
+}
+
+private final String clusterName;
+
+/**
+ * constructor for BrokersTableValuedFunction
+ */
+public BrokersTableValuedFunction(Map params) throws
AnalysisException {
+if (params == null) {
+throw new AnalysisException("params cannot be null");
+}
+String originalClusterName = params.get(CLUSTER_NAME);
+if (originalClusterName != null) {
+if (StringUtils.isBlank(originalClusterName)) {
+throw new AnalysisException("Invalid brokers param value: " +
originalClusterName);
+}
+clusterName = originalClusterName.toLowerCase();
+} else {
+clusterName = null;
+}
+for (String key : params.keySet()) {
+if (key == null || !CLUSTER_NAME.equals(key.toLowerCase())) {
+throw new AnalysisException("'" + key + "' is invalid
property,"
++ " only support property [" + CLUSTER_NAME + "]");
+}
+}
+if
(!Env.getCurrentEnv().getAccessManager().checkDbPriv(ConnectContext.get(),
Review Comment:
I fixed it already.
##
fe/fe-core/src/main/java/org/apache/doris/tablefunction/BrokersTableValuedFunction.java:
##
@@ -0,0 +1,127 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership. The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License. You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied. See the Licens
Re: [PR] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
github-actions[bot] commented on code in PR #60323:
URL: https://github.com/apache/doris/pull/60323#discussion_r3526269317
##
fe/fe-core/src/main/java/org/apache/doris/tablefunction/BrokersTableValuedFunction.java:
##
@@ -0,0 +1,127 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership. The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License. You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied. See the License for the
+// specific language governing permissions and limitations
+// under the License.
+
+package org.apache.doris.tablefunction;
+
+import org.apache.doris.catalog.Column;
+import org.apache.doris.catalog.Env;
+import org.apache.doris.catalog.InfoSchemaDb;
+import org.apache.doris.catalog.ScalarType;
+import org.apache.doris.common.AnalysisException;
+import org.apache.doris.common.ErrorCode;
+import org.apache.doris.datasource.InternalCatalog;
+import org.apache.doris.mysql.privilege.PrivPredicate;
+import org.apache.doris.qe.ConnectContext;
+import org.apache.doris.thrift.TBrokersMetadataParams;
+import org.apache.doris.thrift.TMetaScanRange;
+import org.apache.doris.thrift.TMetadataType;
+
+import com.google.common.collect.ImmutableList;
+import com.google.common.collect.ImmutableMap;
+import org.apache.commons.lang3.StringUtils;
+
+import java.util.List;
+import java.util.Map;
+
+/**
+ * The Implement of table valued function
+ * brokers().
+ */
+public class BrokersTableValuedFunction extends MetadataTableValuedFunction {
+
+public static final String NAME = "brokers";
+private static final String CLUSTER_NAME = "cluster_name";
+private static final ImmutableList SCHEMA = ImmutableList.of(
+new Column("Name", ScalarType.createStringType()),
+new Column("Host", ScalarType.createStringType()),
+new Column("Port", ScalarType.createStringType()),
+new Column("Alive", ScalarType.createStringType()),
+new Column("LastStartTime", ScalarType.createStringType()),
+new Column("LastUpdateTime", ScalarType.createStringType()),
+new Column("ErrMsg", ScalarType.createStringType())
+);
+private static final ImmutableMap COLUMN_TO_INDEX;
+
+static {
+ImmutableMap.Builder builder = new
ImmutableMap.Builder();
+for (int i = 0; i < SCHEMA.size(); i++) {
+builder.put(SCHEMA.get(i).getName().toLowerCase(), i);
+}
+COLUMN_TO_INDEX = builder.build();
+}
+
+private final String clusterName;
+
+/**
+ * constructor for BrokersTableValuedFunction
+ */
+public BrokersTableValuedFunction(Map params) throws
AnalysisException {
+if (params == null) {
+throw new AnalysisException("params cannot be null");
+}
+String originalClusterName = params.get(CLUSTER_NAME);
+if (originalClusterName != null) {
+if (StringUtils.isBlank(originalClusterName)) {
+throw new AnalysisException("Invalid brokers param value: " +
originalClusterName);
+}
+clusterName = originalClusterName.toLowerCase();
+} else {
+clusterName = null;
+}
+for (String key : params.keySet()) {
+if (key == null || !CLUSTER_NAME.equals(key.toLowerCase())) {
+throw new AnalysisException("'" + key + "' is invalid
property,"
++ " only support property [" + CLUSTER_NAME + "]");
+}
+}
+if
(!Env.getCurrentEnv().getAccessManager().checkDbPriv(ConnectContext.get(),
Review Comment:
This privilege check is weaker than the existing broker metadata surfaces.
`SHOW BROKER` gates the same `BrokerMgr.getBrokersInfo()` data behind global
ADMIN or OPERATOR/NODE, and `SHOW PROC '/brokers'` requires ADMIN_OR_NODE, but
this TVF only requires SELECT on `internal.information_schema`. New users get
that information_schema SELECT in their default role, so `select * from
brokers()` would expose broker names, hosts, ports, liveness, and errors to
users who cannot run `SHOW BROKER`. Please align the TVF auth with the existing
broker command/proc privilege before returning these rows.
##
fe/fe-core/src/main/java/org/apache/doris/tablefunction/BrokersTableValuedFunction.java:
##
@@ -0,0 +1,127 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+//
Re: [PR] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
yx-keith commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4888762626 /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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4888442295 # FE Regression Coverage Report Increment line coverage ` 82.89% (63/76)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_7baf9654dd000e3366a12434e50d91018b051464_merge_fe/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_7baf9654dd000e3366a12434e50d91018b051464_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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4888416033 # BE Regression && UT Coverage Report Increment line coverage `6.25% (1/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_7baf9654dd000e3366a12434e50d91018b051464_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_7baf9654dd000e3366a12434e50d91018b051464_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 74.66% (29742/39838) | | Line Coverage | 58.64% (325873/555729) | | Region Coverage | 55.49% (273666/493192) | | Branch Coverage | 56.70% (120406/212339) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4887961691 ClickBench: Total hot run time: 25.98 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools ClickBench test result on commit 7baf9654dd000e3366a12434e50d91018b051464, data reload: false query1 0.000.010.01 query2 0.150.090.09 query3 0.380.240.24 query4 1.610.240.24 query5 0.360.310.31 query6 1.150.670.67 query7 0.040.010.01 query8 0.100.080.07 query9 0.530.380.39 query10 0.590.600.58 query11 0.310.180.20 query12 0.330.190.18 query13 0.520.540.53 query14 0.930.920.91 query15 0.680.590.60 query16 0.380.390.39 query17 1.031.030.99 query18 0.300.300.29 query19 1.921.851.81 query20 0.030.010.02 query21 15.40 0.390.33 query22 4.730.140.13 query23 15.81 0.510.30 query24 2.280.610.49 query25 0.160.110.10 query26 0.780.270.22 query27 0.100.100.09 query28 3.660.900.56 query29 12.49 4.493.54 query30 0.400.270.26 query31 2.760.610.33 query32 3.230.600.47 query33 2.932.982.98 query34 15.88 4.113.35 query35 3.303.253.28 query36 0.660.530.51 query37 0.120.090.09 query38 0.080.060.06 query39 0.070.060.06 query40 0.190.180.17 query41 0.260.080.08 query42 0.090.060.06 query43 0.080.070.07 Total cold run time: 96.8 s Total hot run time: 25.98 s ``` -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4887947700 TPC-DS: Total hot run time: 173353 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 7baf9654dd000e3366a12434e50d91018b051464, data reload: false query5 4322633 481 481 query6 483 225 203 203 query7 4842585 341 341 query8 350 188 178 178 query9 8750400741034007 query10 495 353 283 283 query11 5915237321682168 query12 155 105 103 103 query13 1261590 412 412 query14 6651530650025002 query14_14333432343354323 query15 206 198 188 188 query16 991 451 436 436 query17 1101685 569 569 query18 2728483 347 347 query19 200 183 144 144 query20 115 111 107 107 query21 232 155 134 134 query22 13607 13586 13337 13337 query23 17417 16488 16153 16153 query23_116319 16168 16210 16168 query24 7414170613051305 query24_11314131512951295 query25 533 448 369 369 query26 1312346 216 216 query27 2690588 361 361 query28 4395197120061971 query29 1085599 510 510 query30 331 250 229 229 query31 11391116987 987 query32 113 65 71 65 query33 554 326 267 267 query34 12011185642 642 query35 781 788 695 695 query36 1415140912271227 query37 170 126 93 93 query38 1874170016621662 query39 926 914 910 910 query39_1875 892 896 892 query40 246 167 141 141 query41 72 71 69 69 query42 95 96 93 93 query43 328 324 282 282 query44 1455793 770 770 query45 207 189 182 182 query46 10651192747 747 query47 2415241222562256 query48 405 405 310 310 query49 594 447 319 319 query50 1029419 333 333 query51 4428442143614361 query52 85 84 76 76 query53 274 277 208 208 query54 292 246 251 246 query55 77 73 80 73 query56 318 303 287 287 query57 1430140913131313 query58 285 279 271 271 query59 1555161014431443 query60 315 281 266 266 query61 179 172 173 172 query62 703 651 589 589 query63 251 212 208 208 query64 2572819 649 649 query65 4879479448014794 query66 1762518 391 391 query67 29679 29397 29383 29383 query68 31631585981 981 query69 418 300 267 267 query70 1077990 958 958 query71 344 341 293 293 query72 2976260623792379 query73 824 821 441 441 query74 5149499047344734 query75 2600258522132213 query76 22771186761 761 query77 357 377 288 288 query78 12311 12482 11907 11907 query79 14431136760 760 query80 1314543 461 461 query81 546 327 284 284 query82 653 160 123 123 query83 359 318 290 290 query84 283 170 130 130 query85 995 609 507 507 query86 442 290 287 287 query87 1830183217411741 query88 3757281827462746 query89 457 414 360 360 query90 1952204 195 195 query91 204 186 163 163 query92 67 60 59 59 query93 16611631908 908 query94 783 381 305 305 query95 778 594 465 465 query96 1110759 352 352 query97 2661266625672567 query98 222 206 202 202 query99 1180113510171017 Total cold run time: 260041 ms Total hot run time: 173353 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
Re: [PR] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4887921061 # Cloud UT Coverage Report Increment line coverage ` ` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/7baf9654dd000e3366a12434e50d91018b051464_7baf9654dd000e3366a12434e50d91018b051464_cloud/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/7baf9654dd000e3366a12434e50d91018b051464_7baf9654dd000e3366a12434e50d91018b051464_cloud/report/index.html) | Category | Coverage | |---|| | Function Coverage | 77.39% (1896/2450) | | Line Coverage | 64.45% (34072/52862) | | Region Coverage | 64.89% (17540/27032) | | Branch Coverage | 54.08% (9405/17390) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4887917666 TPC-H: Total hot run time: 29491 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit 7baf9654dd000e3366a12434e50d91018b051464, data reload: false -- Round 1 -- q1 17585 401840344018 q2 2036319 196 196 q3 10276 1395813 813 q4 4690466 337 337 q5 7568845 576 576 q6 180 169 139 139 q7 780 853 622 622 q8 9346155115731551 q9 5629447144274427 q10 6731177515331533 q11 504 347 312 312 q12 744 560 429 429 q13 18077 340727132713 q14 262 258 231 231 q15 q16 783 784 710 710 q17 10641056933 933 q18 6785588456205620 q19 1283127010471047 q20 764 619 532 532 q21 5819266524432443 q22 438 371 309 309 Total cold run time: 101344 ms Total hot run time: 29491 ms - Round 2, with runtime_filter_mode=off - q1 4421432342884288 q2 298 321 214 214 q3 4562505243504350 q4 2071215313871387 q5 4468438243394339 q6 231 181 126 126 q7 1734190317371737 q8 2658229022552255 q9 7831781978467819 q10 4702465942204220 q11 604 463 404 404 q12 754 752 546 546 q13 3479357129272927 q14 334 310 272 272 q15 q16 746 745 689 689 q17 1398138813621362 q18 8229736069376937 q19 1107107910841079 q20 2225220019591959 q21 5311472245224522 q22 529 490 422 422 Total cold run time: 57692 ms Total hot run time: 51854 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4887799720 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4885661738 # FE Regression Coverage Report Increment line coverage ` 77.42% (48/62)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_76db63e11f563c29c180f75cf7ad470ad147a453_merge_fe/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_76db63e11f563c29c180f75cf7ad470ad147a453_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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4885637570 # BE Regression && UT Coverage Report Increment line coverage `0.00% (0/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_76db63e11f563c29c180f75cf7ad470ad147a453_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_76db63e11f563c29c180f75cf7ad470ad147a453_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 73.00% (29082/39838) | | Line Coverage | 56.60% (314538/555729) | | Region Coverage | 53.26% (262666/493192) | | Branch Coverage | 54.21% (115117/212339) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4885338406 # FE UT Coverage Report Increment line coverage ` 53.23% (33/62)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_76db63e11f563c29c180f75cf7ad470ad147a453/fe_increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_76db63e11f563c29c180f75cf7ad470ad147a453/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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4885292152 ClickBench: Total hot run time: 25.74 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools ClickBench test result on commit 76db63e11f563c29c180f75cf7ad470ad147a453, data reload: false query1 0.010.010.01 query2 0.150.080.11 query3 0.440.240.24 query4 1.610.250.26 query5 0.330.310.32 query6 1.160.670.67 query7 0.030.010.00 query8 0.090.070.07 query9 0.550.380.38 query10 0.570.580.57 query11 0.320.170.18 query12 0.320.190.19 query13 0.510.530.51 query14 0.910.910.92 query15 0.670.580.58 query16 0.390.400.38 query17 0.980.961.00 query18 0.300.290.29 query19 1.891.771.80 query20 0.030.010.01 query21 15.39 0.400.32 query22 4.630.140.13 query23 15.85 0.500.30 query24 2.450.580.43 query25 0.150.110.11 query26 0.770.270.21 query27 0.100.100.10 query28 3.600.900.54 query29 12.46 4.433.50 query30 0.390.260.26 query31 2.760.640.34 query32 3.240.610.48 query33 2.963.043.00 query34 15.71 4.093.36 query35 3.283.253.25 query36 0.630.530.50 query37 0.130.090.10 query38 0.080.060.07 query39 0.080.070.06 query40 0.210.180.17 query41 0.130.080.07 query42 0.080.050.06 query43 0.080.070.06 Total cold run time: 96.42 s Total hot run time: 25.74 s ``` -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4885280823 TPC-DS: Total hot run time: 173292 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 76db63e11f563c29c180f75cf7ad470ad147a453, data reload: false query5 4316625 488 488 query6 449 229 198 198 query7 4897588 332 332 query8 330 183 169 169 query9 8817398440163984 query10 470 347 327 327 query11 5856234921722172 query12 154 100 98 98 query13 1302578 412 412 query14 6262530349624962 query14_14291428742944287 query15 214 199 179 179 query16 1009458 424 424 query17 1146698 554 554 query18 2459461 331 331 query19 201 187 142 142 query20 110 111 107 107 query21 241 159 131 131 query22 13558 13635 13442 13442 query23 17222 16492 16285 16285 query23_116299 16196 16258 16196 query24 7602176112911291 query24_11302130312771277 query25 547 466 355 355 query26 1336341 216 216 query27 2609575 371 371 query28 4442201620202016 query29 1077626 504 504 query30 337 263 235 235 query31 11131096988 988 query32 117 64 63 63 query33 556 325 255 255 query34 12061120676 676 query35 768 789 658 658 query36 1403140012531253 query37 170 110 92 92 query38 1872170016541654 query39 932 912 882 882 query39_1874 879 892 879 query40 238 157 142 142 query41 77 61 61 61 query42 91 90 90 90 query43 328 317 280 280 query44 1405771 749 749 query45 199 191 179 179 query46 10591201766 766 query47 2328236622112211 query48 405 429 303 303 query49 592 418 314 314 query50 1067424 325 325 query51 4371433942534253 query52 85 84 74 74 query53 261 275 199 199 query54 278 227 208 208 query55 77 77 66 66 query56 291 304 294 294 query57 1496141912931293 query58 281 257 253 253 query59 1582163414441444 query60 324 271 251 251 query61 154 158 152 152 query62 685 649 593 593 query63 243 205 212 205 query64 2581781 602 602 query65 4873479148224791 query66 1818518 409 409 query67 29581 29482 29368 29368 query68 32411582960 960 query69 429 318 283 283 query70 1104961 968 961 query71 365 369 305 305 query72 3003261924302430 query73 854 806 445 445 query74 5104495248014801 query75 2592257122262226 query76 23451190794 794 query77 332 375 274 274 query78 12408 12374 11847 11847 query79 12161118774 774 query80 593 535 453 453 query81 451 319 279 279 query82 238 158 129 129 query83 318 338 302 302 query84 279 159 128 128 query85 925 603 511 511 query86 342 284 269 269 query87 1819182417441744 query88 3717282028472820 query89 438 412 360 360 query90 2212195 195 195 query91 202 189 156 156 query92 64 61 57 57 query93 15381442936 936 query94 541 336 309 309 query95 831 587 507 507 query96 1140805 327 327 query97 2688268925612561 query98 226 207 197 197 query99 1176115010331033 Total cold run time: 257780 ms Total hot run time: 173292 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
Re: [PR] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4885253722 TPC-H: Total hot run time: 29481 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit 76db63e11f563c29c180f75cf7ad470ad147a453, data reload: false -- Round 1 -- q1 17459 406342244063 q2 2023317 200 200 q3 10221 1373816 816 q4 4669470 337 337 q5 7555850 581 581 q6 178 174 142 142 q7 763 840 620 620 q8 9368155815261526 q9 5626446244324432 q10 6756178115271527 q11 502 353 322 322 q12 725 591 439 439 q13 18105 336527212721 q14 280 260 245 245 q15 q16 798 775 712 712 q17 12551036793 793 q18 6774586255125512 q19 1400127310801080 q20 790 626 597 597 q21 5682272625172517 q22 432 355 299 299 Total cold run time: 101361 ms Total hot run time: 29481 ms - Round 2, with runtime_filter_mode=off - q1 4362446642354235 q2 288 323 213 213 q3 4585496244064406 q4 2037214413471347 q5 4424430243324302 q6 230 176 125 125 q7 1708187417961796 q8 2508218821172117 q9 7881791977387738 q10 4739470242434243 q11 574 428 388 388 q12 735 765 541 541 q13 3373356229032903 q14 312 324 270 270 q15 q16 715 773 651 651 q17 1357135113221322 q18 7983754370167016 q19 1113110110971097 q20 2212219819281928 q21 5248461644094409 q22 512 449 397 397 Total cold run time: 56896 ms Total hot run time: 51444 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4885141427 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4883420045 # FE Regression Coverage Report Increment line coverage ` 77.42% (48/62)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_2d3abcff31d896c46cdb1de7cbcbc67f6af0765f_merge_fe/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_2d3abcff31d896c46cdb1de7cbcbc67f6af0765f_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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4883380399 # BE Regression && UT Coverage Report Increment line coverage `6.25% (1/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_2d3abcff31d896c46cdb1de7cbcbc67f6af0765f_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_2d3abcff31d896c46cdb1de7cbcbc67f6af0765f_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 74.62% (29727/39838) | | Line Coverage | 58.64% (325868/555729) | | Region Coverage | 55.45% (273482/493192) | | Branch Coverage | 56.70% (120386/212339) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4883062905 ClickBench: Total hot run time: 25.93 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools ClickBench test result on commit 2d3abcff31d896c46cdb1de7cbcbc67f6af0765f, data reload: false query1 0.010.010.01 query2 0.150.090.11 query3 0.410.240.24 query4 1.600.250.25 query5 0.330.320.31 query6 1.170.670.67 query7 0.040.010.00 query8 0.090.080.07 query9 0.530.380.38 query10 0.580.590.58 query11 0.300.180.18 query12 0.340.190.18 query13 0.540.530.54 query14 0.930.920.92 query15 0.710.600.61 query16 0.390.390.40 query17 1.001.010.98 query18 0.310.290.29 query19 1.921.761.82 query20 0.020.010.01 query21 15.45 0.380.33 query22 4.810.140.14 query23 15.81 0.490.30 query24 2.550.620.44 query25 0.170.120.10 query26 0.800.270.23 query27 0.100.100.09 query28 3.460.910.52 query29 12.47 4.443.54 query30 0.410.250.25 query31 2.770.600.34 query32 3.230.600.47 query33 2.962.983.04 query34 15.68 4.133.39 query35 3.343.273.31 query36 0.670.510.50 query37 0.120.100.09 query38 0.090.070.07 query39 0.070.060.06 query40 0.220.180.17 query41 0.120.080.09 query42 0.090.070.06 query43 0.080.070.07 Total cold run time: 96.84 s Total hot run time: 25.93 s ``` -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4883048858 TPC-DS: Total hot run time: 173633 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 2d3abcff31d896c46cdb1de7cbcbc67f6af0765f, data reload: false query5 4338639 505 505 query6 500 219 201 201 query7 4860607 331 331 query8 339 197 179 179 query9 8793406740594059 query10 496 348 293 293 query11 5911234321562156 query12 165 104 103 103 query13 1292600 437 437 query14 6311532349694969 query14_14340430743014301 query15 222 201 184 184 query16 1020480 376 376 query17 1127728 586 586 query18 2493480 354 354 query19 223 192 156 156 query20 112 114 113 113 query21 233 161 139 139 query22 13713 13554 13394 13394 query23 17244 16555 16169 16169 query23_116232 16167 16197 16167 query24 7638178513111311 query24_11345133513171317 query25 580 468 400 400 query26 1333351 220 220 query27 2587601 395 395 query28 4473201220202012 query29 1138649 506 506 query30 353 258 224 224 query31 11491102997 997 query32 116 68 64 64 query33 556 324 272 272 query34 12231117639 639 query35 780 794 693 693 query36 1413144112671267 query37 164 113 96 96 query38 1902172216541654 query39 940 934 909 909 query39_1900 891 872 872 query40 287 152 144 144 query41 64 61 61 61 query42 91 88 90 88 query43 318 324 274 274 query44 1452790 759 759 query45 199 182 175 175 query46 10501223768 768 query47 2353237022092209 query48 398 390 297 297 query49 579 418 312 312 query50 1037423 346 346 query51 4465451643204320 query52 85 82 74 74 query53 256 272 198 198 query54 301 227 213 213 query55 74 77 67 67 query56 292 313 291 291 query57 1443142613241324 query58 297 247 260 247 query59 1566164914111411 query60 328 277 250 250 query61 159 158 150 150 query62 700 644 590 590 query63 244 207 210 207 query64 2569761 622 622 query65 4861481947884788 query66 1837497 394 394 query67 29698 29461 29364 29364 query68 32281502960 960 query69 432 342 264 264 query70 1083950 981 950 query71 372 339 297 297 query72 2938267423302330 query73 905 814 449 449 query74 5162495947824782 query75 2645257722382238 query76 23571192812 812 query77 353 379 289 289 query78 12454 12668 12003 12003 query79 14361131793 793 query80 1289543 466 466 query81 554 333 280 280 query82 672 155 122 122 query83 354 315 305 305 query84 286 168 133 133 query85 974 609 507 507 query86 441 293 292 292 query87 1845183117551755 query88 3781282428232823 query89 461 401 349 349 query90 1940199 194 194 query91 199 189 158 158 query92 66 62 57 57 query93 17621520985 985 query94 873 322 308 308 query95 817 560 472 472 query96 1015798 338 338 query97 2683268425562556 query98 231 204 200 200 query99 1180117510241024 Total cold run time: 260752 ms Total hot run time: 173633 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
Re: [PR] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4883017323 TPC-H: Total hot run time: 29630 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit 2d3abcff31d896c46cdb1de7cbcbc67f6af0765f, data reload: false -- Round 1 -- q1 17810 401139983998 q2 2426325 202 202 q3 10645 1496819 819 q4 4758478 342 342 q5 8602886 576 576 q6 360 170 137 137 q7 867 860 628 628 q8 10642 147415231474 q9 6136447844204420 q10 6843180015171517 q11 538 351 325 325 q12 721 568 439 439 q13 18110 341927962796 q14 269 267 239 239 q15 q16 788 785 711 711 q17 995 934 860 860 q18 7040582256515651 q19 1159132211141114 q20 806 669 552 552 q21 5636270225292529 q22 441 361 301 301 Total cold run time: 105592 ms Total hot run time: 29630 ms - Round 2, with runtime_filter_mode=off - q1 4318423642584236 q2 299 314 224 224 q3 4618501144464446 q4 2083219213731373 q5 4422432047784320 q6 252 198 142 142 q7 2076179616721672 q8 2454210421392104 q9 7877779978797799 q10 4789496642374237 q11 587 423 374 374 q12 743 778 542 542 q13 3249367729642964 q14 290 303 283 283 q15 q16 723 787 678 678 q17 1368132013281320 q18 7979716067646764 q19 1131111511071107 q20 2221221919361936 q21 5284459044784478 q22 532 455 428 428 Total cold run time: 57295 ms Total hot run time: 51427 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4882500487 # Cloud UT Coverage Report Increment line coverage ` ` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/2d3abcff31d896c46cdb1de7cbcbc67f6af0765f_2d3abcff31d896c46cdb1de7cbcbc67f6af0765f_cloud/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/2d3abcff31d896c46cdb1de7cbcbc67f6af0765f_2d3abcff31d896c46cdb1de7cbcbc67f6af0765f_cloud/report/index.html) | Category | Coverage | |---|| | Function Coverage | 77.39% (1896/2450) | | Line Coverage | 64.45% (34069/52862) | | Region Coverage | 64.85% (17530/27032) | | Branch Coverage | 54.05% (9399/17390) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4882307387 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4603896740 /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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4599041329 # BE UT Coverage Report Increment line coverage `0.00% (0/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/9d115dd4580be3cac2a375d5b3afb78c2dfffdea_9d115dd4580be3cac2a375d5b3afb78c2dfffdea/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/9d115dd4580be3cac2a375d5b3afb78c2dfffdea_9d115dd4580be3cac2a375d5b3afb78c2dfffdea/report/index.html) | Category | Coverage | |---|| | Function Coverage | 54.14% (21081/38941) | | Line Coverage | 37.63% (199628/530547) | | Region Coverage | 33.92% (156488/461354) | | Branch Coverage | 34.85% (67964/194995) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4598177683 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4569610335 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4568254578 # BE UT Coverage Report Increment line coverage `0.00% (0/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/462918bd59d802eed8d3b0f1334c078bdf20d9f7_462918bd59d802eed8d3b0f1334c078bdf20d9f7/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/462918bd59d802eed8d3b0f1334c078bdf20d9f7_462918bd59d802eed8d3b0f1334c078bdf20d9f7/report/index.html) | Category | Coverage | |---|| | Function Coverage | 54.02% (20992/38861) | | Line Coverage | 37.56% (199016/529821) | | Region Coverage | 33.84% (155957/460878) | | Branch Coverage | 34.85% (67927/194926) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4566782858 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4229494073 # Cloud UT Coverage Report Increment line coverage ` ` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/b228dd71580cf7ebbfeb61fd8a5a3510ae7ac2e2_b228dd71580cf7ebbfeb61fd8a5a3510ae7ac2e2_cloud/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/b228dd71580cf7ebbfeb61fd8a5a3510ae7ac2e2_b228dd71580cf7ebbfeb61fd8a5a3510ae7ac2e2_cloud/report/index.html) | Category | Coverage | |---|| | Function Coverage | 78.48% (1798/2291) | | Line Coverage | 64.13% (32285/50345) | | Region Coverage | 65.10% (16253/24967) | | Branch Coverage | 55.61% (8687/15620) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4229424812 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4227057948 # BE UT Coverage Report Increment line coverage `0.00% (0/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/b228dd71580cf7ebbfeb61fd8a5a3510ae7ac2e2_b228dd71580cf7ebbfeb61fd8a5a3510ae7ac2e2/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/b228dd71580cf7ebbfeb61fd8a5a3510ae7ac2e2_b228dd71580cf7ebbfeb61fd8a5a3510ae7ac2e2/report/index.html) | Category | Coverage | |---|| | Function Coverage | 53.02% (20141/37985) | | Line Coverage | 36.58% (189413/517843) | | Region Coverage | 32.84% (147057/447825) | | Branch Coverage | 33.96% (64360/189534) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4225960146 # Cloud UT Coverage Report Increment line coverage ` ` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/b228dd71580cf7ebbfeb61fd8a5a3510ae7ac2e2_b228dd71580cf7ebbfeb61fd8a5a3510ae7ac2e2_cloud/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/b228dd71580cf7ebbfeb61fd8a5a3510ae7ac2e2_b228dd71580cf7ebbfeb61fd8a5a3510ae7ac2e2_cloud/report/index.html) | Category | Coverage | |---|| | Function Coverage | 78.48% (1798/2291) | | Line Coverage | 64.11% (32276/50345) | | Region Coverage | 65.04% (16238/24967) | | Branch Coverage | 55.60% (8684/15620) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-4225723371 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-3927488998 # BE Regression && UT Coverage Report Increment line coverage `0.00% (0/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 71.63% (26040/36352) | | Line Coverage | 54.24% (272041/501523) | | Region Coverage | 51.58% (226123/438384) | | Branch Coverage | 53.03% (97067/183047) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-3927615275 # FE Regression Coverage Report Increment line coverage ` 82.54% (52/63)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_merge_fe/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-3927576859 # FE Regression Coverage Report Increment line coverage ` 80.95% (51/63)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_merge_fe/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-3927469686 # BE Regression && UT Coverage Report Increment line coverage `0.00% (0/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 71.63% (26040/36352) | | Line Coverage | 54.24% (272023/501523) | | Region Coverage | 51.59% (226166/438384) | | Branch Coverage | 53.02% (97052/183047) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-3927064226 # BE Regression && UT Coverage Report Increment line coverage `6.25% (1/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 73.29% (26642/36350) | | Line Coverage | 56.47% (283111/501310) | | Region Coverage | 53.88% (236027/438021) | | Branch Coverage | 55.62% (101732/182917) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
doris-robot commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-3927031631 # BE UT Coverage Report Increment line coverage `0.00% (0/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/df83ba4a0754ee1b73d385fa1f752d590dcc74ae_df83ba4a0754ee1b73d385fa1f752d590dcc74ae/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/df83ba4a0754ee1b73d385fa1f752d590dcc74ae_df83ba4a0754ee1b73d385fa1f752d590dcc74ae/report/index.html) | Category | Coverage | |---|| | Function Coverage | 52.72% (19557/37093) | | Line Coverage | 36.30% (182524/502762) | | Region Coverage | 32.62% (141585/433999) | | Branch Coverage | 33.65% (61363/182343) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
doris-robot commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-3926590060 TPC-DS: Total hot run time: 183577 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 df83ba4a0754ee1b73d385fa1f752d590dcc74ae, data reload: false query5 4880661 527 527 query6 326 218 222 218 query7 4236468 278 278 query8 323 238 232 232 query9 8783277427452745 query10 532 379 324 324 query11 16978 17392 17281 17281 query12 192 125 124 124 query13 1258481 341 341 query14 6756328430803080 query14_12986291828882888 query15 204 208 178 178 query16 1023506 484 484 query17 1969843 623 623 query18 2917495 360 360 query19 217 240 229 229 query20 147 165 134 134 query21 227 148 139 139 query22 5297517648784878 query23 17177 16818 16525 16525 query23_116634 16568 16676 16568 query24 7290161212161216 query24_11234121212481212 query25 547 480 439 439 query26 1243261 153 153 query27 2788479 307 307 query28 4510186418671864 query29 819 588 500 500 query30 315 253 212 212 query31 884 749 652 652 query32 79 74 72 72 query33 534 385 293 293 query34 950 901 553 553 query35 634 689 610 610 query36 10891124937 937 query37 132 99 80 80 query38 3056292629262926 query39 873 869 838 838 query39_1845 850 825 825 query40 237 162 138 138 query41 71 67 64 64 query42 107 104 105 104 query43 378 384 347 347 query44 query45 208 194 182 182 query46 884 982 611 611 query47 2148213120372037 query48 309 327 242 242 query49 649 471 393 393 query50 693 279 230 230 query51 4081409441014094 query52 106 109 100 100 query53 290 338 295 295 query54 316 292 274 274 query55 94 91 84 84 query56 334 320 321 320 query57 1379134012591259 query58 326 270 277 270 query59 2533269325062506 query60 336 315 313 313 query61 149 147 143 143 query62 629 592 532 532 query63 304 280 274 274 query64 48451241981 981 query65 query66 1393473 354 354 query67 16370 16488 16322 16322 query68 query69 401 295 301 295 query70 983 1003954 954 query71 334 313 300 300 query72 2835267223752375 query73 535 535 322 322 query74 9970990397559755 query75 2852270724692469 query76 23021057680 680 query77 379 379 299 299 query78 11198 11479 10700 10700 query79 1164802 614 614 query80 1338629 534 534 query81 560 279 247 247 query82 1010147 117 117 query83 336 271 243 243 query84 252 114 99 99 query85 880 480 433 433 query86 415 329 287 287 query87 3139307930143014 query88 3597268826942688 query89 424 372 343 343 query90 2014184 174 174 query91 161 153 135 135 query92 74 78 75 75 query93 942 863 507 507 query94 643 326 285 285 query95 608 340 315 315 query96 642 516 231 231 query97 2477249023502350 query98 229 221 223 221 query99 1016989 871 871 Total cold run time: 255311 ms Total hot run time: 183577 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
doris-robot commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-3926533720 TPC-H: Total hot run time: 28843 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100 test result on commit df83ba4a0754ee1b73d385fa1f752d590dcc74ae, data reload: false -- Round 1 -- q1 17649 453743034303 q2 q3 10728 800 525 525 q4 4723353 253 253 q5 8088122810331033 q6 214 178 149 149 q7 826 848 673 673 q8 10716 150413771377 q9 6536482448034803 q10 6887187916141614 q11 457 260 253 253 q12 779 564 477 477 q13 17836 421834153415 q14 229 229 218 218 q15 970 798 790 790 q16 751 717 660 660 q17 775 879 429 429 q18 6678524951605160 q19 1168981 623 623 q20 535 504 412 412 q21 4449191314331433 q22 349 291 243 243 Total cold run time: 101343 ms Total hot run time: 28843 ms - Round 2, with runtime_filter_mode=off - q1 4445432143154315 q2 q3 1776218917141714 q4 882 1164777 777 q5 4046434543724345 q6 192 170 142 142 q7 1740161714841484 q8 2431267525282528 q9 7970756174997499 q10 2674287223702370 q11 499 436 419 419 q12 541 591 461 461 q13 4063437836753675 q14 294 305 272 272 q15 861 861 839 839 q16 723 758 750 750 q17 1194157512651265 q18 7032679767836783 q19 950 998 925 925 q20 2140228220232023 q21 4108347334763473 q22 471 468 395 395 Total cold run time: 49032 ms Total hot run time: 46454 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
doris-robot commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-3925827528 # Cloud UT Coverage Report Increment line coverage ` ` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/df83ba4a0754ee1b73d385fa1f752d590dcc74ae_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_cloud/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/df83ba4a0754ee1b73d385fa1f752d590dcc74ae_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_cloud/report/index.html) | Category | Coverage | |---|| | Function Coverage | 79.33% (1796/2264) | | Line Coverage | 64.80% (32007/49393) | | Region Coverage | 65.52% (15976/24383) | | Branch Coverage | 55.99% (8492/15168) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
heguanhui commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-3925661565 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-3890887676 # FE Regression Coverage Report Increment line coverage ` 77.78% (49/63)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_merge_fe/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-3890815398 # BE Regression && UT Coverage Report Increment line coverage `58.82% (10/17)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_merge/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/60323_df83ba4a0754ee1b73d385fa1f752d590dcc74ae_merge/report/index.html) | Category | Coverage | |---|| | Function Coverage | 73.49% (26591/36184) | | Line Coverage | 56.65% (282962/499461) | | Region Coverage | 54.23% (236341/435776) | | Branch Coverage | 56.00% (101953/182048) | -- 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] [Feature](functions) add brokers tvf function to query broker cluster info [doris]
hello-stephen commented on PR #60323: URL: https://github.com/apache/doris/pull/60323#issuecomment-3890562809 # BE UT Coverage Report Increment line coverage `0.00% (0/16)` :tada: [Increment coverage report](http://coverage.selectdb-in.cc/coverage/df83ba4a0754ee1b73d385fa1f752d590dcc74ae_df83ba4a0754ee1b73d385fa1f752d590dcc74ae/increment_report/index.html) [Complete coverage report](http://coverage.selectdb-in.cc/coverage/df83ba4a0754ee1b73d385fa1f752d590dcc74ae_df83ba4a0754ee1b73d385fa1f752d590dcc74ae/report/index.html) | Category | Coverage | |---|| | Function Coverage | 52.74% (19476/36929) | | Line Coverage | 36.24% (181461/500700) | | Region Coverage | 32.61% (140674/431397) | | Branch Coverage | 33.66% (61044/181344) | -- 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]
