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/19 04:44:50 UTC

[PR] [pipelineX](dependency) remove OrDependency [doris]

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

   ## 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) remove OrDependency [doris]

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

   
   <details>
   <summary>TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'</summary>
   
   ```
   Tpch sf100 test result on commit 8ddf504981e8fe1cd09083bd75f6c21249afd076, data reload: false
   
   run tpch-sf100 query with default conf and session variables
   q1	4875	4622	4627	4622
   q2	356	152	159	152
   q3	2056	1957	1896	1896
   q4	1381	1267	1262	1262
   q5	3872	3950	4027	3950
   q6	254	126	128	126
   q7	1447	875	894	875
   q8	2753	2753	2738	2738
   q9	9535	9503	9395	9395
   q10	3485	3510	3522	3510
   q11	381	248	248	248
   q12	431	287	287	287
   q13	4558	3761	3808	3761
   q14	321	310	283	283
   q15	586	523	524	523
   q16	667	592	584	584
   q17	1115	977	931	931
   q18	7917	7539	7592	7539
   q19	1650	1633	1635	1633
   q20	549	325	313	313
   q21	4399	3946	4005	3946
   q22	475	372	376	372
   Total cold run time: 53063 ms
   Total hot run time: 48946 ms
   
   run tpch-sf100 query with default conf and set session variable runtime_filter_mode=off
   q1	4580	4558	4551	4551
   q2	336	263	246	246
   q3	4038	4006	4007	4006
   q4	2709	2708	2707	2707
   q5	9676	9659	9559	9559
   q6	252	122	120	120
   q7	3026	2455	2501	2455
   q8	4469	4448	4451	4448
   q9	12840	12783	12800	12783
   q10	4066	4123	4145	4123
   q11	821	639	680	639
   q12	977	815	810	810
   q13	4275	3588	3572	3572
   q14	399	366	348	348
   q15	583	510	518	510
   q16	730	680	680	680
   q17	3880	3841	3840	3840
   q18	9728	9203	9142	9142
   q19	1808	1758	1769	1758
   q20	2391	2051	2058	2051
   q21	8773	8605	8807	8605
   q22	920	766	794	766
   Total cold run time: 81277 ms
   Total hot run time: 77719 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) remove OrDependency [doris]

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

   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) remove OrDependency [doris]

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

   clang-tidy review says "All clean, LGTM! :+1:"


-- 
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) remove OrDependency [doris]

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

   TeamCity be ut coverage result:
    Function Coverage: 36.53% (8450/23134) 
    Line Coverage: 28.88% (68687/237866)
    Region Coverage: 27.85% (35522/127553)
    Branch Coverage: 24.60% (18112/73612)
    Coverage Report: http://coverage.selectdb-in.cc/coverage/8ddf504981e8fe1cd09083bd75f6c21249afd076_8ddf504981e8fe1cd09083bd75f6c21249afd076/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) remove OrDependency [doris]

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

   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) remove OrDependency [doris]

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


-- 
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) remove OrDependency [doris]

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

   clang-tidy review says "All clean, LGTM! :+1:"


-- 
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) remove OrDependency [doris]

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

   clang-tidy review says "All clean, LGTM! :+1:"


-- 
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) remove OrDependency [doris]

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

   
   <details>
   <summary>TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'</summary>
   
   ```
   Tpch sf100 test result on commit 85329b399eab77051f81ea1764a149e5cd603fb0, data reload: false
   
   run tpch-sf100 query with default conf and session variables
   q1	4921	4647	4643	4643
   q2	355	151	158	151
   q3	2029	1915	1867	1867
   q4	1384	1262	1232	1232
   q5	3918	3924	3989	3924
   q6	242	132	131	131
   q7	1432	894	893	893
   q8	2775	2760	2737	2737
   q9	9586	9419	9378	9378
   q10	3456	3474	3515	3474
   q11	395	249	259	249
   q12	438	292	295	292
   q13	4556	3802	3758	3758
   q14	315	283	298	283
   q15	574	543	516	516
   q16	659	599	579	579
   q17	1125	977	897	897
   q18	7890	7629	7365	7365
   q19	1641	1676	1668	1668
   q20	544	328	298	298
   q21	4432	4005	4016	4005
   q22	478	375	380	375
   Total cold run time: 53145 ms
   Total hot run time: 48715 ms
   
   run tpch-sf100 query with default conf and set session variable runtime_filter_mode=off
   q1	4575	4581	4583	4581
   q2	353	219	254	219
   q3	4031	4012	3997	3997
   q4	2725	2717	2719	2717
   q5	9584	9596	9558	9558
   q6	251	124	124	124
   q7	3012	2508	2495	2495
   q8	4445	4412	4424	4412
   q9	12913	12786	12824	12786
   q10	4008	4151	4160	4151
   q11	763	652	724	652
   q12	970	802	806	802
   q13	4297	3572	3560	3560
   q14	398	354	334	334
   q15	567	511	515	511
   q16	736	668	663	663
   q17	3861	3910	3949	3910
   q18	9697	9135	9209	9135
   q19	1809	1781	1757	1757
   q20	2385	2065	2060	2060
   q21	8828	8520	8680	8520
   q22	902	783	770	770
   Total cold run time: 81110 ms
   Total hot run time: 77714 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) remove OrDependency [doris]

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

   (From new machine)TeamCity pipeline, clickbench performance test result:
    the sum of best hot time: 44.81 seconds
    stream load tsv:          567 seconds loaded 74807831229 Bytes, about 125 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.6 seconds inserted 10000000 Rows, about 349K ops/s
    storage size: 17100532582 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) remove OrDependency [doris]

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

   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) remove OrDependency [doris]

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

   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) remove OrDependency [doris]

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

   TeamCity be ut coverage result:
    Function Coverage: 36.77% (8434/22937) 
    Line Coverage: 29.26% (68621/234492)
    Region Coverage: 27.87% (35480/127308)
    Branch Coverage: 24.60% (18104/73596)
    Coverage Report: http://coverage.selectdb-in.cc/coverage/85329b399eab77051f81ea1764a149e5cd603fb0_85329b399eab77051f81ea1764a149e5cd603fb0/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) remove OrDependency [doris]

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

   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) remove OrDependency [doris]

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

   (From new machine)TeamCity pipeline, clickbench performance test result:
    the sum of best hot time: 44.56 seconds
    stream load tsv:          567 seconds loaded 74807831229 Bytes, about 125 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.6 seconds inserted 10000000 Rows, about 349K ops/s
    storage size: 17099325416 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