o-nikolas commented on PR #69499: URL: https://github.com/apache/airflow/pull/69499#issuecomment-4906850039
>Why creating new parameters for that? It seems unnecessary. Why not just increasing the value of existing parameters? >> @vincbeck We could just make the default way longer but this does synchronously wait so we shouldn’t unnecessarily use resources to wait. Parameters seem to strike to balance of being usually short but configurable to longer if needed I'm not sure what you mean by being "usually short"? It looks like the default is set to 15mins for those parameters? So the idea of making them short and extending if needed seems like it's not being used here. In reality, I think most people just want the operator to work, even if they have to wait, since the other outcome is much worse. So I'd maybe just simplify them back to constant values like before. But I don't feel tooo strongly about it -- 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]
