apache-airflow-providers-google 15.1.0rc1__py3-none-any.whl → 19.1.0rc1__py3-none-any.whl

airflow/providers/google/marketing_platform/operators/display_video.py

@@ -19,494 +19,23 @@
 
 from __future__ import annotations
 
-import csv
-import json
-import shutil
+import os
 import tempfile
-import urllib.request
+import zipfile
 from collections.abc import Sequence
 from typing import TYPE_CHECKING, Any
-from urllib.parse import urlsplit
 
-from airflow.exceptions import AirflowException
-from airflow.models import BaseOperator
 from airflow.providers.google.cloud.hooks.gcs import GCSHook
 from airflow.providers.google.marketing_platform.hooks.display_video import GoogleDisplayVideo360Hook
+from airflow.providers.google.version_compat import BaseOperator
 
 if TYPE_CHECKING:
-    from airflow.utils.context import Context
-
-
-class GoogleDisplayVideo360CreateQueryOperator(BaseOperator):
-    """
-    Creates a query.
-
-    .. seealso::
-        For more information on how to use this operator, take a look at the guide:
-        ``GoogleDisplayVideo360CreateQueryOperator``
-
-    .. seealso::
-        Check also the official API docs:
-        `https://developers.google.com/bid-manager/v2/queries/create`
-
-    :param body: Report object passed to the request's body as described here:
-        https://developers.google.com/bid-manager/v2/queries#Query
-    :param api_version: The version of the api that will be requested for example 'v3'.
-    :param gcp_conn_id: The connection ID to use when fetching connection info.
-    :param impersonation_chain: Optional service account to impersonate using short-term
-        credentials, or chained list of accounts required to get the access_token
-        of the last account in the list, which will be impersonated in the request.
-        If set as a string, the account must grant the originating account
-        the Service Account Token Creator IAM role.
-        If set as a sequence, the identities from the list must grant
-        Service Account Token Creator IAM role to the directly preceding identity, with first
-        account from the list granting this role to the originating account (templated).
-    """
-
-    template_fields: Sequence[str] = (
-        "body",
-        "impersonation_chain",
-    )
-    template_ext: Sequence[str] = (".json",)
-
-    def __init__(
-        self,
-        *,
-        body: dict[str, Any],
-        api_version: str = "v2",
-        gcp_conn_id: str = "google_cloud_default",
-        impersonation_chain: str | Sequence[str] | None = None,
-        **kwargs,
-    ) -> None:
-        super().__init__(**kwargs)
-        self.body = body
-        self.api_version = api_version
-        self.gcp_conn_id = gcp_conn_id
-        self.impersonation_chain = impersonation_chain
-
-    def prepare_template(self) -> None:
-        # If .json is passed then we have to read the file
-        if isinstance(self.body, str) and self.body.endswith(".json"):
-            with open(self.body) as file:
-                self.body = json.load(file)
-
-    def execute(self, context: Context) -> dict:
-        hook = GoogleDisplayVideo360Hook(
-            gcp_conn_id=self.gcp_conn_id,
-            api_version=self.api_version,
-            impersonation_chain=self.impersonation_chain,
-        )
-        self.log.info("Creating Display & Video 360 query.")
-        response = hook.create_query(query=self.body)
-        query_id = response["queryId"]
-        self.xcom_push(context, key="query_id", value=query_id)
-        self.log.info("Created query with ID: %s", query_id)
-        return response
-
-
-class GoogleDisplayVideo360DeleteReportOperator(BaseOperator):
-    """
-    Deletes a stored query as well as the associated stored reports.
-
-    .. seealso::
-        For more information on how to use this operator, take a look at the guide:
-        :ref:`howto/operator:GoogleDisplayVideo360DeleteReportOperator`
-
-    .. seealso::
-        Check also the official API docs:
-        `https://developers.google.com/bid-manager/v2/queries/delete`
-
-    :param report_id: Report ID to delete.
-    :param report_name: Name of the report to delete.
-    :param api_version: The version of the api that will be requested for example 'v3'.
-    :param gcp_conn_id: The connection ID to use when fetching connection info.
-    :param impersonation_chain: Optional service account to impersonate using short-term
-        credentials, or chained list of accounts required to get the access_token
-        of the last account in the list, which will be impersonated in the request.
-        If set as a string, the account must grant the originating account
-        the Service Account Token Creator IAM role.
-        If set as a sequence, the identities from the list must grant
-        Service Account Token Creator IAM role to the directly preceding identity, with first
-        account from the list granting this role to the originating account (templated).
-    """
-
-    template_fields: Sequence[str] = (
-        "report_id",
-        "impersonation_chain",
-    )
-
-    def __init__(
-        self,
-        *,
-        report_id: str | None = None,
-        report_name: str | None = None,
-        api_version: str = "v2",
-        gcp_conn_id: str = "google_cloud_default",
-        impersonation_chain: str | Sequence[str] | None = None,
-        **kwargs,
-    ) -> None:
-        super().__init__(**kwargs)
-        self.report_id = report_id
-        self.report_name = report_name
-        self.api_version = api_version
-        self.gcp_conn_id = gcp_conn_id
-        self.impersonation_chain = impersonation_chain
-
-        if report_name and report_id:
-            raise AirflowException("Use only one value - `report_name` or `report_id`.")
-
-        if not (report_name or report_id):
-            raise AirflowException("Provide one of the values: `report_name` or `report_id`.")
-
-    def execute(self, context: Context) -> None:
-        hook = GoogleDisplayVideo360Hook(
-            gcp_conn_id=self.gcp_conn_id,
-            api_version=self.api_version,
-            impersonation_chain=self.impersonation_chain,
-        )
-        if self.report_id:
-            reports_ids_to_delete = [self.report_id]
-        else:
-            reports = hook.list_queries()
-            reports_ids_to_delete = [
-                report["queryId"] for report in reports if report["metadata"]["title"] == self.report_name
-            ]
-
-        for report_id in reports_ids_to_delete:
-            self.log.info("Deleting report with id: %s", report_id)
-            hook.delete_query(query_id=report_id)
-            self.log.info("Report deleted.")
-
-
-class GoogleDisplayVideo360DownloadReportV2Operator(BaseOperator):
-    """
-    Retrieves a stored query.
-
-    .. seealso::
-        For more information on how to use this operator, take a look at the guide:
-        :ref:`howto/operator:GoogleDisplayVideo360DownloadReportV2Operator`
-
-    .. seealso::
-        Check also the official API docs:
-        `https://developers.google.com/bid-manager/v2/queries/get`
-
-    :param report_id: Report ID to retrieve.
-    :param bucket_name: The bucket to upload to.
-    :param report_name: The report name to set when uploading the local file.
-    :param chunk_size: File will be downloaded in chunks of this many bytes.
-    :param gzip: Option to compress local file or file data for upload
-    :param api_version: The version of the api that will be requested for example 'v3'.
-    :param gcp_conn_id: The connection ID to use when fetching connection info.
-    :param impersonation_chain: Optional service account to impersonate using short-term
-        credentials, or chained list of accounts required to get the access_token
-        of the last account in the list, which will be impersonated in the request.
-        If set as a string, the account must grant the originating account
-        the Service Account Token Creator IAM role.
-        If set as a sequence, the identities from the list must grant
-        Service Account Token Creator IAM role to the directly preceding identity, with first
-        account from the list granting this role to the originating account (templated).
-    """
-
-    template_fields: Sequence[str] = (
-        "query_id",
-        "report_id",
-        "bucket_name",
-        "report_name",
-        "impersonation_chain",
-    )
-
-    def __init__(
-        self,
-        *,
-        query_id: str,
-        report_id: str,
-        bucket_name: str,
-        report_name: str | None = None,
-        gzip: bool = True,
-        chunk_size: int = 10 * 1024 * 1024,
-        api_version: str = "v2",
-        gcp_conn_id: str = "google_cloud_default",
-        impersonation_chain: str | Sequence[str] | None = None,
-        **kwargs,
-    ) -> None:
-        super().__init__(**kwargs)
-        self.query_id = query_id
-        self.report_id = report_id
-        self.chunk_size = chunk_size
-        self.gzip = gzip
-        self.bucket_name = bucket_name
-        self.report_name = report_name
-        self.api_version = api_version
-        self.gcp_conn_id = gcp_conn_id
-        self.impersonation_chain = impersonation_chain
-
-    def _resolve_file_name(self, name: str) -> str:
-        new_name = name if name.endswith(".csv") else f"{name}.csv"
-        new_name = f"{new_name}.gz" if self.gzip else new_name
-        return new_name
-
-    @staticmethod
-    def _set_bucket_name(name: str) -> str:
-        bucket = name if not name.startswith("gs://") else name[5:]
-        return bucket.strip("/")
-
-    def execute(self, context: Context):
-        hook = GoogleDisplayVideo360Hook(
-            gcp_conn_id=self.gcp_conn_id,
-            api_version=self.api_version,
-            impersonation_chain=self.impersonation_chain,
-        )
-        gcs_hook = GCSHook(
-            gcp_conn_id=self.gcp_conn_id,
-            impersonation_chain=self.impersonation_chain,
-        )
-
-        resource = hook.get_report(query_id=self.query_id, report_id=self.report_id)
-        status = resource.get("metadata", {}).get("status", {}).get("state")
-        if resource and status not in ["DONE", "FAILED"]:
-            raise AirflowException(f"Report {self.report_id} for query {self.query_id} is still running")
-
-        # If no custom report_name provided, use DV360 name
-        file_url = resource["metadata"]["googleCloudStoragePath"]
-        if urllib.parse.urlparse(file_url).scheme == "file":
-            raise AirflowException("Accessing local file is not allowed in this operator")
-        report_name = self.report_name or urlsplit(file_url).path.split("/")[-1]
-        report_name = self._resolve_file_name(report_name)
-
-        # Download the report
-        self.log.info("Starting downloading report %s", self.report_id)
-        with tempfile.NamedTemporaryFile(delete=False) as temp_file:
-            with urllib.request.urlopen(file_url) as response:  # nosec
-                shutil.copyfileobj(response, temp_file, length=self.chunk_size)
-
-            temp_file.flush()
-            # Upload the local file to bucket
-            bucket_name = self._set_bucket_name(self.bucket_name)
-            gcs_hook.upload(
-                bucket_name=bucket_name,
-                object_name=report_name,
-                gzip=self.gzip,
-                filename=temp_file.name,
-                mime_type="text/csv",
-            )
-        self.log.info(
-            "Report %s was saved in bucket %s as %s.",
-            self.report_id,
-            self.bucket_name,
-            report_name,
-        )
-        self.xcom_push(context, key="report_name", value=report_name)
-
-
-class GoogleDisplayVideo360RunQueryOperator(BaseOperator):
-    """
-    Runs a stored query to generate a report.
-
-    .. seealso::
-        For more information on how to use this operator, take a look at the guide:
-        :ref:`howto/operator:GoogleDisplayVideo360RunQueryOperator`
-
-    .. seealso::
-        Check also the official API docs:
-        `https://developers.google.com/bid-manager/v2/queries/run`
-
-    :param report_id: Report ID to run.
-    :param parameters: Parameters for running a report as described here:
-        https://developers.google.com/bid-manager/v2/queries/run
-    :param api_version: The version of the api that will be requested for example 'v3'.
-    :param gcp_conn_id: The connection ID to use when fetching connection info.
-    :param impersonation_chain: Optional service account to impersonate using short-term
-        credentials, or chained list of accounts required to get the access_token
-        of the last account in the list, which will be impersonated in the request.
-        If set as a string, the account must grant the originating account
-        the Service Account Token Creator IAM role.
-        If set as a sequence, the identities from the list must grant
-        Service Account Token Creator IAM role to the directly preceding identity, with first
-        account from the list granting this role to the originating account (templated).
-    """
-
-    template_fields: Sequence[str] = (
-        "query_id",
-        "parameters",
-        "impersonation_chain",
-    )
-
-    def __init__(
-        self,
-        *,
-        query_id: str,
-        parameters: dict[str, Any] | None = None,
-        api_version: str = "v2",
-        gcp_conn_id: str = "google_cloud_default",
-        impersonation_chain: str | Sequence[str] | None = None,
-        **kwargs,
-    ) -> None:
-        super().__init__(**kwargs)
-        self.query_id = query_id
-        self.api_version = api_version
-        self.gcp_conn_id = gcp_conn_id
-        self.parameters = parameters
-        self.impersonation_chain = impersonation_chain
-
-    def execute(self, context: Context) -> dict:
-        hook = GoogleDisplayVideo360Hook(
-            gcp_conn_id=self.gcp_conn_id,
-            api_version=self.api_version,
-            impersonation_chain=self.impersonation_chain,
-        )
-        self.log.info(
-            "Running query %s with the following parameters:\n %s",
-            self.query_id,
-            self.parameters,
-        )
-        response = hook.run_query(query_id=self.query_id, params=self.parameters)
-        self.xcom_push(context, key="query_id", value=response["key"]["queryId"])
-        self.xcom_push(context, key="report_id", value=response["key"]["reportId"])
-        return response
-
-
-class GoogleDisplayVideo360DownloadLineItemsOperator(BaseOperator):
-    """
-    Retrieves line items in CSV format.
-
-    .. seealso::
-        For more information on how to use this operator, take a look at the guide:
-        :ref:`howto/operator:GoogleDisplayVideo360DownloadLineItemsOperator`
-
-    .. seealso::
-        Check also the official API docs:
-        `https://developers.google.com/bid-manager/v1.1/lineitems/downloadlineitems`
-
-    :param request_body: dictionary with parameters that should be passed into.
-            More information about it can be found here:
-            https://developers.google.com/bid-manager/v1.1/lineitems/downloadlineitems
-    """
-
-    template_fields: Sequence[str] = (
-        "request_body",
-        "bucket_name",
-        "object_name",
-        "impersonation_chain",
-    )
-
-    def __init__(
-        self,
-        *,
-        request_body: dict[str, Any],
-        bucket_name: str,
-        object_name: str,
-        gzip: bool = False,
-        api_version: str = "v1.1",
-        gcp_conn_id: str = "google_cloud_default",
-        impersonation_chain: str | Sequence[str] | None = None,
-        **kwargs,
-    ) -> None:
-        super().__init__(**kwargs)
-        self.request_body = request_body
-        self.object_name = object_name
-        self.bucket_name = bucket_name
-        self.gzip = gzip
-        self.api_version = api_version
-        self.gcp_conn_id = gcp_conn_id
-        self.impersonation_chain = impersonation_chain
-
-    def execute(self, context: Context) -> str:
-        gcs_hook = GCSHook(
-            gcp_conn_id=self.gcp_conn_id,
-            impersonation_chain=self.impersonation_chain,
-        )
-        hook = GoogleDisplayVideo360Hook(
-            gcp_conn_id=self.gcp_conn_id,
-            api_version=self.api_version,
-            impersonation_chain=self.impersonation_chain,
-        )
-
-        self.log.info("Retrieving report...")
-        content: list[str] = hook.download_line_items(request_body=self.request_body)
-
-        with tempfile.NamedTemporaryFile("w+") as temp_file:
-            writer = csv.writer(temp_file)
-            writer.writerows(content)
-            temp_file.flush()
-            gcs_hook.upload(
-                bucket_name=self.bucket_name,
-                object_name=self.object_name,
-                filename=temp_file.name,
-                mime_type="text/csv",
-                gzip=self.gzip,
-            )
-        return f"{self.bucket_name}/{self.object_name}"
-
-
-class GoogleDisplayVideo360UploadLineItemsOperator(BaseOperator):
-    """
-    Uploads line items in CSV format.
-
-    .. seealso::
-        For more information on how to use this operator, take a look at the guide:
-        :ref:`howto/operator:GoogleDisplayVideo360UploadLineItemsOperator`
-
-    .. seealso::
-        Check also the official API docs:
-        `https://developers.google.com/bid-manager/v1.1/lineitems/uploadlineitems`
-
-    :param request_body: request to upload line items.
-    :param bucket_name: The bucket form data is downloaded.
-    :param object_name: The object to fetch.
-    :param filename: The filename to fetch.
-    :param dry_run: Upload status without actually persisting the line items.
-    """
-
-    template_fields: Sequence[str] = (
-        "bucket_name",
-        "object_name",
-        "impersonation_chain",
-    )
-
-    def __init__(
-        self,
-        *,
-        bucket_name: str,
-        object_name: str,
-        api_version: str = "v1.1",
-        gcp_conn_id: str = "google_cloud_default",
-        impersonation_chain: str | Sequence[str] | None = None,
-        **kwargs,
-    ) -> None:
-        super().__init__(**kwargs)
-        self.bucket_name = bucket_name
-        self.object_name = object_name
-        self.api_version = api_version
-        self.gcp_conn_id = gcp_conn_id
-        self.impersonation_chain = impersonation_chain
-
-    def execute(self, context: Context) -> None:
-        gcs_hook = GCSHook(
-            gcp_conn_id=self.gcp_conn_id,
-            impersonation_chain=self.impersonation_chain,
-        )
-        hook = GoogleDisplayVideo360Hook(
-            gcp_conn_id=self.gcp_conn_id,
-            api_version=self.api_version,
-            impersonation_chain=self.impersonation_chain,
-        )
-
-        self.log.info("Uploading file %s...")
-        # Saving file in the temporary directory,
-        # downloaded file from the GCS could be a 1GB size or even more
-        with tempfile.NamedTemporaryFile("w+") as f:
-            line_items = gcs_hook.download(
-                bucket_name=self.bucket_name,
-                object_name=self.object_name,
-                filename=f.name,
-            )
-            f.flush()
-            hook.upload_line_items(line_items=line_items)
+    from airflow.providers.common.compat.sdk import Context
 
 
 class GoogleDisplayVideo360CreateSDFDownloadTaskOperator(BaseOperator):
     """
-    Creates SDF operation task.
+    Creates an SDF operation task.
 
     .. seealso::
         For more information on how to use this operator, take a look at the guide:
@@ -542,7 +71,7 @@ class GoogleDisplayVideo360CreateSDFDownloadTaskOperator(BaseOperator):
         self,
         *,
         body_request: dict[str, Any],
-        api_version: str = "v1",
+        api_version: str = "v4",
         gcp_conn_id: str = "google_cloud_default",
         impersonation_chain: str | Sequence[str] | None = None,
         **kwargs,
@@ -564,7 +93,7 @@ class GoogleDisplayVideo360CreateSDFDownloadTaskOperator(BaseOperator):
         operation = hook.create_sdf_download_operation(body_request=self.body_request)
 
         name = operation["name"]
-        self.xcom_push(context, key="name", value=name)
+        context["task_instance"].xcom_push(key="name", value=name)
         self.log.info("Created SDF operation with name: %s", name)
 
         return operation
@@ -613,7 +142,7 @@ class GoogleDisplayVideo360SDFtoGCSOperator(BaseOperator):
         bucket_name: str,
         object_name: str,
         gzip: bool = False,
-        api_version: str = "v1",
+        api_version: str = "v4",
         gcp_conn_id: str = "google_cloud_default",
         impersonation_chain: str | Sequence[str] | None = None,
         **kwargs,
@@ -645,14 +174,24 @@ class GoogleDisplayVideo360SDFtoGCSOperator(BaseOperator):
         media = hook.download_media(resource_name=operation_state["response"]["resourceName"])
 
         self.log.info("Sending file to the Google Cloud Storage...")
-        with tempfile.NamedTemporaryFile() as temp_file:
-            hook.download_content_from_request(temp_file, media, chunk_size=1024 * 1024)
-            temp_file.flush()
-            gcs_hook.upload(
-                bucket_name=self.bucket_name,
-                object_name=self.object_name,
-                filename=temp_file.name,
-                gzip=self.gzip,
-            )
-
+        with tempfile.TemporaryDirectory() as tmp_dir:
+            zip_path = os.path.join(tmp_dir, "sdf.zip")
+
+            # Download the ZIP
+            with open(zip_path, "wb") as f:
+                hook.download_content_from_request(f, media, chunk_size=1024 * 1024)
+
+            # Extract CSV
+            with zipfile.ZipFile(zip_path, "r") as zip_ref:
+                zip_ref.extractall(tmp_dir)
+
+            # Upload CSV file
+            for fname in os.listdir(tmp_dir):
+                if fname.endswith(".csv"):
+                    gcs_hook.upload(
+                        bucket_name=self.bucket_name,
+                        object_name=self.object_name,
+                        filename=os.path.join(tmp_dir, fname),
+                        gzip=False,
+                    )
         return f"{self.bucket_name}/{self.object_name}"

airflow/providers/google/marketing_platform/operators/search_ads.py

@@ -23,11 +23,11 @@ from collections.abc import Sequence
 from functools import cached_property
 from typing import TYPE_CHECKING, Any
 
-from airflow.models import BaseOperator
 from airflow.providers.google.marketing_platform.hooks.search_ads import GoogleSearchAdsReportingHook
+from airflow.providers.google.version_compat import BaseOperator
 
 if TYPE_CHECKING:
-    from airflow.utils.context import Context
+    from airflow.providers.common.compat.sdk import Context
 
 
 class _GoogleSearchAdsBaseOperator(BaseOperator):

airflow/providers/google/marketing_platform/sensors/campaign_manager.py

@@ -22,11 +22,11 @@ from __future__ import annotations
 from collections.abc import Sequence
 from typing import TYPE_CHECKING
 
+from airflow.providers.common.compat.sdk import BaseSensorOperator
 from airflow.providers.google.marketing_platform.hooks.campaign_manager import GoogleCampaignManagerHook
-from airflow.sensors.base import BaseSensorOperator
 
 if TYPE_CHECKING:
-    from airflow.utils.context import Context
+    from airflow.providers.common.compat.sdk import Context
 
 
 class GoogleCampaignManagerReportSensor(BaseSensorOperator):

airflow/providers/google/marketing_platform/sensors/display_video.py

@@ -22,11 +22,11 @@ from collections.abc import Sequence
 from typing import TYPE_CHECKING
 
 from airflow.exceptions import AirflowException
+from airflow.providers.common.compat.sdk import BaseSensorOperator
 from airflow.providers.google.marketing_platform.hooks.display_video import GoogleDisplayVideo360Hook
-from airflow.sensors.base import BaseSensorOperator
 
 if TYPE_CHECKING:
-    from airflow.utils.context import Context
+    from airflow.providers.common.compat.sdk import Context
 
 
 class GoogleDisplayVideo360GetSDFDownloadOperationSensor(BaseSensorOperator):
@@ -59,7 +59,7 @@ class GoogleDisplayVideo360GetSDFDownloadOperationSensor(BaseSensorOperator):
     def __init__(
         self,
         operation_name: str,
-        api_version: str = "v1",
+        api_version: str = "v4",
         gcp_conn_id: str = "google_cloud_default",
         mode: str = "reschedule",
         poke_interval: int = 60 * 5,
@@ -88,63 +88,3 @@ class GoogleDisplayVideo360GetSDFDownloadOperationSensor(BaseSensorOperator):
         if operation and operation.get("done"):
             return True
         return False
-
-
-class GoogleDisplayVideo360RunQuerySensor(BaseSensorOperator):
-    """
-    Sensor for detecting the completion of DV360 reports for API v2.
-
-    .. seealso::
-        For more information on how to use this operator, take a look at the guide:
-        :ref:`howto/operator:GoogleDisplayVideo360RunQuerySensor`
-
-    :param query_id: Query ID for which report was generated
-    :param report_id: Report ID for which you want to wait
-    :param api_version: The version of the api that will be requested for example 'v3'.
-    :param gcp_conn_id: The connection ID to use when fetching connection info.
-    :param impersonation_chain: Optional service account to impersonate using short-term
-        credentials, or chained list of accounts required to get the access_token
-        of the last account in the list, which will be impersonated in the request.
-        If set as a string, the account must grant the originating account
-        the Service Account Token Creator IAM role.
-        If set as a sequence, the identities from the list must grant
-        Service Account Token Creator IAM role to the directly preceding identity, with first
-        account from the list granting this role to the originating account (templated).
-    """
-
-    template_fields: Sequence[str] = (
-        "query_id",
-        "report_id",
-        "impersonation_chain",
-    )
-
-    def __init__(
-        self,
-        *,
-        query_id: str,
-        report_id: str,
-        api_version: str = "v2",
-        gcp_conn_id: str = "google_cloud_default",
-        impersonation_chain: str | Sequence[str] | None = None,
-        **kwargs,
-    ) -> None:
-        super().__init__(**kwargs)
-        self.query_id = query_id
-        self.report_id = report_id
-        self.api_version = api_version
-        self.gcp_conn_id = gcp_conn_id
-        self.impersonation_chain = impersonation_chain
-
-    def poke(self, context: Context) -> bool:
-        hook = GoogleDisplayVideo360Hook(
-            gcp_conn_id=self.gcp_conn_id,
-            api_version=self.api_version,
-            impersonation_chain=self.impersonation_chain,
-        )
-
-        response = hook.get_report(query_id=self.query_id, report_id=self.report_id)
-        status = response.get("metadata", {}).get("status", {}).get("state")
-        self.log.info("STATUS OF THE REPORT %s FOR QUERY %s: %s", self.report_id, self.query_id, status)
-        if response and status in ["DONE", "FAILED"]:
-            return True
-        return False

airflow/providers/google/suite/hooks/calendar.py

@@ -104,7 +104,7 @@ class GoogleCalendarHook(GoogleBaseHook):
             only the participant is returned.
         :param max_results: Optional. Maximum number of events returned on one result page.
             Incomplete pages can be detected by a non-empty ``nextPageToken`` field in the response.
-            By default the value is 250 events. The page size can never be larger than 2500 events
+            By default, the value is 250 events. The page size can never be larger than 2500 events
         :param order_by: Optional. Acceptable values are ``"startTime"`` or "updated"
         :param private_extended_property: Optional. Extended properties constraint specified as
             ``propertyName=value``. Matches only private properties. This parameter might be repeated