MaksYermak commented on PR #45610:
URL: https://github.com/apache/airflow/pull/45610#issuecomment-2630731893

   > > > I'll be happy to approve and merge, after handling the unit test and 
the CI issue (probably merging from `main` shold help). @EugeneYushin - Could 
you please try @MaksYermak 's suggestion?
   > > 
   > > 
   > > Hi! I am afraid we can't merge this code. We are working rn on actually 
deprecating both BigQueryCreateExternalTableOperator and 
BigQueryCreateEmptyTableOperator since there is a lot of misunderstanding 
because of those deprecations of parameters. So there will be a new 
BigQueryCreateTableOperator which will support current logic from both those 
operators
   > 
   > @VladaZakharova What do you think would be the best way to handle this PR? 
Should I just close it with empty resolution?
   
   @EugeneYushin it's better to close this PR, without resolution. Soon we will 
create a new operator `BigQueryCreateTableOperator` and deprecate 
`BigQueryCreateExternalTableOperator` and `BigQueryCreateEmptyTableOperator` 
operators. As a solution for `autodetect`'s  problem you can pass the 
`table_resource` parameter to the Operator as a `dict` or `Table` object.


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