mrhhsg commented on PR #69288: URL: https://github.com/apache/doris/pull/69288#issuecomment-6281616734
<!-- autopilot-response --> Reviewed the current review state at head `be394b4675c2df72f939406ee66e724d52b195c2`. - `PRR_kwDOBfSlxs8AAAABTBibMA` (Code Review, COMMENTED): the review reports no actionable findings. It confirms that the current object is already open and waiting for a key, that removing the extra start/close pair leaves one matching close so enclosing objects can emit their remaining members, and that the added BE rows and regression cases cover the reported failure plus empty/non-empty/deeper objects, sibling ordering, mixed sibling types, empty keys and the array-parent control. No unresolved candidate remains. - `IC_kwDOBfSlxs8AAAABdY8YwQ` (contribution checklist prompt): the PR description covers all five requested points, as noted in the previous round. - `IC_kwDOBfSlxs8AAAABdY8eBA` (`/review`): the review has now completed for this head with no findings, so there is nothing to fix from this comment. - `IC_kwDOBfSlxs8AAAABdZFJQA` (`run buildall`) and the following TPC-H / TPC-DS / ClickBench reports (`IC_kwDOBfSlxs8AAAABdbU2xA`, `IC_kwDOBfSlxs8AAAABdbhfWQ`, `IC_kwDOBfSlxs8AAAABdbnpOw`): benchmark runs for this head, informational only; no code action required. There are no unresolved inline review threads. No code changes were made in this round and the branch head is unchanged. Could a human reviewer take another look and approve if the change looks good? -- 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]
