You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "morningman (via GitHub)" <gi...@apache.org> on 2025/04/23 05:47:35 UTC

[PR] [opt](deps) patch re2 to add ignore_replace_escape option (#50124) [doris]

morningman opened a new pull request, #50324:
URL: https://github.com/apache/doris/pull/50324

   bp #50124


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [opt](deps) patch re2 to add ignore_replace_escape option (#50124) [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #50324:
URL: https://github.com/apache/doris/pull/50324#issuecomment-2823199474

   
   <details>
   <summary>TPC-H: <b>Total hot run time: 40132 ms</b></summary>
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
   Tpch sf100 test result on commit 6289f8f95b16dc2b3cef653b0fe9191e3b7babf2, data reload: false
   
   ------ Round 1 ----------------------------------
   q1	17582	6741	6702	6702
   q2	2080	170	172	170
   q3	10652	1127	1206	1127
   q4	10570	826	715	715
   q5	7776	2838	2807	2807
   q6	215	131	133	131
   q7	976	621	609	609
   q8	9350	1947	2013	1947
   q9	6553	6387	6403	6387
   q10	6979	2229	2295	2229
   q11	468	269	267	267
   q12	415	222	215	215
   q13	17793	2979	3007	2979
   q14	233	210	210	210
   q15	506	457	447	447
   q16	640	596	592	592
   q17	961	566	557	557
   q18	7198	6709	6721	6709
   q19	1422	1023	1048	1023
   q20	473	207	205	205
   q21	3944	3249	3133	3133
   q22	1093	988	971	971
   Total cold run time: 107879 ms
   Total hot run time: 40132 ms
   
   ----- Round 2, with runtime_filter_mode=off -----
   q1	6620	6572	6621	6572
   q2	326	243	228	228
   q3	2872	2753	2891	2753
   q4	2047	1810	1748	1748
   q5	5820	5738	5723	5723
   q6	215	128	129	128
   q7	2240	1782	1780	1780
   q8	3353	3531	3559	3531
   q9	8981	8781	8929	8781
   q10	3538	3525	3525	3525
   q11	593	518	508	508
   q12	840	600	620	600
   q13	8441	3146	3192	3146
   q14	302	261	273	261
   q15	496	454	469	454
   q16	705	666	643	643
   q17	1835	1608	1588	1588
   q18	8168	7801	7812	7801
   q19	1689	1562	1606	1562
   q20	2046	1824	1805	1805
   q21	5541	5320	5339	5320
   q22	1136	1051	1026	1026
   Total cold run time: 67804 ms
   Total hot run time: 59483 ms
   ```
   </details>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [opt](deps) patch re2 to add ignore_replace_escape option (#50124) [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #50324:
URL: https://github.com/apache/doris/pull/50324#issuecomment-2823221770

   
   <details>
   <summary>TPC-DS: <b>Total hot run time: 197659 ms</b></summary>
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
   TPC-DS sf100 test result on commit 6289f8f95b16dc2b3cef653b0fe9191e3b7babf2, data reload: false
   
   query1	1262	901	907	901
   query2	6226	2028	1957	1957
   query3	10822	4248	4368	4248
   query4	60820	29624	23880	23880
   query5	5203	457	452	452
   query6	423	180	185	180
   query7	5455	325	306	306
   query8	311	226	235	226
   query9	8687	2624	2614	2614
   query10	469	270	271	270
   query11	17997	15278	15575	15278
   query12	153	104	109	104
   query13	1478	490	436	436
   query14	9741	6710	7419	6710
   query15	193	187	175	175
   query16	7091	458	500	458
   query17	1190	600	603	600
   query18	1871	335	323	323
   query19	219	157	159	157
   query20	122	110	110	110
   query21	210	107	115	107
   query22	4874	4527	4751	4527
   query23	35037	34150	33955	33955
   query24	6129	2958	2899	2899
   query25	520	384	401	384
   query26	664	168	168	168
   query27	1837	354	359	354
   query28	4016	2445	2430	2430
   query29	678	433	419	419
   query30	242	161	162	161
   query31	978	824	831	824
   query32	68	52	56	52
   query33	469	278	284	278
   query34	931	523	516	516
   query35	852	728	734	728
   query36	1090	978	968	968
   query37	116	65	64	64
   query38	4070	3953	3975	3953
   query39	1519	1467	1471	1467
   query40	216	97	95	95
   query41	51	47	48	47
   query42	113	108	102	102
   query43	525	487	491	487
   query44	1154	812	829	812
   query45	188	172	171	171
   query46	1144	750	734	734
   query47	2014	1958	1971	1958
   query48	484	380	382	380
   query49	732	394	382	382
   query50	862	437	425	425
   query51	7251	7314	7081	7081
   query52	105	96	91	91
   query53	281	189	184	184
   query54	556	476	463	463
   query55	83	75	77	75
   query56	264	237	258	237
   query57	1302	1160	1150	1150
   query58	222	231	230	230
   query59	3223	2956	3079	2956
   query60	277	263	262	262
   query61	161	105	106	105
   query62	796	713	687	687
   query63	230	199	189	189
   query64	1440	679	658	658
   query65	3295	3223	3184	3184
   query66	629	296	312	296
   query67	15945	15743	15758	15743
   query68	4117	594	571	571
   query69	421	257	267	257
   query70	1203	1130	1081	1081
   query71	337	243	245	243
   query72	6390	4191	4350	4191
   query73	758	352	347	347
   query74	10273	9155	9101	9101
   query75	3355	2627	2684	2627
   query76	2137	1028	1070	1028
   query77	486	273	267	267
   query78	10534	9675	9650	9650
   query79	1719	599	583	583
   query80	1442	430	429	429
   query81	549	250	241	241
   query82	1219	91	87	87
   query83	274	143	155	143
   query84	292	82	77	77
   query85	1016	282	281	281
   query86	381	304	286	286
   query87	4414	4267	4245	4245
   query88	3759	2399	2381	2381
   query89	416	300	305	300
   query90	1887	185	182	182
   query91	187	151	150	150
   query92	63	51	50	50
   query93	2211	554	558	554
   query94	743	292	299	292
   query95	355	255	255	255
   query96	624	290	278	278
   query97	3337	3150	3160	3150
   query98	220	211	200	200
   query99	1606	1309	1358	1309
   Total cold run time: 314301 ms
   Total hot run time: 197659 ms
   ```
   </details>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [opt](deps) patch re2 to add ignore_replace_escape option (#50124) [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #50324:
URL: https://github.com/apache/doris/pull/50324#issuecomment-2823234429

   
   <details>
   <summary>ClickBench: <b>Total hot run time: 33 s</b></summary>
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
   ClickBench test result on commit 6289f8f95b16dc2b3cef653b0fe9191e3b7babf2, data reload: false
   
   query1	0.04	0.03	0.03
   query2	0.07	0.03	0.03
   query3	0.24	0.06	0.06
   query4	1.62	0.10	0.10
   query5	0.52	0.50	0.50
   query6	1.15	0.73	0.73
   query7	0.02	0.02	0.01
   query8	0.04	0.03	0.03
   query9	0.56	0.50	0.51
   query10	0.55	0.55	0.57
   query11	0.16	0.11	0.10
   query12	0.13	0.11	0.11
   query13	0.60	0.60	0.59
   query14	2.70	2.73	2.83
   query15	0.90	0.82	0.82
   query16	0.38	0.38	0.41
   query17	1.06	1.00	1.02
   query18	0.25	0.22	0.22
   query19	1.91	1.87	2.09
   query20	0.01	0.01	0.01
   query21	15.37	0.59	0.59
   query22	2.36	2.81	2.47
   query23	16.83	1.33	0.83
   query24	3.12	1.01	1.06
   query25	0.10	0.22	0.15
   query26	0.48	0.14	0.13
   query27	0.04	0.05	0.04
   query28	10.26	0.51	0.53
   query29	12.57	3.20	3.21
   query30	0.25	0.06	0.06
   query31	2.86	0.39	0.38
   query32	3.25	0.46	0.46
   query33	2.98	3.02	3.01
   query34	16.67	4.52	4.49
   query35	4.50	4.52	4.52
   query36	0.68	0.49	0.48
   query37	0.09	0.06	0.06
   query38	0.05	0.04	0.03
   query39	0.03	0.02	0.02
   query40	0.15	0.12	0.12
   query41	0.07	0.02	0.02
   query42	0.04	0.02	0.02
   query43	0.04	0.04	0.03
   Total cold run time: 105.7 s
   Total hot run time: 33 s
   ```
   </details>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [opt](deps) patch re2 to add ignore_replace_escape option (#50124) [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #50324:
URL: https://github.com/apache/doris/pull/50324#issuecomment-2823205394

   # BE UT Coverage Report
   Increment line coverage ` ` :tada:
   
   [Increment coverage report](http://coverage.selectdb-in.cc/coverage/6289f8f95b16dc2b3cef653b0fe9191e3b7babf2_6289f8f95b16dc2b3cef653b0fe9191e3b7babf2/increment_report/index.html)
   [Complete coverage report](http://coverage.selectdb-in.cc/coverage/6289f8f95b16dc2b3cef653b0fe9191e3b7babf2_6289f8f95b16dc2b3cef653b0fe9191e3b7babf2/report/index.html)
   | Category          | Coverage           |
   |-------------------|--------------------|
   | Function Coverage | 40.22% (10556/26248) |
   | Line Coverage     | 30.98% (89287/288214) |
   | Region Coverage   | 30.11% (46057/152965) |
   | Branch Coverage   | 26.62% (23547/88458) |


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [opt](deps) patch re2 to add ignore_replace_escape option (#50124) [doris]

Posted by "dataroaring (via GitHub)" <gi...@apache.org>.
dataroaring merged PR #50324:
URL: https://github.com/apache/doris/pull/50324


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [opt](deps) patch re2 to add ignore_replace_escape option (#50124) [doris]

Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #50324:
URL: https://github.com/apache/doris/pull/50324#issuecomment-2823133168

   
   Thank you for your contribution to Apache Doris.
   Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR).
   
   Please clearly describe your PR:
   1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
   2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
   3. What features were added. Why was this function added?
   4. Which code was refactored and why was this part of the code refactored?
   5. Which functions were optimized and what is the difference before and after the optimization?
       


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [opt](deps) patch re2 to add ignore_replace_escape option (#50124) [doris]

Posted by "morningman (via GitHub)" <gi...@apache.org>.
morningman commented on PR #50324:
URL: https://github.com/apache/doris/pull/50324#issuecomment-2823133304

   run buildall


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org