potiuk commented on PR #31829: URL: https://github.com/apache/airflow/pull/31829#issuecomment-1860904745
Just o repeat there - there was a comment by @hussein-awala, and it waits for @thesuperzapper to responds to it. Also there was my request to make a bit more a description of how the whole interaction of with the Celery looks like (sequence diagrams) while we are at it and want to make sure we know what we are doing with tasks_acks_late. This would be quite useful, espeically that we recently integrated a nice `diagrams` tool which could allow to have a nice, generated diagram with visualisation of what's going on in case of Celery (or we could use mermaid sequence diagrams as well). More links here: * https://github.com/apache/airflow/blob/main/docs/apache-airflow/img/diagram_basic_airflow_architecture.png * https://github.com/apache/airflow/blob/main/docs/apache-airflow/img/diagram_dag_processor_airflow_architecture.png Nicely generated with this pre-commit: https://github.com/apache/airflow/blob/main/scripts/ci/pre_commit/pre_commit_generate_airflow_diagrams.py or those sequence diagrams rendered automatically by GitHub from embedded mermaid diagram sources: https://github.com/apache/airflow/blob/main/CI_DIAGRAMS.md @thesuperzapper - are you working on it ? What's your take here? -- 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]
