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

Issue 657533002: Decreased num alerts in history per page to 5 to avoid exceed memory and response limit (Closed)

Created:
6 years, 2 months ago by Sergiy Byelozyorov
Modified:
6 years, 2 months ago
Reviewers:
Michael Achenbach, ojan
CC:
ojan, blink-reviews, cbiesinger, dsinclair, jochen (gone - plz use gerrit), leviw_travelin_and_unemployed, michaelpg, szager1, teravest
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Project:
blink
Visibility:
Public.

Description

Decreased num alerts in history per page to 5 to avoid exceeding memory and response limit R=machenbach@chromium.org,ojan@chromium.org BUG=416182 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183738

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M Tools/GardeningServer/alerts.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 19 (8 generated)
Sergiy Byelozyorov
6 years, 2 months ago (2014-10-14 12:30:37 UTC) #1
Michael Achenbach
non-owner rubber-stamp lgtm - any issue number for this?
6 years, 2 months ago (2014-10-14 12:40:31 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/657533002/1
6 years, 2 months ago (2014-10-14 12:41:56 UTC) #4
commit-bot: I haz the power
Try jobs failed on following builders: blink_presubmit on tryserver.blink (http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/17367)
6 years, 2 months ago (2014-10-14 12:48:45 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/657533002/1
6 years, 2 months ago (2014-10-14 13:24:28 UTC) #8
commit-bot: I haz the power
Try jobs failed on following builders: blink_presubmit on tryserver.blink (http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/17374)
6 years, 2 months ago (2014-10-14 13:31:00 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/657533002/1
6 years, 2 months ago (2014-10-14 18:36:46 UTC) #12
commit-bot: I haz the power
Try jobs failed on following builders: blink_presubmit on tryserver.blink (http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/17400)
6 years, 2 months ago (2014-10-14 18:45:59 UTC) #14
ojan
I didn't notice this in the original review. I think we should have two different ...
6 years, 2 months ago (2014-10-14 23:50:33 UTC) #16
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/657533002/1
6 years, 2 months ago (2014-10-15 11:19:15 UTC) #18
commit-bot: I haz the power
6 years, 2 months ago (2014-10-15 11:22:58 UTC) #19
Message was sent while issue was closed.
Committed patchset #1 (id:1) as 183738

Powered by Google App Engine
This is Rietveld 408576698