Repository navigation
BigQuery: Table.from_string() doesn't support GCP project ID with org prefix #7827
Description
Environment details
- BigQuery
- Google App-Engine standard (Python 2.7)
- Python 2.7
- google-cloud-bigquery==1.10.0
Code example
from google.cloud import bigquery
from google.cloud.bigquery import Table
# Standard SQL (works fine)
# SELECT * FROM `google.com:my-project.mydataset.mytable` LIMIT 1
# Works fine
table = Table.from_string('my-project.mydataset.mytable')
# Fails
table = Table.from_string('google.com:my-project.mydataset.mytable')
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/tmp/vv/local/lib/python2.7/site-packages/google/cloud/bigquery/table.py", line 815, in from_string
return cls(TableReference.from_string(full_table_id))
File "/tmp/vv/local/lib/python2.7/site-packages/google/cloud/bigquery/table.py", line 246, in from_string
"got {}".format(table_id)
ValueError: Too many parts in table_id. Must be a fully-qualified table ID in standard SQL format. e.g. "project.dataset.table", got google.com:my-project.mydataset.mytable
Activity
joncass commentedon May 21, 2019
I am having this same issue. It looks to me like the fix would be to change line https://github.com/googleapis/google-cloud-python/blob/master/bigquery/google/cloud/bigquery/_helpers.py#L590 from
parts = full_id.split(".")toparts = full_id.rsplit(".", 2). Any objections to that change?tswast commentedon Aug 26, 2019
@emar-kar want to take a look at this issue, too? Related to similar fix in #8877
emar-kar commentedon Aug 26, 2019
@tswast yeah, sure. I’ll take a look tomorrow.
plamut commentedon Aug 30, 2019
@emar-kar Are you currently working on this? Asking to not unnecessarily duplicate the effort.
emar-kar commentedon Aug 30, 2019
@plamut Yeah, thank you for asking. I’m working on it.
plamut commentedon Aug 30, 2019
@emar-kar Great, thanks. I updated the assignee for better visibility.