Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(4597)

Issue 215660043: Added stats to Endpoints API. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years, 11 months ago by cmassaro
Modified:
8 years, 11 months ago
Reviewers:
vadimsh, M-A
CC:
swarming-eng_googlegroups.com
Base URL:
https://chromium.googlesource.com/infra/swarming.git@master
Visibility:
Public.

Description

Added stats to Endpoints API. R=maruel@chromium.org, vadimsh@chromium.org BUG= Committed: https://code.google.com/p/swarming/source/detail?repo=default&r=244a9d760df0a79ee59508dd94f3afbbf886d25b

Patch Set 1 #

Total comments: 2

Patch Set 2 : Used 0 Default #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -7 lines) Patch
M appengine/isolate/endpoint_handlers_api.py View 1 3 chunks +20 lines, -7 lines 0 comments Download

Messages

Total messages: 4
cmassaro
8 years, 11 months ago (2015-03-25 21:07:25 UTC) #1
M-A
lgtm with question https://codereview.appspot.com/215660043/diff/1/appengine/isolate/endpoint_handlers_api.py File appengine/isolate/endpoint_handlers_api.py (right): https://codereview.appspot.com/215660043/diff/1/appengine/isolate/endpoint_handlers_api.py#newcode237 appengine/isolate/endpoint_handlers_api.py:237: offset = request.offset or 0 Why ...
8 years, 11 months ago (2015-03-25 21:08:44 UTC) #2
cmassaro
https://codereview.appspot.com/215660043/diff/1/appengine/isolate/endpoint_handlers_api.py File appengine/isolate/endpoint_handlers_api.py (right): https://codereview.appspot.com/215660043/diff/1/appengine/isolate/endpoint_handlers_api.py#newcode237 appengine/isolate/endpoint_handlers_api.py:237: offset = request.offset or 0 On 2015/03/25 21:08:44, M-A ...
8 years, 11 months ago (2015-03-25 21:11:49 UTC) #3
cmassaro
8 years, 11 months ago (2015-03-25 21:12:01 UTC) #4
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
244a9d760df0a79ee59508dd94f3afbbf886d25b (presubmit successful).
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b