blag commented on PR #23335: URL: https://github.com/apache/airflow/pull/23335#issuecomment-1112666622
> What timeout do you think is enough ? I am happy to increas that if you find it annoying - maybe I know a bit "too well" what's going on here - but I am happy to change it to say 10 seconds? Do you think it is enough? For instances where we care about user input: 30 seconds, minimum. That sounds like a lot, but for somebody who has to read, ponder, and respond, it's not. If all we want to do is add a delay, then I think we should just add a delay and _not_ prompt for user input. Folks who know what they're doing can Ctrl+C out of it and rerun with the appropriate flag, just like they likely do for `npm`, `apt`, `brew`, etc. And if we're just adding a delay, I mean, npm thinks 20 seconds is a good wait time for that, so I would probably start with that time and play around with it from there. And yes, I think you are intimately familiar with Breeze so you can probably recognize exactly what the prompt is just by the length, shape, and color of the lines Breeze is spitting. You probably don't even need to read the words on the screen, and/or can digest it line by line instead of word by word. Other folks, maybe not so much. 😆 -- 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]
