Kyeonghoon
Kyeonghoon
@uranusjr I don't know how to reply to above comment so I wrote a new comment. I don't think I have to make the argument optional since BaseOperator has the...
I changed to use 'timeout' parameter. I think I don't have to make this parameter optional since BaseSensorOperator has default value of timeout https://github.com/apache/airflow/blob/6618c5f90d037d57e9f3bf1e90cd0712426d6caa/airflow/sensors/base.py#L138 and have validating logic to have...
@hussein-awala Hello, I changed to use timeout parameter and set default value of 60 if timeout has value of 'conf.getfloat("sensors", "default_timeout")'.
@hussein-awala hello, I changed to use timeout parameter Can I get review for change?
@hussein-awala Thank you for your feedback. I added deferrable_timeout parameter instead of overrding timeout. but I'm wondering if this is proper naming.
@hussein-awala From the behavior you mentioned, it seems that whether I use execution_timeout or timeout to set _timeout_sec, they both lead to the same termination conditions. Upon further reflection, I'm...
@hussein-awala Hello, I'm waiting for your answer. Could you recommend how to deal with this? Or If this issue is resolved by other PR, let me know if I can...
@hussein-awala Hello, I understand you have a busy schedule, and I just wanted to gently remind you about the review of my pull request. You mentioned planning to review it...
@hussein-awala Hello, I hope this message finds you well. I wanted to follow up regarding the pull request I submitted. It has been a few weeks since I last mentioned...
There has been no feedback or response for several weeks. Therefore, I will close this PR. This content may be addressed in a different PR. Thank you.