ashb commented on PR #53435: URL: https://github.com/apache/airflow/pull/53435#issuecomment-3134001662
@karenbraganz Ah, if this PR is where you first access them, then yes making them eager load here is probably the right thing to do. However one thing we need to consider is if eagerloading them is going to have performance impact on a rarely used case (I don't have the context on this path or PR to say either way), and if it might better to load it from the DB in some other way only when 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]
