Programmer-RD-AI commented on PR #53356: URL: https://github.com/apache/airflow/pull/53356#issuecomment-3124244162
Hi, Sorry for the Delayed Response. > Would it be simpler if the BaseAzureServiceBusTrigger just raised the TriggerEvent with the service_bus_client directly? it would be simpler the only reason we have the base class is so that we can handle both the queue's and subscriptions and since that their hooks are different. > I understand the idea behind using asyncio.Queue here, but IMO, we don't need to be constrained by the current methods available in MessageHook or use an asyncio.Queue just to accommodate the MessageHook interface. what is the best way you would recommend? I built it around the hooks so that it is easier but it is a bit more coupled together -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
