Abhi2939 commented on issue #76723: URL: https://github.com/apache/airflow/issues/76723#issuecomment-6738647327
Hi @rt849122, this looks like a reasonable request, and I’d be happy to work on it. I saw you ticked that you’re willing to submit a PR, so I wanted to check first: are you already working on this? If so, I’m glad to review or help with tests instead. If not, I can take it. My rough plan, in case it helps: Add a config option controlling the default keep_task_state value, defaulting to the current behavior (discard on clear), so existing deployments aren’t affected. Apply it where the API request omits the field. Expose it to the UI so the clear-task dialog’s default matches it (the existing browser-local preference would still override). Add tests and a config description. @maintainers, do you have a preference for which config section and name this should live under? I’d also like to confirm you’re fine with a deployment-wide default here before I open a PR. -- 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]
