shuke987 opened a new pull request, #69063:
URL: https://github.com/apache/doris/pull/69063
### What problem does this PR solve?
Issue Number: None
Related PR: #69008, #69013
Routine Code Review Runner telemetry still downloads trace IO after each
upload. After the duplicate-read fix, 17 completed runs on October 14 averaged
26.2 seconds for the telemetry step and 12.2 seconds for uploads. All 110
traces across the day's 28 completed exports passed readback verification.
Remove `--verify` and its redundant default polling options from the
workflow invocation to avoid routine readback cost. Keep the exporter unchanged
so `--verify` remains available for targeted diagnostics. Gzip, upload
response/partial-success checks, retries, and `continue-on-error: true` remain
in place. Routine CI will check upload acknowledgement but no longer assert
downstream queryable IO completeness.
### Release note
None
### Check List (For Author)
- Test
- [x] Unit Test: all 44 existing exporter tests pass, including gzip
failure, OTLP partial-success, and verification coverage.
- [x] Manual/static validation: YAML parsing and assertions confirm
verify is disabled, gzip is enabled, telemetry remains non-blocking, and the
five-minute step timeout is preserved; diff whitespace check passes.
- [x] Workflow lint: actionlint v1.7.7 passes with ShellCheck/Pyflakes
disabled and only its known unsupported `concurrency.queue` diagnostic ignored.
- Behavior changed:
- [x] Yes. Routine CI telemetry uploads without automatic readback;
explicit diagnostic verification remains supported.
- Does this need documentation?
- [x] No. An inline workflow comment explains the diagnostic opt-in.
Review: one workflow changes; the existing invocation handles both main and
subagent uploads. No exporter, submission, concurrency, credential, or
persistence logic changes. Expected savings are the removed readback work, not
a guaranteed total duration. No live upload or workflow dispatch was performed
during this validation.
### 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]
