Page MenuHomePhabricator

Use the url limit when sendings metrics to statsv
Closed, ResolvedPublic

Description

The current implementation sends 5 metrics per request, lets make it actually check the limit instead and always try to send as many metrics as possible (but below the limit) as discussed with @Krinkle

Event Timeline

Peter claimed this task.
Peter raised the priority of this task from to High.
Peter updated the task description. (Show Details)
Peter added a project: Performance-Team.

Change 244422 had a related patch set uploaded (by Phedenskog):
Check the statsv limit size before sending metrics

https://gerrit.wikimedia.org/r/244422

Krinkle renamed this task from Use the current limit (1000 characters) when send the metrics to statsv to Use the url limit when sendings metrics to statsv.Oct 19 2015, 6:22 PM
Krinkle set Security to None.

Change 244422 abandoned by Phedenskog:
Check the statsv limit size before sending metrics

Reason:
Got into rebase hell, lets just add this on top, so I can continue to fix the last couple of things.

https://gerrit.wikimedia.org/r/244422

Change 248295 had a related patch set uploaded (by Phedenskog):
Check the statsv limit size before sending metrics

https://gerrit.wikimedia.org/r/248295

Change 248295 merged by jenkins-bot:
Check the statsv limit size before sending metrics

https://gerrit.wikimedia.org/r/248295

Now running in Jenkins, will close this when I see that we have the size of bytes for images.