mvfc commented on code in PR #50001:
URL: https://github.com/apache/airflow/pull/50001#discussion_r2070728182


##########
providers/microsoft/azure/src/airflow/providers/microsoft/azure/operators/powerbi.py:
##########
@@ -167,3 +171,182 @@ def execute_complete(self, context: Context, event: 
dict[str, str]) -> Any:
             )
             if event["status"] == "error":
                 raise AirflowException(event["message"])
+
+
+class PowerBIWorkspaceListOperator(BaseOperator):
+    """
+    Gets a list of workspaces where the service principal from the connection 
is assigned as admin.
+
+    .. seealso::
+        For more information on how to use this operator, take a look at the 
guide:
+        :ref:`howto/operator:PowerBIWorkspaceListOperator`
+
+    :param conn_id: Airflow Connection ID that contains the connection 
information for the Power BI account used for authentication.
+    """
+
+    def __init__(
+        self,
+        *,
+        conn_id: str = PowerBIHook.default_conn_name,
+        timeout: float = 60 * 60 * 24 * 7,

Review Comment:
   @gopidesupavan Oh the timeout value I just took what @ambika-garg used on 
the other operators. Took it to be the Microsoft default for None (a week). 
Just left it there to be inline with the other existing operator, triggers and 
hook. 



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