wenzhenghu commented on PR #68694: URL: https://github.com/apache/doris/pull/68694#issuecomment-6162047294
Review follow-up for [the inline review](https://github.com/apache/doris/pull/68694#pullrequestreview-5497912185): all 12 discussions have individual replies and resolution dispositions. The update fixes long-log chunk truncation, cleans servlet/transient-worker boundaries, removes Streaming context pre-installation, adds the LIFO assertion/direct includes, and avoids empty idle-worker MDC scopes. Flight convenience APIs were removed; the protobuf suffix overload remains because `ResultReceiver` uses it. The internal RPC scope parameter and separate preparation-boundary resets were retained with code-based explanations. Both configuration defaults remain unchanged. Validation of the review-fix code included 138 independent FE tests, 27 ASAN BE query-log tests, and runtime FE/BE correlation for 34 query IDs. Those tests preceded the later master merge; I refreshed the merged-head source for these replies, without claiming a new full run there. Cross-PR landing adaptation and the separate documentation PR remain explicit follow-ups. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
