Skip to content

[fix](Azure)Fix Azure endpoint detection for sovereign clouds with force-global config support#60903

Open
CalvinKirs wants to merge 1 commit intoapache:masterfrom
CalvinKirs:master-azure
Open

[fix](Azure)Fix Azure endpoint detection for sovereign clouds with force-global config support#60903
CalvinKirs wants to merge 1 commit intoapache:masterfrom
CalvinKirs:master-azure

Conversation

@CalvinKirs
Copy link
Member

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

@Thearas
Copy link
Contributor

Thearas commented Feb 28, 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

1 similar comment
@CalvinKirs
Copy link
Member Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
============================================
q1	17604	4510	4312	4312
q2	q3	10644	782	520	520
q4	4675	355	249	249
q5	7549	1195	1021	1021
q6	167	177	147	147
q7	777	842	674	674
q8	9284	1451	1274	1274
q9	4865	4696	4652	4652
q10	6753	1876	1638	1638
q11	435	262	244	244
q12	689	565	473	473
q13	17758	4240	3426	3426
q14	227	238	216	216
q15	907	788	788	788
q16	712	714	681	681
q17	721	872	431	431
q18	5922	5379	5161	5161
q19	1244	990	606	606
q20	515	500	385	385
q21	4920	2025	1551	1551
q22	366	332	267	267
Total cold run time: 96734 ms
Total hot run time: 28716 ms

----- Round 2, with runtime_filter_mode=off -----
============================================
q1	4577	4517	4635	4517
q2	q3	1814	2200	1762	1762
q4	916	1209	807	807
q5	4022	4304	4339	4304
q6	209	181	141	141
q7	1782	1628	1496	1496
q8	2581	2670	2570	2570
q9	7473	7247	7572	7247
q10	2636	2827	2424	2424
q11	512	447	414	414
q12	522	584	450	450
q13	3965	4365	3630	3630
q14	283	317	314	314
q15	900	837	822	822
q16	730	770	697	697
q17	1207	1534	1283	1283
q18	7016	6648	6759	6648
q19	888	899	932	899
q20	2108	2206	2149	2149
q21	3928	3451	3372	3372
q22	468	425	378	378
Total cold run time: 48537 ms
Total hot run time: 46324 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 183370 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 1594b1e59d3a67844c7f8a8ed9b74e9c72d70488, data reload: false

query5	5646	649	522	522
query6	328	217	204	204
query7	4216	458	263	263
query8	341	236	227	227
query9	8755	2750	2707	2707
query10	537	382	330	330
query11	16879	17479	16961	16961
query12	209	134	120	120
query13	1362	477	353	353
query14	8349	3339	3082	3082
query14_1	2989	2944	2953	2944
query15	214	197	181	181
query16	1057	418	491	418
query17	1527	775	653	653
query18	2750	448	348	348
query19	223	210	185	185
query20	139	128	137	128
query21	230	135	117	117
query22	4754	5033	4755	4755
query23	17205	16863	16558	16558
query23_1	16571	16707	16716	16707
query24	6786	1620	1220	1220
query24_1	1221	1218	1232	1218
query25	564	488	459	459
query26	1239	255	145	145
query27	2759	459	279	279
query28	4529	1875	1853	1853
query29	777	553	468	468
query30	313	255	210	210
query31	847	728	641	641
query32	80	69	69	69
query33	504	329	283	283
query34	903	907	565	565
query35	625	676	593	593
query36	1073	1090	985	985
query37	141	90	86	86
query38	2969	2920	2865	2865
query39	871	891	854	854
query39_1	846	827	829	827
query40	233	151	133	133
query41	61	59	58	58
query42	107	105	104	104
query43	371	383	362	362
query44	
query45	201	189	181	181
query46	875	985	601	601
query47	2101	2133	2049	2049
query48	312	321	235	235
query49	626	453	381	381
query50	673	284	215	215
query51	4107	4131	4051	4051
query52	105	106	98	98
query53	294	326	283	283
query54	302	263	251	251
query55	93	84	78	78
query56	313	300	321	300
query57	1348	1328	1248	1248
query58	291	274	275	274
query59	2613	2721	2558	2558
query60	341	329	323	323
query61	149	145	144	144
query62	626	584	547	547
query63	313	277	272	272
query64	4828	1261	983	983
query65	
query66	1368	445	374	374
query67	16318	16364	16385	16364
query68	
query69	394	304	302	302
query70	958	955	972	955
query71	331	310	295	295
query72	2773	2602	2398	2398
query73	541	543	327	327
query74	10023	9930	9792	9792
query75	2862	2736	2440	2440
query76	2250	1038	672	672
query77	366	385	335	335
query78	11283	11481	10751	10751
query79	1167	792	601	601
query80	1388	609	543	543
query81	556	289	246	246
query82	1005	147	116	116
query83	339	257	245	245
query84	252	111	102	102
query85	906	485	425	425
query86	461	309	296	296
query87	3124	3086	3039	3039
query88	3508	2686	2658	2658
query89	426	366	350	350
query90	1952	177	175	175
query91	161	157	132	132
query92	78	71	72	71
query93	990	823	504	504
query94	644	314	280	280
query95	561	411	314	314
query96	623	526	227	227
query97	2476	2479	2395	2395
query98	233	221	214	214
query99	1017	979	915	915
Total cold run time: 255415 ms
Total hot run time: 183370 ms

}

private static final String AZURE_ENDPOINT_SUFFIX = ".blob.core.windows.net";
private static final String[] AZURE_ACCOUNT_KEY_ENDPOINTS = {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not read from config?
Making it configurable with default list will make it extensible.

[fix](Azure)Fix Azure endpoint detection for sovereign clouds with force-global config support
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.

4 participants