-
Notifications
You must be signed in to change notification settings - Fork 31
Closed
Labels
api: spannerIssues related to the googleapis/python-spanner-django API.Issues related to the googleapis/python-spanner-django API.priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Description
I think we have a bug in Cursor.executemany()
method. Given the code it runs Cursor.execute()
in a cycle, while execute()
is actually replacing result of every previous call with a result from the last one.
So, if we, for example, will call executemany()
with several reads, we'll get only results of the last one:
But in fact both reads, with SingerId=12
and with SingerId=15
, were executed and we got results.
@c24t, we already have a proposition of how to deal with executemany()
in the design doc, PTAL at StreamedManyResultSets
section.
Metadata
Metadata
Assignees
Labels
api: spannerIssues related to the googleapis/python-spanner-django API.Issues related to the googleapis/python-spanner-django API.priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.