Vamsi-klu commented on PR #69748: URL: https://github.com/apache/airflow/pull/69748#issuecomment-5140216652
@potiuk Thanks for the design pushback — agreed on the delivery mechanism. Redesign in the latest push (3 changes): 1. **No trigger-side warning** — `HttpTrigger` stays silent on construct/deserialize so Triggerer restarts do not spam logs (and Dag authors never saw those anyway). 2. **No `catch_warnings`/`filterwarnings` suppression** — single emission site removes the warn-then-unwarn coupling to message text. 3. **One `self.log.warning` from `HttpOperator.execute()`** when `deferrable=True` and the method is outside the RFC 9110 §9.2.2 idempotent set — once per task attempt (not parse-time `__init__`, not per pagination page). PUT/DELETE stay silent; PATCH/POST/custom methods warn. Tests cover lowercase methods, `method is None`, pagination cardinality, trigger reconstruct, and the idempotent set. Ready for another look. -- 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]
