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
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
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
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
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
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
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
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
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