BasPH commented on PR #36483:
URL: https://github.com/apache/airflow/pull/36483#issuecomment-1872078454

   I agree with explaining monitoring of Airflow's meta-database in the docs. 
However, this chapter contains a lot of (IMO) unnecessary details which makes 
it difficult to extract the key information.
   
   How about restructuring it as:
   - 1 sentence explaining "everything you see and do in Airflow is registered 
in the database, therefore it's important to monitor"
   - 1 sentence with disclaimer that monitoring the database is up to yourself, 
not part of Airflow
   - 1 paragraph stating "Here are a few key considerations when monitoring 
your Airflow database" with the bulletpoints from your text
   - 1 paragraph explaining how to enable query logging together with the "why" 
and a warning that it can have a performance impact


-- 
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-unsubscr...@airflow.apache.org

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

Reply via email to