New issue
Advanced search Search tips

Issue 613978 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Docs: figure out how infra pollers handle too many refs

Project Member Reported by tandrii@chromium.org, May 23 2016

Issue description

The refs in gitiles https://repo/url.git/+refs?format=json will now show all gerrit patches, including closed ones. I am not sure how this scales and whether we have any implicit assumptions that refs endpoint will return   mostly bounded number of refs. 
 
Labels: Pri-2
Labels: Type-Bug
Labels: Proj-Gerrit-Migration
Status: Available (was: Untriaged)
Labels: -Restrict-View-Google -Pri-2 Pri-3
I think we'll see this as we migrate V8 and WebRTC :) It's hard to predict this.
Labels: -Pri-3 Milestone-Launch Pri-2
Summary: Docs: figure out how infra pollers handle too many refs (was: Gerrit cls: figure out how infra pollers handle too many refs)
We should send an email out to infra informing them about potential issues. Also document it within the playbook?

Comment 6 by aga...@chromium.org, Jan 13 2017

Labels: -Milestone-Launch Milestone-Dogfood
Owner: andyb...@chromium.org
Status: Assigned (was: Available)
Right now, the chromite refs json is >4MB. Chromium/src.git will hit that in a year and keep growing past that. We need the ability to get a subset of the refs.

Proposed api:
foo.googlesource.com/repo/+refs/heads?format=json only returns refs in the "refs/heads" namespace.
foo.googlesource.com/repo/+refs/changes?format=json only returns CL refs

Might also be useful to have a paging/continuation token system just for the sake of good design.

Comment 7 by aga...@chromium.org, Mar 27 2017

Owner: aga...@chromium.org
Status: Fixed (was: Assigned)
The ?format=text and ?format=json pages now support filtering refs by path.

Sign in to add a comment