Skip to content
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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

perf: remove redundant array deepcopy #26

Merged
merged 7 commits into from Oct 7, 2020
Merged

perf: remove redundant array deepcopy #26

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
bab93e0
perf(bigquery): remove redundant array deepcopy
grooveygr Feb 6, 2020
7343175
Merge branch 'master' into master
tseaver Jul 30, 2020
38a529e
Merge branch 'master' into master
tseaver Aug 6, 2020
0a1643e
Merge branch 'master' into master
tswast Sep 24, 2020
f60bd11
Merge branch 'master' into master
tswast Oct 6, 2020
c9d862d
add check to ignore REPEATED mode
tswast Oct 6, 2020
54852ae
Update google/cloud/bigquery/_helpers.py
tswast Oct 6, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 5 additions & 1 deletion google/cloud/bigquery/_helpers.py
View file
Open in desktop
Expand Up @@ -458,7 +458,11 @@ def _record_field_to_json(fields, row_value):


def _single_field_to_json(field, row_value):
"""Convert a single (non-repeating) field into JSON-serializable values.
"""Convert a single field into JSON-serializable values.

Ignores mode so that this can function for ARRAY / REPEATING fiels
tswast marked this conversation as resolved.
Show resolved Hide resolved
without requiring a deepcopy of the field. See:
https://github.com/googleapis/python-bigquery/issues/6

Args:
field (google.cloud.bigquery.schema.SchemaField):
Expand Down
6 changes: 6 additions & 0 deletions tests/unit/test__helpers.py
View file
Open in desktop
Expand Up @@ -834,6 +834,12 @@ def test_w_scalar(self):
converted = self._call_fut(field, original)
self.assertEqual(converted, str(original))

def test_w_scalar_ignores_mode(self):
field = _make_field("STRING", mode="REPEATED")
original = "hello world"
converted = self._call_fut(field, original)
self.assertEqual(converted, original)


class Test_repeated_field_to_json(unittest.TestCase):
def _call_fut(self, field, value):
Expand Down