gingeekrishna commented on issue #66796: URL: https://github.com/apache/airflow/issues/66796#issuecomment-4548592297
Opened PR #67572 which implements the fix. **Key changes:** - Adds `SerializedDagModel.get_count()` using `scalar_one()` instead of `scalar() or 0`, so a `None` result raises rather than silently returning 0 - Emits a `serialized_dag.count` gauge from `emit_metrics()` in `dag_processing/manager.py`, gated behind `scheduler.emit_serialized_dag_count_metric` (default `True`) for deployments that want to opt out of the extra DB round-trip - On DB failure, logs the error and increments a `serialized_dag.count_error` counter so dashboards can alert on missing samples rather than showing a stale value - Full unit test coverage for all branches including the disabled-config path -- 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]
