feat(bigquery): check json_rows arg type in insert_rows_json() by IlyaFaer · Pull Request #10162 · googleapis/google-cloud-python · GitHub

@tswast, maybe we should do the same thing for insert_rows()? The situation in it is very close to insert_rows_json():

google-cloud-python/bigquery/google/cloud/bigquery/client.py

Lines 2345 to 2364 in cb32c36

    
               def insert_rows(self, table, rows, selected_fields=None, **kwargs): 
        
                   """Insert rows into a table via the streaming API. 
        
                   See 
        
                   https://cloud.google.com/bigquery/docs/reference/rest/v2/tabledata/insertAll 
        
                   Args: 
        
                       table (Union[ \ 
        
                           google.cloud.bigquery.table.Table, \ 
        
                           google.cloud.bigquery.table.TableReference, \ 
        
                           str, \ 
        
                       ]): 
        
                           The destination table for the row data, or a reference to it. 
        
                       rows (Union[Sequence[Tuple], Sequence[dict]]): 
        
                           Row data to be inserted. If a list of tuples is given, each 
        
                           tuple should contain data for each schema field on the 
        
                           current table and in the same order as the schema fields. If 
        
                           a list of dictionaries is given, the keys must include all 
        
                           required fields in the schema. Keys which do not correspond 
        
                           to a field in the schema are ignored.

Made it with isinstance(). It'll be True for sequences, but not for dicts:

maybe we should do the same thing for insert_rows()? The situation in it is very close to insert_rows_json():

Yes, that's a good idea.

IlyaFaer added 2 commits January 15, 2020 13:43

feat(bigquery): check json_rows arg type in insert_rows_json()

fc1f245

Spelling

cb32c36

IlyaFaer added type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. api: bigquery Issues related to the BigQuery API. labels Jan 16, 2020

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 16, 2020

IlyaFaer commented Jan 16, 2020

View reviewed changes

IlyaFaer marked this pull request as ready for review January 16, 2020 13:46

IlyaFaer requested a review from a team January 16, 2020 13:46

tswast approved these changes Jan 17, 2020

View reviewed changes

tswast merged commit ae54dc3 into googleapis:master Jan 17, 2020

IlyaFaer mentioned this pull request Jan 20, 2020

feat(bigquery): check rows arg type in insert_rows() #10174

Merged

This was referenced Jan 30, 2020

[CHANGE ME] Re-generated container to pick up changes in the API or client library generator. #10256

Closed

[CHANGE ME] Re-generated datastore to pick up changes in the API or client library generator. #10261

Closed

                               the mappings describing one or more problems with the row.
                       """
+                      if not isinstance(json_rows, collections_abc.Sequence):
+                          raise TypeError("json_rows argument should be a sequence of dicts")

feat(bigquery): check json_rows arg type in insert_rows_json() by IlyaFaer · Pull Request #10162 · googleapis/google-cloud-python · GitHub

Uh oh!

feat(bigquery): check json_rows arg type in insert_rows_json() - #10162

feat(bigquery): check json_rows arg type in insert_rows_json()#10162
tswast merged 2 commits into
googleapis:masterfrom
MaxxleLLC:type_error_on_dict

IlyaFaer commented Jan 16, 2020

Uh oh!

IlyaFaer left a comment •
edited
Loading

Uh oh!

Uh oh!

IlyaFaer Jan 16, 2020

Uh oh!

tswast commented Jan 17, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	def insert_rows(self, table, rows, selected_fields=None, **kwargs):
	"""Insert rows into a table via the streaming API.

	See
	https://cloud.google.com/bigquery/docs/reference/rest/v2/tabledata/insertAll

	Args:
	table (Union[ \
	google.cloud.bigquery.table.Table, \
	google.cloud.bigquery.table.TableReference, \
	str, \
	]):
	The destination table for the row data, or a reference to it.
	rows (Union[Sequence[Tuple], Sequence[dict]]):
	Row data to be inserted. If a list of tuples is given, each
	tuple should contain data for each schema field on the
	current table and in the same order as the schema fields. If
	a list of dictionaries is given, the keys must include all
	required fields in the schema. Keys which do not correspond
	to a field in the schema are ignored.

feat(bigquery): check json_rows arg type in insert_rows_json() by IlyaFaer · Pull Request #10162 · googleapis/google-cloud-python · GitHub

Uh oh!

feat(bigquery): check json_rows arg type in insert_rows_json() - #10162

feat(bigquery): check json_rows arg type in insert_rows_json()#10162tswast merged 2 commits intogoogleapis:mastergoogleapis/google-cloud-python:masterfrom MaxxleLLC:type_error_on_dictMaxxleLLC/google-cloud-python:type_error_on_dictCopy head branch name to clipboard

Conversation

IlyaFaer commented Jan 16, 2020

Uh oh!

IlyaFaer left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

IlyaFaer Jan 16, 2020

Choose a reason for hiding this comment

Uh oh!

tswast commented Jan 17, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

feat(bigquery): check json_rows arg type in insert_rows_json()#10162
tswast merged 2 commits into
googleapis:masterfrom
MaxxleLLC:type_error_on_dict

IlyaFaer left a comment •
edited
Loading