Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-26 Thread via GitHub
yiguolei merged PR #35399: URL: https://github.com/apache/doris/pull/35399 -- 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-unsubscr...@doris.apa

Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-25 Thread via GitHub
github-actions[bot] commented on PR #35399: URL: https://github.com/apache/doris/pull/35399#issuecomment-2132080724 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

Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-25 Thread via GitHub
doris-robot commented on PR #35399: URL: https://github.com/apache/doris/pull/35399#issuecomment-2132079086 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+y

[PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-25 Thread via GitHub
Gabriel39 opened a new pull request, #35399: URL: https://github.com/apache/doris/pull/35399 ## Proposed changes pick #35329 ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto:d...@doris.

Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-25 Thread via GitHub
yiguolei merged PR #35329: URL: https://github.com/apache/doris/pull/35329 -- 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-unsubscr...@doris.apa

Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-25 Thread via GitHub
github-actions[bot] commented on PR #35329: URL: https://github.com/apache/doris/pull/35329#issuecomment-2132046125 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 UR

Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-25 Thread via GitHub
github-actions[bot] commented on PR #35329: URL: https://github.com/apache/doris/pull/35329#issuecomment-2132046118 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 GitHu

Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-25 Thread via GitHub
yiguolei commented on code in PR #35329: URL: https://github.com/apache/doris/pull/35329#discussion_r1614688093 ## be/src/pipeline/exec/exchange_sink_operator.cpp: ## @@ -155,12 +155,17 @@ Status ExchangeSinkLocalState::open(RuntimeState* state) { size_t dep_id = 0;

Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-23 Thread via GitHub
doris-robot commented on PR #35329: URL: https://github.com/apache/doris/pull/35329#issuecomment-2128473266 ClickBench: Total hot run time: 30.03 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools

Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-23 Thread via GitHub
doris-robot commented on PR #35329: URL: https://github.com/apache/doris/pull/35329#issuecomment-2128469248 TPC-DS: Total hot run time: 168453 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS

Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-23 Thread via GitHub
doris-robot commented on PR #35329: URL: https://github.com/apache/doris/pull/35329#issuecomment-2128460791 TeamCity be ut coverage result: Function Coverage: 35.65% (9017/25294) Line Coverage: 27.31% (74567/273054) Region Coverage: 26.54% (38602/145432) Branch Coverage: 23.4

Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-23 Thread via GitHub
doris-robot commented on PR #35329: URL: https://github.com/apache/doris/pull/35329#issuecomment-2128460236 TPC-H: Total hot run time: 40032 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100

Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-23 Thread via GitHub
github-actions[bot] commented on PR #35329: URL: https://github.com/apache/doris/pull/35329#issuecomment-2128434199 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

Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-23 Thread via GitHub
Gabriel39 commented on PR #35329: URL: https://github.com/apache/doris/pull/35329#issuecomment-2128429899 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 un

Re: [PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-23 Thread via GitHub
doris-robot commented on PR #35329: URL: https://github.com/apache/doris/pull/35329#issuecomment-2128429862 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+y

[PR] [fix](shuffle) Do not return error if local recvr is null [doris]

2024-05-23 Thread via GitHub
Gabriel39 opened a new pull request, #35329: URL: https://github.com/apache/doris/pull/35329 ## Proposed changes errCode = 2, detailMessage = (172.30.0.248)[INTERNAL_ERROR]_local_recvr is null: 27 If downstream exchange operator finish earlier (e.g. if limited rows has reached