uranusjr commented on PR #71815: URL: https://github.com/apache/airflow/pull/71815#issuecomment-5352445267
Instead of an argument in DBDagBag to make it double-duty, I feel it’s cleaner if we make the `_use_cache=True` variant a separate subclass i.e. `class CachedDBDagBag`. This way, the arguments will be cleaner (no more complicated optional arguments + checks in `__init__`, and the stats functions only need to be on the class they’re actually needed. -- 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]
