Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(341)

Issue 2360383002: Use an empty list for tags that there are too many (Closed)

Created:
4 years, 3 months ago by kjlubick
Modified:
4 years, 3 months ago
Reviewers:
M-A Ruel
CC:
chromium-reviews, infra-reviews+luci-py_chromium.org
Base URL:
git@github.com:luci/luci-py@updatelinks
Target Ref:
refs/heads/master
Project:
luci-py
Visibility:
Public.

Description

Use an empty list for tags that there are too many This way the client-side UI knows the columns it should display, even if there are too many values to keep track of. BUG=631047 Committed: https://github.com/luci/luci-py/commit/6de7f7cac69339a8a8c56a1f52cf04bf388ee448

Patch Set 1 #

Patch Set 2 : Remove logs #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -4 lines) Patch
M appengine/swarming/handlers_backend.py View 1 1 chunk +3 lines, -4 lines 0 comments Download

Depends on Patchset:

Dependent Patchsets:

Messages

Total messages: 7 (3 generated)
kjlubick
4 years, 3 months ago (2016-09-22 19:15:01 UTC) #2
M-A Ruel
lgtm
4 years, 3 months ago (2016-09-22 20:31:28 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2360383002/20001
4 years, 3 months ago (2016-09-23 11:34:46 UTC) #5
commit-bot: I haz the power
4 years, 3 months ago (2016-09-23 11:38:21 UTC) #7
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://github.com/luci/luci-py/commit/6de7f7cac69339a8a8c56a1f52cf04bf388ee448

Powered by Google App Engine
This is Rietveld 408576698