Skip to content

Conversation

hust-hhb
Copy link
Contributor

@hust-hhb hust-hhb commented Nov 7, 2024

pr #43261 doesn't clear GetDeleteBitmapResponse correctly, this pr fix this problem
pick pr #43358

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Check List (For Author)

  • Test

    • Regression test
    • Unit Test
    • Manual test (add detailed scripts or steps below)
    • No need to test or manual test. Explain why:
      • This is a refactor/code format and no logic has been changed.
      • Previous test can cover this change.
      • No code files have been changed.
      • Other reason
  • Behavior changed:

    • No.
    • Yes.
  • Does this need documentation?

    • No.
    • Yes.
  • Release note

    None

Check List (For Reviewer who merge this PR)

  • Confirm the release note
  • Confirm test cases
  • Confirm document
  • Add branch pick label

@hust-hhb
Copy link
Contributor Author

hust-hhb commented Nov 7, 2024

run buildall

Copy link
Contributor

github-actions bot commented Nov 7, 2024

clang-tidy review says "All clean, LGTM! 👍"

@doris-robot
Copy link

TPC-H: Total hot run time: 40325 ms
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
Tpch sf100 test result on commit 3d6e58552e2d4fbb2d44e3d0429c0a233ca28786, data reload: false

------ Round 1 ----------------------------------
q1	17566	7417	7329	7329
q2	2088	165	157	157
q3	10688	1031	1131	1031
q4	10561	730	723	723
q5	7739	2866	2736	2736
q6	240	150	145	145
q7	984	606	602	602
q8	9611	1943	1983	1943
q9	6947	6419	6426	6419
q10	6956	2296	2257	2257
q11	446	254	255	254
q12	403	213	206	206
q13	17774	2980	2985	2980
q14	245	209	228	209
q15	558	515	509	509
q16	657	609	597	597
q17	965	514	550	514
q18	7091	6515	6547	6515
q19	3260	1050	977	977
q20	474	192	192	192
q21	3913	3275	3092	3092
q22	1097	964	938	938
Total cold run time: 110263 ms
Total hot run time: 40325 ms

----- Round 2, with runtime_filter_mode=off -----
q1	7457	7364	7279	7279
q2	406	301	316	301
q3	2979	2867	2833	2833
q4	2051	1801	1783	1783
q5	5654	5696	5691	5691
q6	233	150	146	146
q7	2184	1809	1733	1733
q8	3378	3551	3438	3438
q9	8840	8891	8903	8891
q10	3554	3521	3516	3516
q11	601	506	490	490
q12	818	629	627	627
q13	16443	3194	3192	3192
q14	306	267	289	267
q15	591	538	553	538
q16	709	670	667	667
q17	1848	1618	1608	1608
q18	8189	7844	7558	7558
q19	1867	1460	1576	1460
q20	2065	1854	1883	1854
q21	5515	5359	5283	5283
q22	1122	1028	1022	1022
Total cold run time: 76810 ms
Total hot run time: 60177 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 193040 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 3d6e58552e2d4fbb2d44e3d0429c0a233ca28786, data reload: false

query1	2420	2245	2290	2245
query2	6236	2188	2132	2132
query3	14439	10081	229	229
query4	33065	23324	23374	23324
query5	3777	451	430	430
query6	282	202	188	188
query7	3978	311	313	311
query8	291	239	235	235
query9	9410	2703	2691	2691
query10	449	256	251	251
query11	17822	15178	15266	15178
query12	152	99	97	97
query13	1536	452	413	413
query14	8484	7045	7337	7045
query15	225	182	178	178
query16	7724	527	521	521
query17	1542	574	553	553
query18	2139	606	608	606
query19	242	185	177	177
query20	121	112	111	111
query21	207	99	107	99
query22	4655	4462	4094	4094
query23	34211	33824	33655	33655
query24	12546	3371	3254	3254
query25	673	395	389	389
query26	1800	183	185	183
query27	2667	304	297	297
query28	7911	2499	2473	2473
query29	1061	418	405	405
query30	393	303	301	301
query31	1005	787	803	787
query32	88	55	53	53
query33	757	278	279	278
query34	978	497	489	489
query35	863	709	699	699
query36	1078	926	962	926
query37	268	72	72	72
query38	3941	3896	3857	3857
query39	1493	1408	1421	1408
query40	252	100	101	100
query41	53	50	52	50
query42	109	100	98	98
query43	536	510	486	486
query44	1261	795	792	792
query45	182	163	168	163
query46	1144	712	727	712
query47	1895	1848	1819	1819
query48	465	365	372	365
query49	1091	403	389	389
query50	828	405	411	405
query51	7045	7097	7053	7053
query52	104	89	98	89
query53	264	191	185	185
query54	1291	458	453	453
query55	77	76	77	76
query56	275	247	249	247
query57	1216	1107	1083	1083
query58	237	222	222	222
query59	3127	3232	2932	2932
query60	280	257	270	257
query61	131	129	127	127
query62	886	655	666	655
query63	210	196	186	186
query64	5402	723	708	708
query65	3270	3162	3201	3162
query66	1388	326	313	313
query67	15854	15193	15201	15193
query68	4858	588	586	586
query69	433	270	264	264
query70	1148	1139	1134	1134
query71	420	264	254	254
query72	6127	3897	4125	3897
query73	752	353	341	341
query74	10443	8975	8944	8944
query75	3353	2614	2645	2614
query76	2669	1026	981	981
query77	373	274	262	262
query78	10652	9655	9436	9436
query79	9056	604	603	603
query80	1524	428	430	428
query81	547	243	249	243
query82	1369	114	118	114
query83	191	157	166	157
query84	300	77	76	76
query85	1495	303	285	285
query86	460	302	297	297
query87	4341	4179	4212	4179
query88	5681	2438	2423	2423
query89	528	293	283	283
query90	2127	187	185	185
query91	185	145	139	139
query92	59	53	46	46
query93	7925	544	529	529
query94	882	279	295	279
query95	336	244	245	244
query96	626	284	284	284
query97	3423	3112	3145	3112
query98	210	195	194	194
query99	1619	1273	1281	1273
Total cold run time: 323914 ms
Total hot run time: 193040 ms

@dataroaring dataroaring merged commit e30560b into apache:branch-3.0 Nov 8, 2024
21 of 24 checks passed
@gavinchou gavinchou mentioned this pull request Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants