Add caching for task counts

Add caching for task counts

The UI only allows for selection to the nearest minute, so memcache
assumes the end timestamp is also at a minute resolution.

BUG=673774
Review-Url: https://codereview.chromium.org/2697333002
Committed: https://github.com/luci/luci-py/commit/61230b5f8392628a96ce30eb1d1c1379e78b9cff

[kjlubick, 2017-02-16 16:26:52]

kjlubick@google.com changed reviewers:
+ maruel@chromium.org

[M-A Ruel, 2017-02-16 18:15:19]

lgtm with fixes

https://codereview.chromium.org/2697333002/diff/1/appengine/swarming/handlers...
File appengine/swarming/handlers_endpoints.py (right):

https://codereview.chromium.org/2697333002/diff/1/appengine/swarming/handlers...
appengine/swarming/handlers_endpoints.py:11: from google.appengine.api import
memcache as gae_memcache
why rename?

https://codereview.chromium.org/2697333002/diff/1/appengine/swarming/handlers...
appengine/swarming/handlers_endpoints.py:516: count = gae_memcache.get(mem_key)
use a namespace, e.g. 'tasks_count'

https://codereview.chromium.org/2697333002/diff/1/appengine/swarming/handlers...
appengine/swarming/handlers_endpoints.py:522: gae_memcache.add(mem_key, count,
120)
technically the entry is valid for a while since you encode the end time, you
could use 24*60*60 as the expiration.

[kjlubick, 2017-02-17 14:49:13]

The CQ bit was checked by kjlubick@google.com

[kjlubick, 2017-02-17 14:49:14]

The patchset sent to the CQ was uploaded after l-g-t-m from maruel@chromium.org
Link to the patchset: https://codereview.chromium.org/2697333002/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2017-02-17 14:49:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-17 14:53:19]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487342953845820,
"parent_rev": "ff0ee7ec014926370e98e576f767f0381d18c2c1", "commit_rev":
"61230b5f8392628a96ce30eb1d1c1379e78b9cff"}

[commit-bot: I haz the power, 2017-02-17 14:53:22]

Description was changed from

==========
Add caching for task counts

The UI only allows for selection to the nearest minute, so memcache
assumes the end timestamp is also at a minute resolution.

BUG=673774
==========

to

==========
Add caching for task counts

The UI only allows for selection to the nearest minute, so memcache
assumes the end timestamp is also at a minute resolution.

BUG=673774

Review-Url: https://codereview.chromium.org/2697333002
Committed:
https://github.com/luci/luci-py/commit/61230b5f8392628a96ce30eb1d1c1379e78b9cff
==========

[commit-bot: I haz the power, 2017-02-17 14:53:23]

Committed patchset #3 (id:40001) as
https://github.com/luci/luci-py/commit/61230b5f8392628a96ce30eb1d1c1379e78b9cff

[kjlubick, 2017-02-17 15:54:50]

https://codereview.chromium.org/2697333002/diff/1/appengine/swarming/handlers...
File appengine/swarming/handlers_endpoints.py (right):

https://codereview.chromium.org/2697333002/diff/1/appengine/swarming/handlers...
appengine/swarming/handlers_endpoints.py:11: from google.appengine.api import
memcache as gae_memcache
On 2017/02/16 at 18:15:19, M-A Ruel wrote:
> why rename?

It was to be consistent with components/utils.py.  I'll un-rename it.

https://codereview.chromium.org/2697333002/diff/1/appengine/swarming/handlers...
appengine/swarming/handlers_endpoints.py:522: gae_memcache.add(mem_key, count,
120)
On 2017/02/16 at 18:15:19, M-A Ruel wrote:
> technically the entry is valid for a while since you encode the end time, you
could use 24*60*60 as the expiration.

Done