github-actions[bot] commented on code in PR #66272:
URL: https://github.com/apache/doris/pull/66272#discussion_r3682008771


##########
.github/workflows/code-review-runner.yml:
##########
@@ -708,3 +811,35 @@ jobs:
           fi
           echo "Codex automated review failed: ${error_msg}"
           exit 1
+
+      - name: Sync Code Review check for current head

Review Comment:
   [P1] Reserve a finalization window before cleanup
   
   This status write is not guaranteed to run within the declared time budget. 
The job is capped at 120 minutes, `Run automated code review` can consume 115 
minutes only after setup, and Litefuse then has another 5 minutes before the 
session/auth network syncs, which have no step timeout. Even with zero setup, 
the two bounded steps can exhaust the whole job budget, so a review that 
successfully submits and verifies near its limit can be canceled during cleanup 
and leave the required `code-review` context pending indefinitely. `always()` 
controls conditional scheduling but cannot extend the job lifetime. Please move 
the terminal status update ahead of best-effort cleanup, bound the remaining 
network steps, and shorten the review timeout enough to reserve finalization 
time.
   
   



-- 
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]

Reply via email to