goutamadwant commented on PR #11963:
URL: https://github.com/apache/seatunnel/pull/11963#issuecomment-5418837033

   thanks @nzw921rx  merge queue approach looks great. just couple of things to 
confirm:
   
   - I see the job timeout is 30 minutes, but the configured retries can take 
longer, so some retries may never run ? 
   - I do not see the primary merge queue build command listed in the local 
verification. The current green build appears to be from the existing CI 
workflow. Could you confirm whether the new command was tested separately? or 
any plan?
   
   Could we run the exact command once and adjust the timeout based on its 
duration? Let me know thanks! but great plan. 


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

Reply via email to