kaxil commented on code in PR #54670:
URL: https://github.com/apache/airflow/pull/54670#discussion_r2285617393


##########
providers/standard/src/airflow/providers/standard/triggers/hitl.py:
##########
@@ -107,7 +107,7 @@ async def run(self) -> AsyncIterator[TriggerEvent]:
                     )
                     return
 
-                await sync_to_async(update_htil_detail_response)(
+                await sync_to_async(update_hitl_detail_response)(

Review Comment:
   Need to be careful here! Since it depends on the Task SDK version.
   
   So if older Task SDK version is used, this method won't exist! so you 
probably need a `try..except` unfortunately --- unless this version of Standard 
provider hasn't been released!
   
   Although at the same time this version isn't usable without Airflow 3.1 so 
should be ok.
   
   @Lee-W fyi !



-- 
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]

Reply via email to