yiguolei commented on PR #70302:
URL: https://github.com/apache/doris/pull/70302#issuecomment-6856545643
<!-- doris-repo-review:v1:begin -->
### Local pipeline review — ✅ PASS
```yaml
schema: doris-repo-review/v1
status: PASS
pr: apache/doris#70302
commit: b1ed31c503f1d3a861340a05803d0cbf19ed9024
base: e82d2650e38e57886679faecf0ad5569169c54aa
reviewed_at: 2026-11-13T22:54+08:00
reviewer: yiguolei
model: gpt-6-astra
effort: xhigh
findings: {blocker: 0, major: 0, minor: 0, nit: 0}
rounds: 1
converged: true
```
**Notes for maintainers**
- The previous unit-test construction blocker is fixed at
[`function_geo_test.cpp:398-417`](https://github.com/apache/doris/blob/b1ed31c503f1d3a861340a05803d0cbf19ed9024/be/test/exprs/function/function_geo_test.cpp#L398-L417):
each constant now has one nested row and N logical rows. The same runner
covers `st_contains`, `st_within`, invalid constants and the A,A,B,A
across-block sequence.
- Re-reviewed all six changed files and the constant/NULL wrappers, context
clone/open ownership, edge tolerance and cache invalidation. No new actionable
findings; one review round converged with complete file coverage.
- Validation: source review and pure-text build-hygiene checks (passed). No
build, unit/regression-test execution, benchmark or cluster run was performed
in this review; the author's test results were not independently reproduced.
<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]
