bharanidharan14 commented on code in PR #24625:
URL: https://github.com/apache/airflow/pull/24625#discussion_r918579687


##########
airflow/providers/microsoft/azure/hooks/asb.py:
##########
@@ -114,6 +119,93 @@ def delete_queue(self, queue_name: str) -> None:
         with self.get_conn() as service_mgmt_conn:
             service_mgmt_conn.delete_queue(queue_name)
 
+    def create_subscription(

Review Comment:
   I agree with your point instead adding method in hook its better to add in 
operator, it will be lighter weight. But my only doubt can I use client methods 
 in operator call the `get_conn()` method in operator, no where I can seeing in 
the repo using client method directly in operator



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