Gabriel39 opened a new pull request, #68597:
URL: https://github.com/apache/doris/pull/68597
### What problem does this PR solve?
Related PR: #66941
`test_iceberg_variant_read` can fail its STRING leaf projection assertion
after the SQL result has already passed the 4,096-row check. The profile fetch
only requires counter names to be present, which does not ensure their final
values have arrived.
In the observed failure, the profile HTTP request started at `03:16:04.601`,
overlapping the final BE report processed at `03:16:04.604`. The archived
profile subsequently contained `VariantLeafProjectionRowGroupColumns = 1`,
`VariantDirectLeafRows = 8192`, and zero residual/full projection and
reconstruction counters.
Reuse the suite's bounded positive-counter wait for the STRING query. Allow
the helper to also require the counters expected to remain zero, preserving
every existing assertion and presence check. Permanently zero positive counters
still time out with counter diagnostics; unexpected reconstruction still fails.
This targets `branch-4.1`, where the affected STRING assertion exists.
### Release note
None
### Check List (For Author)
- Test
- [x] Manual test: ran a Groovy 4 harness that executes the suite's
actual parsing/wait closures and STRING assertions against deterministic
profile snapshots. The delayed-counter sequence reproduced the original
assertion failure before the fix and passed afterward. Also verified an
already-ready profile, a permanent-zero timeout, and rejection of nonzero
reconstruction. This is a focused replay, not an end-to-end cluster run.
- [x] Manual test: full Groovy suite syntax check and `git diff --check`
passed.
- Full external regression was not run locally; CI validation is pending.
- Behavior changed:
- [x] No production behavior change; only regression profile
synchronization changes.
- Does this need documentation?
- [x] No.
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label
--
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]
