Bowrna commented on code in PR #38400:
URL: https://github.com/apache/airflow/pull/38400#discussion_r1537109916


##########
docs/apache-airflow/administration-and-deployment/logging-monitoring/metrics.rst:
##########
@@ -219,6 +219,7 @@ Name                                                
Description
 =================================================== 
========================================================================
 ``dagbag_size``                                     Number of DAGs found when 
the scheduler ran a scan based on its
                                                     configuration
+``dag_processing.processes_count``                  Number of currently 
running DAG parsing processes
 ``dag_processing.import_errors``                    Number of errors from 
trying to parse DAG files
 ``dag_processing.total_parse_time``                 Seconds taken to scan and 
import ``dag_processing.file_path_queue_size`` DAG files

Review Comment:
   @potiuk @ferruzzi @hussein-awala Do you think param like 
``dag_processing.total_parse_time` need to be under the metrics `gauges`? This 
is not a docs issue, but in the code this metric is set as gauge value



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