Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fix](s3) do not replace https scheme if specified #44242 #44243

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

morningman
Copy link
Contributor

cherry-pick #44242

@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.

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?

@morningman
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17570	7356	7271	7271
q2	2062	156	159	156
q3	10696	1062	1139	1062
q4	10553	807	794	794
q5	7721	2758	2812	2758
q6	232	144	145	144
q7	965	601	591	591
q8	9344	1894	1962	1894
q9	6514	6377	6430	6377
q10	6952	2249	2254	2249
q11	464	251	259	251
q12	407	219	211	211
q13	17782	2972	2999	2972
q14	229	210	207	207
q15	558	520	518	518
q16	654	594	613	594
q17	954	529	504	504
q18	7197	6681	6431	6431
q19	1371	1110	1032	1032
q20	2838	2648	2676	2648
q21	3879	3237	3010	3010
q22	1078	977	975	975
Total cold run time: 110020 ms
Total hot run time: 42649 ms

----- Round 2, with runtime_filter_mode=off -----
q1	7293	7225	7276	7225
q2	343	239	237	237
q3	2996	2896	2868	2868
q4	2025	1769	1806	1769
q5	5620	5745	5812	5745
q6	228	143	140	140
q7	2129	1759	1775	1759
q8	3246	3511	3497	3497
q9	8942	8912	8824	8824
q10	3552	3512	3528	3512
q11	594	494	494	494
q12	818	612	611	611
q13	16523	3105	3176	3105
q14	288	273	274	273
q15	560	510	525	510
q16	703	661	676	661
q17	1865	1643	1641	1641
q18	8043	7709	7598	7598
q19	6873	1596	1548	1548
q20	2031	1863	1831	1831
q21	5372	5304	5240	5240
q22	1109	1015	1026	1015
Total cold run time: 81153 ms
Total hot run time: 60103 ms

@morningman morningman merged commit a346e77 into apache:branch-3.0 Nov 19, 2024
22 of 24 checks passed
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.

2 participants