You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/12/06 12:48:29 UTC

[GitHub] [airflow] potiuk commented on pull request #20069: Add migration-wait-timeout to the helm chart

potiuk commented on pull request #20069:
URL: https://github.com/apache/airflow/pull/20069#issuecomment-986744395


   Question - what woudl be the cases where timing out on migration is "wrong" ? Woudl not that leave the DB in a state that is not expected ?  However I think that one is fine to merge as long as we merge https://github.com/apache/airflow/pull/18439 - because then instead of "hanging" migration you'd get hard failure  (which is much better).
   
   Is that the idea behind it ?
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org