New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: remove unused annotation in results_table #11548
Conversation
PiperOrigin-RevId: 553938801 Source-Link: googleapis/googleapis@ff66907 Source-Link: googleapis/googleapis-gen@ee14ddb Copy-Tag: eyJwIjoicGFja2FnZXMvZ29vZ2xlLWNsb3VkLWRhdGFwbGV4Ly5Pd2xCb3QueWFtbCIsImgiOiJlZTE0ZGRiMWZjOWU5MTliMWEyZTEwYTZjYzAxZTJmODFlMWMzYzQxIn0=
@staticmethod | ||
def table_path( | ||
project: str, | ||
dataset: str, | ||
table: str, | ||
) -> str: | ||
"""Returns a fully-qualified table string.""" | ||
return "projects/{project}/datasets/{dataset}/tables/{table}".format( | ||
project=project, | ||
dataset=dataset, | ||
table=table, | ||
) | ||
|
||
@staticmethod | ||
def parse_table_path(path: str) -> Dict[str, str]: | ||
"""Parses a table path into its component segments.""" | ||
m = re.match( | ||
r"^projects/(?P<project>.+?)/datasets/(?P<dataset>.+?)/tables/(?P<table>.+?)$", | ||
path, | ||
) | ||
return m.groupdict() if m else {} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a breaking change however the methods were only added on July 25th. In b/294420502, there is a note that this method would not have worked because the API only accepts
"//bigquery.googleapis.com/projects/{project}/datasets/{dataset}/tables/{table}" and not "projects/{project}/datasets/{dataset}/tables/{table}". Approving based on the discussions in b/294420502
PiperOrigin-RevId: 553938801
Source-Link: googleapis/googleapis@ff66907
Source-Link: https://github.com/googleapis/googleapis-gen/commit/ee14ddb1fc9e919b1a2e10a6cc01e2f81e1c3c41
Copy-Tag: eyJwIjoicGFja2FnZXMvZ29vZ2xlLWNsb3VkLWRhdGFwbGV4Ly5Pd2xCb3QueWFtbCIsImgiOiJlZTE0ZGRiMWZjOWU5MTliMWEyZTEwYTZjYzAxZTJmODFlMWMzYzQxIn0=