Bug 127130 – build.webkit.org/dashboard should not request 50 revisions from trac each time

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Alexey Proskuryakov]

The dashboard should be smarter about how much data it requests from Trac. Currently, it requests 50 most recent revisions, which is OK for the first request, but makes little sense to do every 45 seconds.

We should ask for fewer revisions, and fall back to a larger range on the rare occasions when there are lots of them landed at once.

[Alexey Proskuryakov]

Also, there is some code in Trac to maintain client sessions. I'm not sure what exactly it does, maybe it could be used to optimize performance of periodic requests?

We'd need to allow CORS credentials for the timeline on the server, and enable XHR.withCredentials on the client to have cookies sent.

[Alexey Proskuryakov]

On the second thought, so session tracking through cookies could get tricky when reloading the dashboard page, or having two dashboard windows open for any reason.

[Alexey Proskuryakov]

Created attachment 236791 [details]
proposed patch

Found a different way to make the trac request faster. We don't necessarily get more close to "give me what changed since the last time", but Trac processes date based requests much faster.

Talking about sessions, those don't seem to help, they are for logged in users and only used for credentials and settings like time zone. Having a session would actually be harmful here, as we'd get times converted.

[WebKit Commit Bot]

Comment on attachment 236791 [details]
proposed patch

Clearing flags on attachment: 236791

Committed r172747: <http://trac.webkit.org/changeset/172747>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.