potiuk commented on PR #56187: URL: https://github.com/apache/airflow/pull/56187#issuecomment-3570033969
> Yeah basically split the methods into factory (core) + actuals (shared), factory uses `conf` Yeah. Basically "divide and conquer" is a classic way of dealing with such issues. If something is standing in the way of isolation, split it into smaller, independent pieces that can be shared and bound together in the "user" of the shared code. Basically leave control code outside of the shared code (and possibly partially WET rather than DRY. This way you can "pass" conf from outside to the shared code that is common - it's more of "injecting" configuration in this case. -- 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]
