sjyangkevin commented on issue #54016: URL: https://github.com/apache/airflow/issues/54016#issuecomment-3156723375
Hi @Lee-W I would like to share my approach for this enhancement at high-level and let me know if it aligns with your thoughts. 1. Add a new argument to the init when constructing the HITL operator. I think “respondents” probably is a good name for this. 2. there is a validation function in the HITLOperator, I think we can use that to check if the respondents in the list are all valid users, otherwise, raise an validation error. I think probably user id/name (the one used to login to airflow) can be a good option, or the email address, can be valid values in the `respondents`. 3. from the trigger side, probably we need to keep track of this list of respondents, and run a validation when the response is received and if the response is coming from one of the specified respondents. 4. wondering if it’s possible to receive response from an invalid respondents (i.e., those not in the list), in this case, should the trigger reports this error and keep poking for the response from valid respondents? Thanks -- 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]
