[Findit] Cache bigquery api client |
||||
Issue descriptionRight now the client from bigquery_helper is created every time a call is made, cache this client so it's only constructed once.
,
Apr 20 2018
,
May 22 2018
,
May 31 2018
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/bc52ad839536b6cae8bbb8b2459c6fff6105b555 commit bc52ad839536b6cae8bbb8b2459c6fff6105b555 Author: Yuke Liao <liaoyuke@chromium.org> Date: Thu May 31 21:16:11 2018 [Findit] Refactoring bigquery helper. This CL refactors the biquery helper to cache the client, cleans up stale comments and correctly deals with job incomplete errors. Bug: 807501 Change-Id: I7bc3b80b0610d38d6d34f7fbadb6ad91735c9dc6 Reviewed-on: https://chromium-review.googlesource.com/1079609 Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Reviewed-by: Shuotao Gao <stgao@chromium.org> [modify] https://crrev.com/bc52ad839536b6cae8bbb8b2459c6fff6105b555/appengine/findit/services/test/bigquery_helper_test.py [modify] https://crrev.com/bc52ad839536b6cae8bbb8b2459c6fff6105b555/appengine/findit/services/bigquery_helper.py
,
May 31 2018
|
||||
►
Sign in to add a comment |
||||
Comment 1 by wylieb@chromium.org
, Feb 22 2018Status: Assigned (was: Available)