turbaszek commented on a change in pull request #8477:
URL: https://github.com/apache/airflow/pull/8477#discussion_r415957681



##########
File path: airflow/providers/google/cloud/operators/bigquery.py
##########
@@ -1301,8 +1302,8 @@ class BigQueryUpdateDatasetOperator(BaseOperator):
 
     @apply_defaults
     def __init__(self,
-                 dataset_id: str,
                  dataset_resource: dict,
+                 dataset_id: Optional[str] = None,

Review comment:
       Added

##########
File path: airflow/providers/google/cloud/hooks/bigquery.py
##########
@@ -349,56 +358,49 @@ def get_dataset_tables(self, dataset_id: str, project_id: 
Optional[str] = None,
         :param page_token: (Optional) page token, returned from a previous 
call,
             identifying the result set.
         :type page_token: str
-
-        :return: map containing the list of tables + metadata.
+        :param retry: How to retry the RPC.
+        :type retry: google.api_core.retry.Retry
+        :return: List of tables associated with the dataset.
         """
-        service = self.get_service()
-
-        optional_params = {}  # type: Dict[str, Union[str, int]]
-        if max_results:
-            optional_params['maxResults'] = max_results
-        if page_token:
-            optional_params['pageToken'] = page_token
-
-        dataset_project_id = project_id or self.project_id
+        project_id = project_id or self.project_id
 
-        return (service.tables().list(  # pylint: disable=no-member
-            projectId=dataset_project_id,
-            datasetId=dataset_id,
-            **optional_params).execute(num_retries=self.num_retries))
+        tables = Client(client_info=self.client_info).list_tables(

Review comment:
       Done




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to