You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/08/18 04:40:02 UTC

[GitHub] [airflow] sudohainguyen opened a new issue, #25781: BigQueryGetDataOperator does not support passing project_id

sudohainguyen opened a new issue, #25781:
URL: https://github.com/apache/airflow/issues/25781

   ### Apache Airflow Provider(s)
   
   google
   
   ### Versions of Apache Airflow Providers
   
   apache-airflow-providers-google==8.3.0
   
   ### Apache Airflow version
   
   2.3.2
   
   ### Operating System
   
   MacOS
   
   ### Deployment
   
   Other
   
   ### Deployment details
   
   _No response_
   
   ### What happened
   
   Can not actively pass project_id as an argument when using `BigQueryGetDataOperator`. This operator internally fallbacks into `default` project id.
   
   ### What you think should happen instead
   
   Should let developers pass project_id when needed
   
   ### How to reproduce
   
   _No response_
   
   ### Anything else
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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-unsubscribe@airflow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] gustavom2998 commented on issue #25781: BigQueryGetDataOperator does not support passing project_id

Posted by GitBox <gi...@apache.org>.
gustavom2998 commented on issue #25781:
URL: https://github.com/apache/airflow/issues/25781#issuecomment-1227844369

   Found an issue/bug while using the code from the last merge. In [line 443](https://github.com/apache/airflow/blob/main/airflow/providers/google/cloud/operators/bigquery.py#L443) we need to add the following row to the get_schema operation, otherwise the check goes to the wrong project - so the operator won't work. I managed to get it working locally by making the following change:
   
   ```python
   schema: Dict[str, list] = hook.get_schema(
       dataset_id=self.dataset_id,
       table_id=self.table_id,
       project_id=self.project_id
   )
   ```


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] mik-laj closed issue #25781: BigQueryGetDataOperator does not support passing project_id

Posted by GitBox <gi...@apache.org>.
mik-laj closed issue #25781: BigQueryGetDataOperator does not support passing project_id
URL: https://github.com/apache/airflow/issues/25781


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org