engine: config still loaded N+1 times in run_interval + run_job #121

New issue

Closed

opened 2026-07-05 15:52:49 +00:00 by the.auditor · 0 comments

the.auditor commented

2026-07-05 15:52:49 +00:00

Owner

load_config() is called in both run_interval() and run_job(). For an interval of N jobs, config is loaded N+1 times since notifications.py now needs the config in run_job().

Consider passing the config from run_interval() to run_job() to eliminate redundant loads.

From PR #118 review suggestion.

`load_config()` is called in both `run_interval()` and `run_job()`. For an interval of N jobs, config is loaded N+1 times since `notifications.py` now needs the config in `run_job()`. Consider passing the config from `run_interval()` to `run_job()` to eliminate redundant loads. _From PR #118 review suggestion._

the.auditor added the

Kind/Enhancement

Priority

Low

labels

2026-07-05 15:52:54 +00:00