Skip to content

fix/issue-67945: Adding warning message to execute_async method#67962

Open
jroachgolf84 wants to merge 1 commit into
apache:mainfrom
jroachgolf84:fix/issue-67945
Open

fix/issue-67945: Adding warning message to execute_async method#67962
jroachgolf84 wants to merge 1 commit into
apache:mainfrom
jroachgolf84:fix/issue-67945

Conversation

@jroachgolf84
Copy link
Copy Markdown
Collaborator

@jroachgolf84 jroachgolf84 commented Jun 3, 2026

Description

Adding a warning message in the execute_async method in the HttpOperator class when deferring to the HttpTrigger. Per the original issue:

execute_async() defers immediately without making the HTTP request. The actual HTTP call happens inside HttpTrigger.run(). When the Triggerer restarts, the Trigger is re-instantiated from serialize() and run() is called again from scratch — sending the same POST request a second time.

closes: #67945

Testing

Unit tests were added for this change. These tests validate that an error message is properly output (or not) for certain methods. These tests can be executed using the following commands:

breeze testing providers-tests providers/http/tests/unit/http/operators/test_http.py::TestHttpOperator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HttpOperator: deferrable=True may send duplicate requests on Triggerer restart

1 participant