Skip to content

[feat](auth integration)unify auth integration audit metadata#61172

Merged
CalvinKirs merged 2 commits intoapache:masterfrom
CalvinKirs:master-common-ai-params
Mar 12, 2026
Merged

[feat](auth integration)unify auth integration audit metadata#61172
CalvinKirs merged 2 commits intoapache:masterfrom
CalvinKirs:master-common-ai-params

Conversation

@CalvinKirs
Copy link
Member

@CalvinKirs CalvinKirs commented Mar 10, 2026

What problem does this PR solve?

AUTHENTICATION INTEGRATION metadata does not currently keep unified audit information such as creator, creation time, last modifier,
and last modification time. This makes it inconsistent with other user-driven metadata objects and makes future audit/display/
persistence work harder to extend.

This PR unifies audit metadata for authentication integrations.

What is changed

  • Introduce a reusable UserAuditMetadata base class for persisted audit fields of user-driven DDL objects.
  • Make AuthenticationIntegrationMeta inherit from UserAuditMetadata.
  • Persist the following fields for authentication integrations:
    • createUser
    • createTime
    • alterUser
    • modifyTime
  • Update create/alter metadata builders so that:
    • CREATE AUTHENTICATION INTEGRATION initializes creator/modifier and timestamps
    • ALTER AUTHENTICATION INTEGRATION preserves creator/create time and only updates last modifier/modify time

@Thearas
Copy link
Contributor

Thearas commented Mar 10, 2026

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?

@CalvinKirs
Copy link
Member Author

run buildall

@CalvinKirs CalvinKirs changed the title [feat] unify auth integration audit metadata [feat](auth integration)unify auth integration audit metadata Mar 10, 2026
@CalvinKirs
Copy link
Member Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
============================================
q1	17598	4518	4308	4308
q2	q3	10647	764	523	523
q4	4678	370	257	257
q5	7557	1229	1039	1039
q6	183	177	148	148
q7	813	845	685	685
q8	9839	1507	1323	1323
q9	5384	4730	4730	4730
q10	6319	1959	1658	1658
q11	483	265	245	245
q12	750	558	466	466
q13	18038	3007	2160	2160
q14	230	232	215	215
q15	928	807	822	807
q16	774	719	686	686
q17	716	846	442	442
q18	5837	5506	5214	5214
q19	1266	987	636	636
q20	498	505	389	389
q21	4840	1985	1536	1536
q22	404	335	264	264
Total cold run time: 97782 ms
Total hot run time: 27731 ms

----- Round 2, with runtime_filter_mode=off -----
============================================
q1	4734	4741	4690	4690
q2	q3	3874	4329	3834	3834
q4	869	1206	782	782
q5	4090	4393	4313	4313
q6	193	183	140	140
q7	1750	1669	1480	1480
q8	2502	2743	2557	2557
q9	7708	7243	7401	7243
q10	3865	3991	3671	3671
q11	542	443	440	440
q12	492	594	446	446
q13	3064	3222	2709	2709
q14	290	303	272	272
q15	864	806	827	806
q16	718	773	745	745
q17	1170	1435	1386	1386
q18	7269	6780	6579	6579
q19	972	939	895	895
q20	2089	2184	1989	1989
q21	3937	3486	3317	3317
q22	473	441	378	378
Total cold run time: 51465 ms
Total hot run time: 48672 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 153575 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 8608ee5a56e49ce630c183b0efbe9cee59f849dd, data reload: false

query5	4320	669	527	527
query6	335	225	202	202
query7	4205	463	266	266
query8	340	238	226	226
query9	8686	2746	2698	2698
query10	524	365	332	332
query11	7470	5861	5533	5533
query12	186	127	122	122
query13	1258	461	368	368
query14	5783	3790	3595	3595
query14_1	2828	2831	2824	2824
query15	212	197	187	187
query16	996	487	474	474
query17	1112	729	616	616
query18	2448	454	368	368
query19	215	212	182	182
query20	138	129	129	129
query21	226	148	129	129
query22	5039	5287	4818	4818
query23	15934	15639	15428	15428
query23_1	15605	16071	15843	15843
query24	7996	1718	1307	1307
query24_1	1299	1275	1297	1275
query25	605	640	425	425
query26	1568	281	156	156
query27	2832	514	316	316
query28	4740	1985	2012	1985
query29	830	600	492	492
query30	314	298	245	245
query31	1457	1385	1285	1285
query32	84	83	73	73
query33	511	367	297	297
query34	965	955	582	582
query35	652	709	654	654
query36	1207	1278	1090	1090
query37	145	102	89	89
query38	2970	2970	2926	2926
query39	885	877	843	843
query39_1	824	827	838	827
query40	231	150	133	133
query41	62	61	59	59
query42	293	305	292	292
query43	243	246	226	226
query44	
query45	199	195	183	183
query46	884	1010	619	619
query47	2167	2177	2070	2070
query48	301	308	221	221
query49	626	453	395	395
query50	679	275	212	212
query51	4133	4109	4103	4103
query52	291	292	282	282
query53	294	340	283	283
query54	299	272	260	260
query55	105	89	85	85
query56	326	325	323	323
query57	1374	1357	1288	1288
query58	293	283	288	283
query59	1343	1474	1335	1335
query60	342	335	323	323
query61	151	151	149	149
query62	627	592	539	539
query63	304	278	276	276
query64	4985	1282	1004	1004
query65	
query66	1462	483	362	362
query67	16393	16305	16302	16302
query68	
query69	389	314	295	295
query70	982	883	982	883
query71	327	301	301	301
query72	2721	2770	2491	2491
query73	546	552	326	326
query74	10016	9959	9799	9799
query75	2830	2752	2475	2475
query76	2270	1039	663	663
query77	349	373	308	308
query78	11182	11320	10675	10675
query79	2539	802	610	610
query80	1771	626	544	544
query81	560	279	243	243
query82	981	152	120	120
query83	329	288	239	239
query84	257	121	95	95
query85	896	504	449	449
query86	417	316	298	298
query87	3153	3098	3019	3019
query88	3540	2662	2628	2628
query89	423	380	345	345
query90	2024	177	174	174
query91	165	171	138	138
query92	79	71	76	71
query93	1182	845	507	507
query94	637	321	305	305
query95	595	403	328	328
query96	631	508	235	235
query97	2498	2483	2416	2416
query98	262	223	226	223
query99	1032	1000	923	923
Total cold run time: 236704 ms
Total hot run time: 153575 ms

@CalvinKirs
Copy link
Member Author

run cloud_p0

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Mar 12, 2026
@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@CalvinKirs CalvinKirs merged commit 09331aa into apache:master Mar 12, 2026
30 of 32 checks passed
@CalvinKirs CalvinKirs deleted the master-common-ai-params branch March 12, 2026 05:18
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.1.x reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants