yuseok89 commented on code in PR #71922:
URL: https://github.com/apache/airflow/pull/71922#discussion_r4011050670
##########
providers/http/src/airflow/providers/http/triggers/http.py:
##########
@@ -220,6 +220,9 @@ class HttpSensorTrigger(BaseTrigger):
:param extra_options: Additional kwargs to pass when creating a request.
For example, ``run(json=obj)`` is passed as
``aiohttp.ClientSession().get(json=obj)``
:param poke_interval: Time to sleep using asyncio
+ :param initial_delay: Time to sleep before the first request. Used when the
Review Comment:
I could not find another trigger using the same `initial_delay` parameter. I
added it because a failed `response_check` creates a new trigger, which would
otherwise send its first request immediately instead of respecting
`poke_interval`.
Since this introduces a new trigger parameter, I would be happy to continue
discussing in this PR whether keeping it local to `HttpSensorTrigger` is
appropriate or whether a different design would be preferable.
--
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]
