Re: [PR] [Bug](sink) update error status on BufferControlBlock::close [doris]

2024-09-03 Thread via GitHub
BiteThet merged PR #40116: URL: https://github.com/apache/doris/pull/40116 -- 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

Re: [PR] [Bug](sink) update error status on BufferControlBlock::close [doris]

2024-09-03 Thread via GitHub
github-actions[bot] commented on PR #40116: URL: https://github.com/apache/doris/pull/40116#issuecomment-2325897377 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] [Bug](sink) update error status on BufferControlBlock::close [doris]

2024-09-03 Thread via GitHub
github-actions[bot] commented on PR #40116: URL: https://github.com/apache/doris/pull/40116#issuecomment-2325897442 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] [Bug](sink) update error status on BufferControlBlock::close [doris]

2024-09-01 Thread via GitHub
github-actions[bot] commented on PR #40116: URL: https://github.com/apache/doris/pull/40116#issuecomment-2323650405 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] [Bug](sink) update error status on BufferControlBlock::close [doris]

2024-09-01 Thread via GitHub
BiteThet commented on PR #40116: URL: https://github.com/apache/doris/pull/40116#issuecomment-2323644999 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

Re: [PR] [Bug](sink) update error status on BufferControlBlock::close [doris]

2024-08-30 Thread via GitHub
yiguolei commented on code in PR #40116: URL: https://github.com/apache/doris/pull/40116#discussion_r1739597860 ## be/src/runtime/buffer_control_block.cpp: ## @@ -248,6 +248,10 @@ Status BufferControlBlock::get_arrow_batch(std::shared_ptr* Status BufferControlBlock::close(co

Re: [PR] [Bug](sink) update error status on BufferControlBlock::close [doris]

2024-08-29 Thread via GitHub
BiteThet commented on PR #40116: URL: https://github.com/apache/doris/pull/40116#issuecomment-2317098680 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

Re: [PR] [Bug](sink) update error status on BufferControlBlock::close [doris]

2024-08-29 Thread via GitHub
github-actions[bot] commented on PR #40116: URL: https://github.com/apache/doris/pull/40116#issuecomment-2317082613 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

[PR] [Bug](sink) update error status on BufferControlBlock::close [doris]

2024-08-29 Thread via GitHub
BiteThet opened a new pull request, #40116: URL: https://github.com/apache/doris/pull/40116 ## Proposed changes update error status on BufferControlBlock::close -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th