eldenmoon commented on PR #70487:
URL: https://github.com/apache/doris/pull/70487#issuecomment-6813586054
<!-- doris-repo-review:v1:begin -->
### Local pipeline review — ✅ PASS
```yaml
schema: doris-repo-review/v1
status: PASS
pr: apache/doris#70487
commit: de768c17b5df176a2c49bd5af736e2636167ce36
base: 9e598d38789571a7fc30652aa054f035bd4f1184
reviewed_at: 2026-11-12T11:27+08:00
reviewer: eldenmoon
model: gpt-6-astra
effort: xhigh
findings: {blocker: 0, major: 0, minor: 0, nit: 0}
rounds: 1
converged: true
```
**Notes for maintainers**
- Read-only source review of the full six-commit stack, with the final
metadata refactor distinguished from the five commits inherited from #70481. No
builds, tests, sanitizers, or benchmarks were run in this review;
author-reported results were not independently reproduced.
- `be/src/core/value/variant/variant_batch_builder.cpp:220` — key-owner
capacity grows geometrically. Old/new descriptor arrays coexist during growth,
and abort retains descriptor capacity until batch finish/destruction; this
review does not claim universally lower peak memory or a reproduced ASAN
speedup.
- The original review scripts and document verifier were used with
authenticated GitHub connector read snapshots in place of an unauthenticated
native gh session. This receipt was rendered with --dry-run for copying; no
GitHub comment or review was submitted.
<sub>Reviewed locally with the `doris-repo-review` pipeline. Repository
policy may accept this receipt for the matching commit; it is not a human
Apache approval.</sub>
<!-- doris-repo-review:v1:end -->
--
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]
