dstandish commented on PR #39046: URL: https://github.com/apache/airflow/pull/39046#issuecomment-2060367225
> > hi @o-nikolas when do we / should we _not_ refresh executor? > > Hey @dstandish! Without more context I'm not sure if you're disagreeing with the proposed changes, or the description/code isn't very clear. But the gist of the proposal is to keep the executor fairly stable but allow wiping/refreshing it when clearing a task (which is a very clear user action of "I want to wipe this thing and start over"). The executor shouldn't be changing with every dag parse or even during execution/between retries. why wouldn’t you want the executor to be updated if the user changed it between retries? That seems desirable? It seems to me you’d just want to guard against any negative consequences if it changed during execution? Like clearly you’d want to finish if it already started. But not sure why we shouldn’t allow the current code to be respected otherwise -- 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]
