You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "Gabriel39 (via GitHub)" <gi...@apache.org> on 2023/11/21 11:02:44 UTC

[PR] [pipelineX](dependency) split different dependencies [doris]

Gabriel39 opened a new pull request, #27366:
URL: https://github.com/apache/doris/pull/27366

   ## Proposed changes
   
   Issue Number: close #xxx
   
   <!--Describe your changes.-->
   
   ## Further comments
   
   If this is a relatively large or complex change, kick off the discussion at [dev@doris.apache.org](mailto:dev@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc...
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1821944152

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


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "Gabriel39 (via GitHub)" <gi...@apache.org>.
Gabriel39 commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1820710873

   run buildall


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1821278460

   TeamCity be ut coverage result:
    Function Coverage: 36.55% (8445/23107) 
    Line Coverage: 28.84% (68657/238030)
    Region Coverage: 27.81% (35505/127672)
    Branch Coverage: 24.56% (18105/73730)
    Coverage Report: http://coverage.selectdb-in.cc/coverage/28d200c9a979a2d5cb7c1e9e0a742e4c870c8c77_28d200c9a979a2d5cb7c1e9e0a742e4c870c8c77/report/index.html


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "Gabriel39 (via GitHub)" <gi...@apache.org>.
Gabriel39 commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1821942400

   run buildall


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1820807675

   (From new machine)TeamCity pipeline, clickbench performance test result:
    the sum of best hot time: 45.63 seconds
    stream load tsv:          580 seconds loaded 74807831229 Bytes, about 123 MB/s
    stream load json:         18 seconds loaded 2358488459 Bytes, about 124 MB/s
    stream load orc:          65 seconds loaded 1101869774 Bytes, about 16 MB/s
    stream load parquet:          34 seconds loaded 861443392 Bytes, about 24 MB/s
    insert into select:          28.1 seconds inserted 10000000 Rows, about 355K ops/s
    storage size: 17158197435 Bytes


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1821944180

   PR approved by anyone and no changes requested.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "Gabriel39 (via GitHub)" <gi...@apache.org>.
Gabriel39 commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1820832829

   run buildall


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1822046938

   TeamCity be ut coverage result:
    Function Coverage: 36.56% (8448/23107) 
    Line Coverage: 28.85% (68671/238032)
    Region Coverage: 27.81% (35509/127672)
    Branch Coverage: 24.57% (18112/73730)
    Coverage Report: http://coverage.selectdb-in.cc/coverage/80faa1e308bc1c93436b6afcef9e1d481fe65ed4_80faa1e308bc1c93436b6afcef9e1d481fe65ed4/report/index.html


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "Gabriel39 (via GitHub)" <gi...@apache.org>.
Gabriel39 commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1821996872

   run buildall


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1820934516

   (From new machine)TeamCity pipeline, clickbench performance test result:
    the sum of best hot time: 47.78 seconds
    stream load tsv:          585 seconds loaded 74807831229 Bytes, about 121 MB/s
    stream load json:         18 seconds loaded 2358488459 Bytes, about 124 MB/s
    stream load orc:          66 seconds loaded 1101869774 Bytes, about 15 MB/s
    stream load parquet:          34 seconds loaded 861443392 Bytes, about 24 MB/s
    insert into select:          29.1 seconds inserted 10000000 Rows, about 343K ops/s
    storage size: 17098446774 Bytes


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "Gabriel39 (via GitHub)" <gi...@apache.org>.
Gabriel39 commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1820958977

   run buildall


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1820915555

   
   <details>
   <summary>TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'</summary>
   
   ```
   Tpch sf100 test result on commit 2fe4b071609424618746b650fcad92335b222230, data reload: false
   
   run tpch-sf100 query with default conf and session variables
   q1	4868	4649	4666	4649
   q2	355	158	155	155
   q3	2094	1998	1907	1907
   q4	1383	1273	1234	1234
   q5	3955	3963	4031	3963
   q6	242	131	131	131
   q7	1426	893	900	893
   q8	2715	2783	2747	2747
   q9	9728	9744	9595	9595
   q10	3448	3539	3521	3521
   q11	378	250	254	250
   q12	435	296	304	296
   q13	4587	3828	3868	3828
   q14	315	285	280	280
   q15	586	539	521	521
   q16	667	587	581	581
   q17	1125	970	952	952
   q18	7811	7307	7347	7307
   q19	1672	1669	1686	1669
   q20	542	297	297	297
   q21	4370	3948	3911	3911
   q22	477	367	367	367
   Total cold run time: 53179 ms
   Total hot run time: 49054 ms
   
   run tpch-sf100 query with default conf and set session variable runtime_filter_mode=off
   q1	4605	4582	4563	4563
   q2	345	212	272	212
   q3	4012	3999	4004	3999
   q4	2693	2685	2687	2685
   q5	9633	9598	9683	9598
   q6	242	121	127	121
   q7	3009	2491	2448	2448
   q8	4432	4433	4460	4433
   q9	13246	13117	13026	13026
   q10	4126	4198	4196	4196
   q11	789	647	659	647
   q12	970	821	829	821
   q13	4272	3605	3617	3605
   q14	383	343	359	343
   q15	580	518	523	518
   q16	744	661	689	661
   q17	3959	3853	3860	3853
   q18	9505	9083	8927	8927
   q19	1799	1792	1763	1763
   q20	2387	2082	2047	2047
   q21	8783	8684	8574	8574
   q22	932	768	802	768
   Total cold run time: 81446 ms
   Total hot run time: 77808 ms
   ```
   </details>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1820794360

   
   <details>
   <summary>TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'</summary>
   
   ```
   Tpch sf100 test result on commit 8eebe6a4eaa1af2522c0f797b5ee02bcf0a276e1, data reload: false
   
   run tpch-sf100 query with default conf and session variables
   q1	4926	4635	4692	4635
   q2	359	152	164	152
   q3	2030	1921	1896	1896
   q4	1382	1244	1199	1199
   q5	3963	3946	4060	3946
   q6	244	132	134	132
   q7	1409	872	895	872
   q8	2720	2767	2758	2758
   q9	9824	9651	9485	9485
   q10	3476	3525	3524	3524
   q11	374	242	243	242
   q12	446	293	290	290
   q13	4569	3824	3808	3808
   q14	319	290	292	290
   q15	586	537	523	523
   q16	671	582	583	582
   q17	1123	988	948	948
   q18	7706	7394	7258	7258
   q19	1648	1694	1677	1677
   q20	534	328	304	304
   q21	4372	3929	3972	3929
   q22	481	370	367	367
   Total cold run time: 53162 ms
   Total hot run time: 48817 ms
   
   run tpch-sf100 query with default conf and set session variable runtime_filter_mode=off
   q1	4597	4558	4573	4558
   q2	338	228	258	228
   q3	3996	4006	3995	3995
   q4	2704	2704	2733	2704
   q5	9672	9671	9663	9663
   q6	243	126	126	126
   q7	3026	2475	2460	2460
   q8	4420	4414	4401	4401
   q9	13200	13085	13096	13085
   q10	4097	4187	4186	4186
   q11	761	629	626	626
   q12	973	812	797	797
   q13	4261	3607	3572	3572
   q14	391	346	347	346
   q15	577	522	521	521
   q16	758	686	669	669
   q17	3878	3886	3837	3837
   q18	9505	8877	9058	8877
   q19	1820	1767	1762	1762
   q20	2377	2092	2067	2067
   q21	8866	8619	8669	8619
   q22	879	762	804	762
   Total cold run time: 81339 ms
   Total hot run time: 77861 ms
   ```
   </details>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "Gabriel39 (via GitHub)" <gi...@apache.org>.
Gabriel39 commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1821995004

   run buildall


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1820848732

   TeamCity be ut coverage result:
    Function Coverage: 36.56% (8448/23108) 
    Line Coverage: 28.86% (68674/237977)
    Region Coverage: 27.82% (35511/127641)
    Branch Coverage: 24.57% (18111/73704)
    Coverage Report: http://coverage.selectdb-in.cc/coverage/8eebe6a4eaa1af2522c0f797b5ee02bcf0a276e1_8eebe6a4eaa1af2522c0f797b5ee02bcf0a276e1/report/index.html


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1821094866

   TeamCity be ut coverage result:
    Function Coverage: 36.55% (8445/23108) 
    Line Coverage: 28.84% (68656/238032)
    Region Coverage: 27.81% (35501/127670)
    Branch Coverage: 24.56% (18106/73726)
    Coverage Report: http://coverage.selectdb-in.cc/coverage/edbe7cbbb060e186d11bdbdd5b57a1dc24ce163b_edbe7cbbb060e186d11bdbdd5b57a1dc24ce163b/report/index.html


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1821194557

   (From new machine)TeamCity pipeline, clickbench performance test result:
    the sum of best hot time: 47.37 seconds
    stream load tsv:          581 seconds loaded 74807831229 Bytes, about 122 MB/s
    stream load json:         18 seconds loaded 2358488459 Bytes, about 124 MB/s
    stream load orc:          65 seconds loaded 1101869774 Bytes, about 16 MB/s
    stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
    insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
    storage size: 17099208045 Bytes


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1822104928

   
   <details>
   <summary>TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'</summary>
   
   ```
   Tpch sf100 test result on commit 80faa1e308bc1c93436b6afcef9e1d481fe65ed4, data reload: false
   
   run tpch-sf100 query with default conf and session variables
   q1	4874	4641	4634	4634
   q2	367	155	157	155
   q3	2013	1933	1919	1919
   q4	1384	1249	1246	1246
   q5	3965	3952	3979	3952
   q6	247	126	126	126
   q7	1430	872	897	872
   q8	2734	2760	2755	2755
   q9	9873	9655	9578	9578
   q10	3462	3513	3516	3513
   q11	383	246	253	246
   q12	441	296	297	296
   q13	4581	3815	3788	3788
   q14	313	297	283	283
   q15	587	535	531	531
   q16	676	588	587	587
   q17	1115	929	896	896
   q18	7881	7309	7430	7309
   q19	1688	1645	1671	1645
   q20	564	323	306	306
   q21	4267	3969	3935	3935
   q22	478	368	368	368
   Total cold run time: 53323 ms
   Total hot run time: 48940 ms
   
   run tpch-sf100 query with default conf and set session variable runtime_filter_mode=off
   q1	4555	4591	4569	4569
   q2	344	226	282	226
   q3	3991	3991	3964	3964
   q4	2684	2678	2679	2678
   q5	9664	9634	9583	9583
   q6	240	121	121	121
   q7	3028	2471	2445	2445
   q8	4458	4430	4442	4430
   q9	13181	13079	13087	13079
   q10	4129	4184	4221	4184
   q11	740	644	663	644
   q12	995	803	808	803
   q13	4257	3551	3563	3551
   q14	394	347	349	347
   q15	575	524	532	524
   q16	738	693	694	693
   q17	3890	3889	3834	3834
   q18	9533	9025	9079	9025
   q19	1806	1761	1773	1761
   q20	2375	2083	2072	2072
   q21	8806	8853	8418	8418
   q22	926	772	756	756
   Total cold run time: 81309 ms
   Total hot run time: 77707 ms
   ```
   </details>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1821248460

   (From new machine)TeamCity pipeline, clickbench performance test result:
    the sum of best hot time: 45.81 seconds
    stream load tsv:          577 seconds loaded 74807831229 Bytes, about 123 MB/s
    stream load json:         18 seconds loaded 2358488459 Bytes, about 124 MB/s
    stream load orc:          65 seconds loaded 1101869774 Bytes, about 16 MB/s
    stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
    insert into select:          28.8 seconds inserted 10000000 Rows, about 347K ops/s
    storage size: 17100889066 Bytes


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1821998119

   run buildall


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1822043059

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


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1822107746

   (From new machine)TeamCity pipeline, clickbench performance test result:
    the sum of best hot time: 46.13 seconds
    stream load tsv:          583 seconds loaded 74807831229 Bytes, about 122 MB/s
    stream load json:         18 seconds loaded 2358488459 Bytes, about 124 MB/s
    stream load orc:          65 seconds loaded 1101869774 Bytes, about 16 MB/s
    stream load parquet:          33 seconds loaded 861443392 Bytes, about 24 MB/s
    insert into select:          28.7 seconds inserted 10000000 Rows, about 348K ops/s
    storage size: 17099676227 Bytes


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on code in PR #27366:
URL: https://github.com/apache/doris/pull/27366#discussion_r1400419901


##########
be/src/pipeline/exec/aggregation_source_operator.cpp:
##########
@@ -79,37 +83,46 @@ Status AggLocalState::init(RuntimeState* state, LocalStateInfo& info) {
     return Status::OK();
 }
 
+Status AggLocalState::_destroy_agg_status(vectorized::AggregateDataPtr data) {

Review Comment:
   warning: method '_destroy_agg_status' can be made static [readability-convert-member-functions-to-static]
   
   be/src/pipeline/exec/aggregation_source_operator.h:110:
   ```diff
   -     Status _destroy_agg_status(vectorized::AggregateDataPtr data);
   +     static Status _destroy_agg_status(vectorized::AggregateDataPtr data);
   ```
   



##########
be/src/pipeline/exec/aggregation_source_operator.cpp:
##########
@@ -376,9 +438,9 @@
                             key_columns[0]->insert_data(nullptr, 0);
                             auto mapped = agg_method.hash_table->template get_null_key_data<
                                     vectorized::AggregateDataPtr>();
-                            for (size_t i = 0; i < _shared_state->aggregate_evaluators.size(); ++i)
-                                _shared_state->aggregate_evaluators[i]->insert_result_info(
-                                        mapped + _dependency->offsets_of_aggregate_states()[i],
+                            for (size_t i = 0; i < shared_state.aggregate_evaluators.size(); ++i)

Review Comment:
   warning: statement should be inside braces [readability-braces-around-statements]
   
   ```suggestion
                               for (size_t i = 0; i < shared_state.aggregate_evaluators.size(); ++i) {
   ```
   
   be/src/pipeline/exec/aggregation_source_operator.cpp:443:
   ```diff
   -                                         value_columns[i].get());
   +                                         value_columns[i].get());
   + }
   ```
   



##########
be/src/pipeline/exec/analytic_sink_operator.cpp:
##########
@@ -66,6 +66,126 @@
     return Status::OK();
 }
 
+bool AnalyticSinkLocalState::_whether_need_next_partition(
+        vectorized::BlockRowPos& found_partition_end) {
+    auto& shared_state = *_shared_state;
+    if (shared_state.input_eos ||
+        (shared_state.current_row_position <
+         shared_state.partition_by_end.pos)) { //now still have partition data
+        return false;
+    }
+    if ((shared_state.partition_by_eq_expr_ctxs.empty() && !shared_state.input_eos) ||
+        (found_partition_end.pos == 0)) { //no partition, get until fetch to EOS
+        return true;
+    }
+    if (!shared_state.partition_by_eq_expr_ctxs.empty() &&
+        found_partition_end.pos == shared_state.all_block_end.pos &&
+        !shared_state.input_eos) { //current partition data calculate done
+        return true;
+    }
+    return false;
+}
+
+//_partition_by_columns,_order_by_columns save in blocks, so if need to calculate the boundary, may find in which blocks firstly
+vectorized::BlockRowPos AnalyticSinkLocalState::_compare_row_to_find_end(

Review Comment:
   warning: method '_compare_row_to_find_end' can be made static [readability-convert-member-functions-to-static]
   
   be/src/pipeline/exec/analytic_sink_operator.h:81:
   ```diff
   -     vectorized::BlockRowPos _compare_row_to_find_end(int idx, vectorized::BlockRowPos start,
   +     static vectorized::BlockRowPos _compare_row_to_find_end(int idx, vectorized::BlockRowPos start,
   ```
   



##########
be/src/pipeline/exec/data_queue.cpp:
##########
@@ -41,7 +41,8 @@ DataQueue::DataQueue(int child_count, WriteDependency* dependency)
           _cur_bytes_in_queue(child_count),
           _cur_blocks_nums_in_queue(child_count),
           _flag_queue_idx(0),
-          _dependency(dependency) {
+          _source_dependency(nullptr),

Review Comment:
   warning: member initializer for '_source_dependency' is redundant [modernize-use-default-member-init]
   
   ```suggestion
             ,
   ```
   



##########
be/src/pipeline/exec/analytic_sink_operator.h:
##########
@@ -45,20 +45,45 @@ class AnalyticSinkOperator final : public StreamingOperator<AnalyticSinkOperator
     bool can_write() override { return _node->can_write(); }
 };
 
+class AnalyticSinkDependency final : public Dependency {
+public:
+    using SharedState = AnalyticSharedState;
+    AnalyticSinkDependency(int id, int node_id)
+            : Dependency(id, node_id, "AnalyticSinkDependency", true) {}
+    ~AnalyticSinkDependency() override = default;
+};
+
 class AnalyticSinkOperatorX;
 
-class AnalyticSinkLocalState : public PipelineXSinkLocalState<AnalyticDependency> {
+class AnalyticSinkLocalState : public PipelineXSinkLocalState<AnalyticSinkDependency> {
     ENABLE_FACTORY_CREATOR(AnalyticSinkLocalState);
 
 public:
     AnalyticSinkLocalState(DataSinkOperatorXBase* parent, RuntimeState* state)
-            : PipelineXSinkLocalState<AnalyticDependency>(parent, state) {}
+            : PipelineXSinkLocalState<AnalyticSinkDependency>(parent, state) {}
 
     Status init(RuntimeState* state, LocalSinkStateInfo& info) override;
 
 private:
     friend class AnalyticSinkOperatorX;
 
+    bool _refresh_need_more_input() {

Review Comment:
   warning: method '_refresh_need_more_input' can be made static [readability-convert-member-functions-to-static]
   
   ```suggestion
       static bool _refresh_need_more_input() {
   ```
   



##########
be/src/pipeline/exec/analytic_source_operator.h:
##########
@@ -71,6 +79,22 @@ class AnalyticLocalState final : public PipelineXLocalState<AnalyticDependency>
     void _insert_result_info(int64_t current_block_rows);
 
     void _update_order_by_range();
+    bool _refresh_need_more_input() {

Review Comment:
   warning: method '_refresh_need_more_input' can be made static [readability-convert-member-functions-to-static]
   
   ```suggestion
       static bool _refresh_need_more_input() {
   ```
   



##########
be/src/pipeline/exec/streaming_aggregation_sink_operator.h:
##########
@@ -93,6 +93,16 @@ class StreamingAggSinkLocalState final
     Status _pre_agg_with_serialized_key(doris::vectorized::Block* in_block,
                                         doris::vectorized::Block* out_block);
     bool _should_expand_preagg_hash_tables();
+    void _make_nullable_output_key(vectorized::Block* block) {

Review Comment:
   warning: method '_make_nullable_output_key' can be made static [readability-convert-member-functions-to-static]
   
   ```suggestion
       static void _make_nullable_output_key(vectorized::Block* block) {
   ```
   



##########
be/src/pipeline/exec/analytic_sink_operator.cpp:
##########
@@ -66,6 +66,126 @@ Status AnalyticSinkLocalState::init(RuntimeState* state, LocalSinkStateInfo& inf
     return Status::OK();
 }
 
+bool AnalyticSinkLocalState::_whether_need_next_partition(

Review Comment:
   warning: method '_whether_need_next_partition' can be made static [readability-convert-member-functions-to-static]
   
   be/src/pipeline/exec/analytic_sink_operator.h:84:
   ```diff
   -     bool _whether_need_next_partition(vectorized::BlockRowPos& found_partition_end);
   +     static bool _whether_need_next_partition(vectorized::BlockRowPos& found_partition_end);
   ```
   



##########
be/src/pipeline/exec/analytic_source_operator.cpp:
##########
@@ -37,8 +37,129 @@ AnalyticLocalState::AnalyticLocalState(RuntimeState* state, OperatorXBase* paren
           _agg_functions_size(0),
           _agg_functions_created(false) {}
 
+//_partition_by_columns,_order_by_columns save in blocks, so if need to calculate the boundary, may find in which blocks firstly
+vectorized::BlockRowPos AnalyticLocalState::_compare_row_to_find_end(int idx,

Review Comment:
   warning: method '_compare_row_to_find_end' can be made static [readability-convert-member-functions-to-static]
   
   be/src/pipeline/exec/analytic_source_operator.h:93:
   ```diff
   -     vectorized::BlockRowPos _compare_row_to_find_end(int idx, vectorized::BlockRowPos start,
   +     static vectorized::BlockRowPos _compare_row_to_find_end(int idx, vectorized::BlockRowPos start,
   ```
   



##########
be/src/pipeline/exec/aggregation_source_operator.cpp:
##########
@@ -301,15 +341,37 @@
     return Status::OK();
 }
 
+Status AggLocalState::_merge_spilt_data() {

Review Comment:
   warning: method '_merge_spilt_data' can be made static [readability-convert-member-functions-to-static]
   
   be/src/pipeline/exec/aggregation_source_operator.h:112:
   ```diff
   -     Status _merge_spilt_data();
   +     static Status _merge_spilt_data();
   ```
   



##########
be/src/pipeline/exec/analytic_source_operator.cpp:
##########
@@ -37,8 +37,129 @@
           _agg_functions_size(0),
           _agg_functions_created(false) {}
 
+//_partition_by_columns,_order_by_columns save in blocks, so if need to calculate the boundary, may find in which blocks firstly
+vectorized::BlockRowPos AnalyticLocalState::_compare_row_to_find_end(int idx,
+                                                                     vectorized::BlockRowPos start,
+                                                                     vectorized::BlockRowPos end,
+                                                                     bool need_check_first) {
+    auto& shared_state = *_shared_state;
+    int64_t start_init_row_num = start.row_num;
+    vectorized::ColumnPtr start_column =
+            shared_state.input_blocks[start.block_num].get_by_position(idx).column;
+    vectorized::ColumnPtr start_next_block_column = start_column;
+
+    DCHECK_LE(start.block_num, end.block_num);
+    DCHECK_LE(start.block_num, shared_state.input_blocks.size() - 1);
+    int64_t start_block_num = start.block_num;
+    int64_t end_block_num = end.block_num;
+    int64_t mid_blcok_num = end.block_num;
+    // To fix this problem: https://github.com/apache/doris/issues/15951
+    // in this case, the partition by column is last row of block, so it's pointed to a new block at row = 0, range is: [left, right)
+    // From the perspective of order by column, the two values are exactly equal.
+    // so the range will be get wrong because it's compare_at == 0 with next block at row = 0
+    if (need_check_first && end.block_num > 0 && end.row_num == 0) {
+        end.block_num--;
+        end_block_num--;
+        end.row_num = shared_state.input_blocks[end_block_num].rows();
+    }
+    //binary search find in which block
+    while (start_block_num < end_block_num) {
+        mid_blcok_num = (start_block_num + end_block_num + 1) >> 1;
+        start_next_block_column =
+                shared_state.input_blocks[mid_blcok_num].get_by_position(idx).column;
+        //Compares (*this)[n] and rhs[m], this: start[init_row]  rhs: mid[0]
+        if (start_column->compare_at(start_init_row_num, 0, *start_next_block_column, 1) == 0) {
+            start_block_num = mid_blcok_num;
+        } else {
+            end_block_num = mid_blcok_num - 1;
+        }
+    }
+
+    // have check the start.block_num:  start_column[start_init_row_num] with mid_blcok_num start_next_block_column[0]
+    // now next block must not be result, so need check with end_block_num: start_next_block_column[last_row]
+    if (end_block_num == mid_blcok_num - 1) {
+        start_next_block_column =
+                shared_state.input_blocks[end_block_num].get_by_position(idx).column;
+        int64_t block_size = shared_state.input_blocks[end_block_num].rows();
+        if ((start_column->compare_at(start_init_row_num, block_size - 1, *start_next_block_column,
+                                      1) == 0)) {
+            start.block_num = end_block_num + 1;
+            start.row_num = 0;
+            return start;
+        }
+    }
+
+    //check whether need get column again, maybe same as first init
+    // if the start_block_num have move to forword, so need update start block num and compare it from row_num=0
+    if (start_block_num != start.block_num) {
+        start_init_row_num = 0;
+        start.block_num = start_block_num;
+        start_column = shared_state.input_blocks[start.block_num].get_by_position(idx).column;
+    }
+    //binary search, set start and end pos
+    int64_t start_pos = start_init_row_num;
+    int64_t end_pos = shared_state.input_blocks[start.block_num].rows();
+    //if end_block_num haven't moved, only start_block_num go to the end block
+    //so could use the end.row_num for binary search
+    if (start.block_num == end.block_num) {
+        end_pos = end.row_num;
+    }
+    while (start_pos < end_pos) {
+        int64_t mid_pos = (start_pos + end_pos) >> 1;
+        if (start_column->compare_at(start_init_row_num, mid_pos, *start_column, 1)) {
+            end_pos = mid_pos;
+        } else {
+            start_pos = mid_pos + 1;
+        }
+    }
+    start.row_num = start_pos; //update row num, return the find end
+    return start;
+}
+
+vectorized::BlockRowPos AnalyticLocalState::_get_partition_by_end() {
+    auto& shared_state = *_shared_state;
+    if (shared_state.current_row_position <
+        shared_state.partition_by_end.pos) { //still have data, return partition_by_end directly
+        return shared_state.partition_by_end;
+    }
+
+    if (shared_state.partition_by_eq_expr_ctxs.empty() ||
+        (shared_state.input_total_rows == 0)) { //no partition_by, the all block is end
+        return shared_state.all_block_end;
+    }
+
+    vectorized::BlockRowPos cal_end = shared_state.all_block_end;
+    for (size_t i = 0; i < shared_state.partition_by_eq_expr_ctxs.size();
+         ++i) { //have partition_by, binary search the partiton end
+        cal_end = _compare_row_to_find_end(shared_state.partition_by_column_idxs[i],
+                                           shared_state.partition_by_end, cal_end);
+    }
+    cal_end.pos = shared_state.input_block_first_row_positions[cal_end.block_num] + cal_end.row_num;
+    return cal_end;
+}
+
+bool AnalyticLocalState::_whether_need_next_partition(

Review Comment:
   warning: method '_whether_need_next_partition' can be made static [readability-convert-member-functions-to-static]
   
   be/src/pipeline/exec/analytic_source_operator.h:96:
   ```diff
   -     bool _whether_need_next_partition(vectorized::BlockRowPos& found_partition_end);
   +     static bool _whether_need_next_partition(vectorized::BlockRowPos& found_partition_end);
   ```
   



##########
be/src/pipeline/exec/aggregation_source_operator.h:
##########
@@ -90,6 +108,24 @@ class AggLocalState final : public PipelineXLocalState<AggDependency> {
     Status _serialize_with_serialized_key_result_with_spilt_data(RuntimeState* state,
                                                                  vectorized::Block* block,
                                                                  SourceState& source_state);
+    Status _destroy_agg_status(vectorized::AggregateDataPtr data);
+    Status _reset_hash_table();
+    Status _merge_spilt_data();
+    void _make_nullable_output_key(vectorized::Block* block) {

Review Comment:
   warning: method '_make_nullable_output_key' can be made static [readability-convert-member-functions-to-static]
   
   ```suggestion
       static void _make_nullable_output_key(vectorized::Block* block) {
   ```
   



##########
be/src/pipeline/exec/hashjoin_build_sink.cpp:
##########
@@ -175,6 +175,68 @@ void HashJoinBuildSinkLocalState::init_short_circuit_for_probe() {
              p._join_op == TJoinOp::LEFT_ANTI_JOIN);
 }
 
+Status HashJoinBuildSinkLocalState::_do_evaluate(vectorized::Block& block,

Review Comment:
   warning: method '_do_evaluate' can be made static [readability-convert-member-functions-to-static]
   
   ```suggestion
   static Status HashJoinBuildSinkLocalState::_do_evaluate(vectorized::Block& block,
   ```
   



##########
be/src/pipeline/exec/data_queue.cpp:
##########
@@ -41,7 +41,8 @@
           _cur_bytes_in_queue(child_count),
           _cur_blocks_nums_in_queue(child_count),
           _flag_queue_idx(0),
-          _dependency(dependency) {
+          _source_dependency(nullptr),
+          _sink_dependency(nullptr) {

Review Comment:
   warning: member initializer for '_sink_dependency' is redundant [modernize-use-default-member-init]
   
   ```suggestion
             {
   ```
   



##########
be/src/pipeline/pipeline_x/dependency.h:
##########
@@ -222,33 +199,28 @@ class RuntimeFilterDependency final : public Dependency {
 public:
     RuntimeFilterDependency(int id, int node_id, std::string name)
             : Dependency(id, node_id, name) {}
-    RuntimeFilterDependency* filter_blocked_by(PipelineXTask* task);
+    Dependency* is_blocked_by(PipelineXTask* task);

Review Comment:
   warning: annotate this function with 'override' or (rarely) 'final' [modernize-use-override]
   
   ```suggestion
       Dependency* is_blocked_by(PipelineXTask* task) override;
   ```
   



##########
be/src/pipeline/exec/hashjoin_build_sink.cpp:
##########
@@ -175,6 +175,68 @@
              p._join_op == TJoinOp::LEFT_ANTI_JOIN);
 }
 
+Status HashJoinBuildSinkLocalState::_do_evaluate(vectorized::Block& block,
+                                                 vectorized::VExprContextSPtrs& exprs,
+                                                 RuntimeProfile::Counter& expr_call_timer,
+                                                 std::vector<int>& res_col_ids) {
+    for (size_t i = 0; i < exprs.size(); ++i) {
+        int result_col_id = -1;
+        // execute build column
+        {
+            SCOPED_TIMER(&expr_call_timer);
+            RETURN_IF_ERROR(exprs[i]->execute(&block, &result_col_id));
+        }
+
+        // TODO: opt the column is const
+        block.get_by_position(result_col_id).column =
+                block.get_by_position(result_col_id).column->convert_to_full_column_if_const();
+        res_col_ids[i] = result_col_id;
+    }
+    return Status::OK();
+}
+
+std::vector<uint16_t> HashJoinBuildSinkLocalState::_convert_block_to_null(
+        vectorized::Block& block) {
+    std::vector<uint16_t> results;
+    for (int i = 0; i < block.columns(); ++i) {
+        if (auto& column_type = block.safe_get_by_position(i); !column_type.type->is_nullable()) {
+            DCHECK(!column_type.column->is_nullable());
+            column_type.column = make_nullable(column_type.column);
+            column_type.type = make_nullable(column_type.type);
+            results.emplace_back(i);
+        }
+    }
+    return results;
+}
+
+Status HashJoinBuildSinkLocalState::_extract_join_column(

Review Comment:
   warning: method '_extract_join_column' can be made static [readability-convert-member-functions-to-static]
   
   ```suggestion
   static Status HashJoinBuildSinkLocalState::_extract_join_column(
   ```
   



##########
be/src/pipeline/exec/hashjoin_probe_operator.cpp:
##########
@@ -402,6 +402,48 @@ Status HashJoinProbeOperatorX::pull(doris::RuntimeState* state, vectorized::Bloc
     return Status::OK();
 }
 
+Status HashJoinProbeLocalState::_extract_join_column(vectorized::Block& block,

Review Comment:
   warning: method '_extract_join_column' can be made static [readability-convert-member-functions-to-static]
   
   ```suggestion
   static Status HashJoinProbeLocalState::_extract_join_column(vectorized::Block& block,
   ```
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1820961396

   TeamCity be ut coverage result:
    Function Coverage: 36.55% (8447/23108) 
    Line Coverage: 28.85% (68678/238035)
    Region Coverage: 27.82% (35522/127671)
    Branch Coverage: 24.57% (18117/73726)
    Coverage Report: http://coverage.selectdb-in.cc/coverage/2fe4b071609424618746b650fcad92335b222230_2fe4b071609424618746b650fcad92335b222230/report/index.html


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1821238033

   
   <details>
   <summary>TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'</summary>
   
   ```
   Tpch sf100 test result on commit 28d200c9a979a2d5cb7c1e9e0a742e4c870c8c77, data reload: false
   
   run tpch-sf100 query with default conf and session variables
   q1	4893	4700	4666	4666
   q2	355	183	162	162
   q3	2024	1905	1902	1902
   q4	1388	1243	1242	1242
   q5	3960	3938	3997	3938
   q6	249	138	132	132
   q7	1397	896	887	887
   q8	2733	2774	2744	2744
   q9	9720	9581	9541	9541
   q10	10244	3534	3515	3515
   q11	388	242	254	242
   q12	434	294	293	293
   q13	4538	3820	3780	3780
   q14	336	299	289	289
   q15	595	537	529	529
   q16	665	584	582	582
   q17	1118	981	967	967
   q18	7754	7482	7441	7441
   q19	1667	1656	1648	1648
   q20	538	306	289	289
   q21	4349	3925	3948	3925
   q22	473	378	383	378
   Total cold run time: 59818 ms
   Total hot run time: 49092 ms
   
   run tpch-sf100 query with default conf and set session variable runtime_filter_mode=off
   q1	4598	4607	4571	4571
   q2	340	237	317	237
   q3	3995	3974	3970	3970
   q4	2706	2697	2691	2691
   q5	9582	9570	9557	9557
   q6	240	121	128	121
   q7	3002	2488	2504	2488
   q8	4410	4416	4413	4413
   q9	13157	13077	12937	12937
   q10	4089	4194	4171	4171
   q11	805	661	675	661
   q12	970	816	804	804
   q13	4283	3555	3543	3543
   q14	375	352	368	352
   q15	576	516	527	516
   q16	732	669	654	654
   q17	3900	3818	3852	3818
   q18	9536	8933	8993	8933
   q19	1786	1776	1754	1754
   q20	2387	2108	2057	2057
   q21	8676	8661	8612	8612
   q22	884	842	762	762
   Total cold run time: 81029 ms
   Total hot run time: 77622 ms
   ```
   </details>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "Gabriel39 (via GitHub)" <gi...@apache.org>.
Gabriel39 commented on PR #27366:
URL: https://github.com/apache/doris/pull/27366#issuecomment-1821120469

   run buildall


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [pipelineX](dependency) split different dependencies [doris]

Posted by "Gabriel39 (via GitHub)" <gi...@apache.org>.
Gabriel39 merged PR #27366:
URL: https://github.com/apache/doris/pull/27366


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org