tvalentyn commented on code in PR #40225: URL: https://github.com/apache/beam/pull/40225#discussion_r4170237341
########## sdks/python/apache_beam/io/gcp/bigquery.py: ########## @@ -29,10 +29,11 @@ Also, for programming convenience, instances of TableReference and TableSchema have a string representation that can be used for the corresponding arguments: - - TableReference can be a PROJECT:DATASET.TABLE, PROJECT.DATASET.TABLE or - DATASET.TABLE string. Lakehouse runtime catalog (BigLake metastore) tables - use PROJECT.CATALOG.NAMESPACE.TABLE, which maps to a composite - CATALOG.NAMESPACE dataset id. + - TableReference can be a PROJECT:DATASET.TABLE, PROJECT.DATASET.TABLE, + DATASET.TABLE or, for Lakehouse runtime catalog (BigLake metastore) + tables, PROJECT.CATALOG.NAMESPACE.TABLE, which maps to a composite + CATALOG.NAMESPACE dataset id. A Lakehouse reference must include the + project id: a three-part string is read as PROJECT.DATASET.TABLE. Review Comment: nit: I'd drop : 'a three-part string is read as PROJECT.DATASET.TABLE', since it's slightly confusing. The statement after `:` might be interpreted by a reader as an illustration of the statement before `:` , but it is not the case (it is an explanation). -- 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]
