shahar1 commented on code in PR #48419: URL: https://github.com/apache/airflow/pull/48419#discussion_r2015024607
########## airflow-core/newsfragments/48419.significant.rst: ########## @@ -0,0 +1,18 @@ +Unused attribute ``num_times_parse_dags`` of SchedulerJobRunner has been removed. + +This change removes an unused attribute from the SchedulerJobRunner class to clean up the codebase. + +* Types of change + + * [ ] Dag changes + * [ ] Config changes + * [ ] API changes + * [ ] CLI changes + * [ ] Behaviour changes + * [ ] Plugin changes + * [ ] Dependency changes + * [x] Code interface changes + +* Migration rules needed +.. e.g., +.. * Attribute ``num_times_parse_dags`` removed from ``airflow.jobs.scheduler_job.SchedulerJobRunner`` Review Comment: Not entirely sure that we need a migration rule, as I highly doubt that someone actually imports the SchedulerJobRunner directly (unless they actually customize Airflow). Kept it commented out for now. -- 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]
