You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "Mryange (via GitHub)" <gi...@apache.org> on 2024/02/21 03:29:15 UTC
[PR] [refine](pipelinex) get sink local state does not require an id. [doris]
Mryange opened a new pull request, #31195:
URL: https://github.com/apache/doris/pull/31195
## Proposed changes
The current runtime state is one pipeline task per task, and within one task, there will only be one sink.
<!--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] [refine](pipelinex) get sink local state does not require an id. [doris]
Posted by "Mryange (via GitHub)" <gi...@apache.org>.
Mryange commented on PR #31195:
URL: https://github.com/apache/doris/pull/31195#issuecomment-1955815046
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] [refine](pipelinex) get sink local state does not require an id. [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #31195:
URL: https://github.com/apache/doris/pull/31195#issuecomment-1957091729
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] [refine](pipelinex) get sink local state does not require an id. [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #31195:
URL: https://github.com/apache/doris/pull/31195#issuecomment-1955814997
Thank you for your contribution to Apache Doris.
Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR)
--
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] [refine](pipelinex) get sink local state does not require an id. [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #31195:
URL: https://github.com/apache/doris/pull/31195#issuecomment-1955829176
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] [refine](pipelinex) get sink local state does not require an id. [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #31195:
URL: https://github.com/apache/doris/pull/31195#issuecomment-1955858403
<details>
<summary>TPC-H: <b>Total hot run time: 41519 ms</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
Tpch sf100 test result on commit 25947d86b7d730e8cf36cdc43849da63072e681d, data reload: false
------ Round 1 ----------------------------------
q1 17663 4990 4874 4874
q2 2037 140 134 134
q3 10574 1059 1008 1008
q4 4645 968 982 968
q5 7693 3278 3303 3278
q6 191 138 139 138
q7 1254 780 788 780
q8 9240 2082 2095 2082
q9 7610 6691 6686 6686
q10 8304 2648 2653 2648
q11 409 224 224 224
q12 762 330 320 320
q13 17995 3704 3671 3671
q14 293 256 258 256
q15 584 515 502 502
q16 495 406 413 406
q17 929 848 838 838
q18 7525 6685 6586 6586
q19 1570 1507 1498 1498
q20 633 359 360 359
q21 6168 3920 3993 3920
q22 887 368 343 343
Total cold run time: 107461 ms
Total hot run time: 41519 ms
----- Round 2, with runtime_filter_mode=off -----
q1 4837 4829 4821 4821
q2 294 193 182 182
q3 3585 3603 3581 3581
q4 2554 2558 2534 2534
q5 5743 5733 5764 5733
q6 210 129 135 129
q7 2249 1673 1683 1673
q8 3015 3105 3114 3105
q9 8714 8680 8674 8674
q10 6783 4222 4227 4222
q11 530 439 410 410
q12 778 556 557 556
q13 6877 3424 3405 3405
q14 256 254 250 250
q15 606 536 529 529
q16 495 458 439 439
q17 1683 1607 1643 1607
q18 8289 7707 7684 7684
q19 1648 1635 1645 1635
q20 2105 1831 1829 1829
q21 6742 6311 6270 6270
q22 588 516 513 513
Total cold run time: 68581 ms
Total hot run time: 59781 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] [refine](pipelinex) get sink local state does not require an id. [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #31195:
URL: https://github.com/apache/doris/pull/31195#issuecomment-1955875552
TeamCity be ut coverage result:
Function Coverage: 35.72% (8558/23959)
Line Coverage: 27.60% (69418/251540)
Region Coverage: 26.73% (36020/134746)
Branch Coverage: 23.55% (18426/78238)
Coverage Report: http://coverage.selectdb-in.cc/coverage/25947d86b7d730e8cf36cdc43849da63072e681d_25947d86b7d730e8cf36cdc43849da63072e681d/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] [refine](pipelinex) get sink local state does not require an id. [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #31195:
URL: https://github.com/apache/doris/pull/31195#issuecomment-1957091560
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] [refine](pipelinex) get sink local state does not require an id. [doris]
Posted by "yiguolei (via GitHub)" <gi...@apache.org>.
yiguolei merged PR #31195:
URL: https://github.com/apache/doris/pull/31195
--
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