mrhhsg commented on PR #63528: URL: https://github.com/apache/doris/pull/63528#issuecomment-6271844311
<!-- autopilot-response --> Re-audited the full paginated review history at head `a073a018f32`. - Review threads: all 20 review threads are resolved. The latest static review (5572248555) reports no new verified finding and an empty blocking set. - Remaining P2 note: the memory-accounting concern in the existing thread (discussion_r4257770328) was answered by the author as out of scope for this PR, and the latest review explicitly does not repost it; no further change is made here. - Conflicts: none remain. `git merge-tree --write-tree HEAD origin/master` is clean against `origin/master` (`41a58afba2e`), and GitHub reports `mergeable: true`. The `blocked` merge state comes from the pending review decision, not from a conflict. No history rewrite was performed. - Verification on this head: FE UT (`ColumnTest`, `ComplexTypeMappingLoadTest`, `NereidsLoadScanProviderTest`, `AlterTableCommandTest`, `ColumnDefinitionTest`: 74 tests, 0 failures), BE UT (9 suites, 251 tests, all passed), and local regression `datatype_p0/complex_types/test_complex_default_value` passed. No source change was needed, so the head is unchanged at `a073a018f32`. The review decision is still `CHANGES_REQUESTED`; a human re-review is 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: [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]
