ephraimbuddy opened a new pull request, #71720:
URL: https://github.com/apache/airflow/pull/71720

   The Dag processor's manager issues these statements once per parsed file, 
and how many was written down nowhere, so a change could add round trips to 
that path unnoticed.
   
   Measure it: a call costs 10 statements per file plus 3 per Dag the file 
defines, and a sweep pays that fixed price once per persistence call -- 
currently one call per file. Those counts are calibrated against Postgres, and 
the tests carrying them are marked for it, since statement counts differ by 
dialect.
   
   The sweep cost is measured through _collect_results and derives both prices 
from its own measurements, so it carries no dialect-specific number, runs on 
any backend, and tracks whatever strategy the manager uses.
   
   Timing this proved unreliable: on a throttled single-core container, repeat 
runs of identical code vary by more than a third, far larger than the savings 
under consideration. Counting statements is exact.
   
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [x] Yes (please specify the tool below)
   
   


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