Skip to content

[refactor](type) Remove ColumnItemType#59890

Merged
Gabriel39 merged 5 commits intoapache:masterfrom
Gabriel39:dev_0114
Jan 16, 2026
Merged

[refactor](type) Remove ColumnItemType#59890
Gabriel39 merged 5 commits intoapache:masterfrom
Gabriel39:dev_0114

Conversation

@Gabriel39
Copy link
Copy Markdown
Contributor

What problem does this PR solve?

Issue Number: close #xxx

Related PR: #xxx

Problem Summary:

Release note

None

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.

Check List (For Reviewer who merge this PR)

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

@Gabriel39 Gabriel39 requested a review from zclllyybb as a code owner January 14, 2026 13:48
@hello-stephen
Copy link
Copy Markdown
Contributor

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See 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?

@Gabriel39
Copy link
Copy Markdown
Contributor Author

run buildall

2 similar comments
@Gabriel39
Copy link
Copy Markdown
Contributor Author

run buildall

@Gabriel39
Copy link
Copy Markdown
Contributor Author

run buildall

@Gabriel39
Copy link
Copy Markdown
Contributor Author

run buildall

1 similar comment
@Gabriel39
Copy link
Copy Markdown
Contributor Author

run buildall

@Gabriel39
Copy link
Copy Markdown
Contributor Author

run buildall

@Gabriel39
Copy link
Copy Markdown
Contributor Author

run buildall

@Gabriel39
Copy link
Copy Markdown
Contributor Author

run buildall

@Gabriel39
Copy link
Copy Markdown
Contributor Author

run buildall

@doris-robot
Copy link
Copy Markdown

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

------ Round 1 ----------------------------------
q1	17682	4251	4129	4129
q2	2073	362	255	255
q3	10108	1309	749	749
q4	10211	789	317	317
q5	7482	2156	1876	1876
q6	193	178	140	140
q7	972	818	677	677
q8	9307	1527	1233	1233
q9	5020	4760	4661	4661
q10	6822	1835	1401	1401
q11	543	311	317	311
q12	756	760	605	605
q13	17825	3887	3188	3188
q14	298	308	278	278
q15	599	518	519	518
q16	700	696	638	638
q17	735	762	603	603
q18	6712	6615	7303	6615
q19	1128	1061	695	695
q20	469	408	285	285
q21	3461	2687	2762	2687
q22	1172	1090	1057	1057
Total cold run time: 104268 ms
Total hot run time: 32918 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4556	4404	4388	4388
q2	367	405	330	330
q3	2311	2814	2405	2405
q4	1532	1960	1490	1490
q5	4453	4316	4392	4316
q6	217	170	127	127
q7	2030	1912	1899	1899
q8	2581	2424	2422	2422
q9	7275	7303	7329	7303
q10	2453	2698	2377	2377
q11	548	465	465	465
q12	712	821	652	652
q13	3780	3927	3096	3096
q14	274	304	264	264
q15	535	490	482	482
q16	626	678	631	631
q17	1132	1251	1359	1251
q18	7295	7244	7340	7244
q19	913	860	872	860
q20	1906	1988	1825	1825
q21	4616	4285	4247	4247
q22	1093	1038	988	988
Total cold run time: 51205 ms
Total hot run time: 49062 ms

@doris-robot
Copy link
Copy Markdown

TPC-DS: Total hot run time: 178325 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 f85d273aa1726bcb255ad044e90c2ca0111dba7f, data reload: false

query5	4368	654	502	502
query6	328	233	220	220
query7	4232	487	291	291
query8	354	274	250	250
query9	8713	3207	3236	3207
query10	532	411	343	343
query11	15394	15411	14971	14971
query12	191	116	116	116
query13	1254	549	448	448
query14	6218	3197	2894	2894
query14_1	2775	2732	2801	2732
query15	209	197	180	180
query16	1011	508	476	476
query17	1108	704	582	582
query18	2534	453	348	348
query19	239	228	205	205
query20	127	120	118	118
query21	218	146	124	124
query22	3977	4216	3966	3966
query23	16167	15738	15440	15440
query23_1	15570	15594	15487	15487
query24	7116	1583	1247	1247
query24_1	1232	1255	1231	1231
query25	570	476	435	435
query26	1251	296	195	195
query27	2705	472	304	304
query28	4506	2422	2403	2403
query29	761	535	435	435
query30	321	248	215	215
query31	803	658	560	560
query32	84	76	75	75
query33	522	362	310	310
query34	955	922	570	570
query35	719	764	664	664
query36	932	982	869	869
query37	150	107	102	102
query38	2797	2820	2772	2772
query39	777	753	761	753
query39_1	738	706	724	706
query40	228	140	125	125
query41	71	68	67	67
query42	110	108	111	108
query43	498	487	470	470
query44	1448	821	828	821
query45	192	191	176	176
query46	906	988	609	609
query47	1490	1487	1460	1460
query48	359	361	283	283
query49	612	437	362	362
query50	682	291	220	220
query51	3942	3825	3848	3825
query52	109	113	98	98
query53	304	328	286	286
query54	302	277	274	274
query55	87	82	83	82
query56	329	323	310	310
query57	1051	995	1017	995
query58	286	269	270	269
query59	2338	2303	2341	2303
query60	361	351	338	338
query61	162	159	161	159
query62	392	364	326	326
query63	307	271	275	271
query64	4915	1313	994	994
query65	3822	3754	3701	3701
query66	1449	440	345	345
query67	15858	15808	15638	15638
query68	2437	1180	857	857
query69	474	378	345	345
query70	1099	1046	977	977
query71	348	327	306	306
query72	5331	3282	3406	3282
query73	694	783	358	358
query74	8828	8794	8571	8571
query75	2769	2838	2538	2538
query76	2315	1112	713	713
query77	415	399	318	318
query78	10029	10147	9221	9221
query79	1149	963	650	650
query80	1533	575	496	496
query81	565	269	234	234
query82	1028	157	121	121
query83	357	281	260	260
query84	262	123	108	108
query85	1204	520	467	467
query86	424	304	316	304
query87	2909	2942	2814	2814
query88	3915	2900	2852	2852
query89	404	363	339	339
query90	1889	191	173	173
query91	188	179	147	147
query92	86	78	74	74
query93	1188	1051	604	604
query94	653	340	318	318
query95	606	351	333	333
query96	721	555	255	255
query97	2349	2429	2303	2303
query98	231	217	207	207
query99	645	600	538	538
Total cold run time: 251237 ms
Total hot run time: 178325 ms

@doris-robot
Copy link
Copy Markdown

ClickBench: Total hot run time: 27.26 s
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
ClickBench test result on commit f85d273aa1726bcb255ad044e90c2ca0111dba7f, data reload: false

query1	0.06	0.04	0.06
query2	0.11	0.06	0.05
query3	0.26	0.09	0.09
query4	1.60	0.12	0.11
query5	0.28	0.26	0.26
query6	1.15	0.67	0.66
query7	0.04	0.03	0.02
query8	0.06	0.04	0.05
query9	0.57	0.53	0.51
query10	0.57	0.57	0.56
query11	0.15	0.10	0.10
query12	0.15	0.12	0.12
query13	0.61	0.60	0.60
query14	0.96	0.95	0.96
query15	0.80	0.80	0.79
query16	0.44	0.42	0.44
query17	0.99	1.04	1.05
query18	0.24	0.22	0.22
query19	2.00	1.90	1.92
query20	0.05	0.01	0.01
query21	15.44	0.29	0.15
query22	5.18	0.06	0.05
query23	15.77	0.27	0.10
query24	1.09	0.27	0.18
query25	0.08	0.07	0.11
query26	0.15	0.14	0.14
query27	0.09	0.06	0.05
query28	3.26	1.10	0.89
query29	12.56	4.16	3.31
query30	0.28	0.14	0.13
query31	2.83	0.68	0.41
query32	3.25	0.58	0.47
query33	3.12	3.02	3.13
query34	16.02	5.15	4.49
query35	4.48	4.45	4.43
query36	0.67	0.50	0.50
query37	0.13	0.07	0.07
query38	0.08	0.04	0.05
query39	0.04	0.03	0.04
query40	0.19	0.15	0.14
query41	0.09	0.03	0.03
query42	0.05	0.04	0.03
query43	0.05	0.04	0.04
Total cold run time: 95.99 s
Total hot run time: 27.26 s

@github-actions github-actions Bot added the approved Indicates a PR has been approved by one committer. label Jan 16, 2026
@github-actions
Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actions
Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@hello-stephen
Copy link
Copy Markdown
Contributor

BE UT Coverage Report

Increment line coverage 44.76% (427/954) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 52.99% (19025/35904)
Line Coverage 39.07% (176218/451067)
Region Coverage 33.66% (136561/405689)
Branch Coverage 34.70% (59031/170112)

@hello-stephen
Copy link
Copy Markdown
Contributor

skip check_coverage

@Gabriel39 Gabriel39 merged commit 44818ad into apache:master Jan 16, 2026
26 of 28 checks passed
Gabriel39 added a commit to Gabriel39/incubator-doris that referenced this pull request Jan 16, 2026
yiguolei pushed a commit that referenced this pull request Jan 17, 2026
### What problem does this PR solve?

Issue Number: close #xxx

Related PR: #xxx

Problem Summary:

### Release note

None

### Check List (For Author)

- Test <!-- At least one of them must be included. -->
    - [ ] Regression test
    - [ ] Unit Test
    - [ ] Manual test (add detailed scripts or steps below)
    - [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
        - [ ] Previous test can cover this change.
        - [ ] No code files have been changed.
        - [ ] Other reason <!-- Add your reason?  -->

- Behavior changed:
    - [ ] No.
    - [ ] Yes. <!-- Explain the behavior change -->

- Does this need documentation?
    - [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg:
apache/doris-website#1214 -->

### Check List (For Reviewer who merge this PR)

- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR
should merge into -->
ybtsdst pushed a commit to ybtsdst/doris that referenced this pull request Feb 27, 2026
### What problem does this PR solve?

Issue Number: close #xxx

Related PR: #xxx

Problem Summary:

### Release note

None

### Check List (For Author)

- Test <!-- At least one of them must be included. -->
    - [ ] Regression test
    - [ ] Unit Test
    - [ ] Manual test (add detailed scripts or steps below)
    - [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
        - [ ] Previous test can cover this change.
        - [ ] No code files have been changed.
        - [ ] Other reason <!-- Add your reason?  -->

- Behavior changed:
    - [ ] No.
    - [ ] Yes. <!-- Explain the behavior change -->

- Does this need documentation?
    - [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg:
apache/doris-website#1214 -->

### Check List (For Reviewer who merge this PR)

- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR
should merge into -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. dev/4.0.3-merged reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants