suite report generator should hit slave db rather than master |
||||||||
Issue descriptionRelated to crbug.com/695287 Evidence points to suite report generator causing undue extra load on master db, causing sporadic overload.
,
Feb 25 2017
Excellent. Afaik tko overload is not a problem so far, though hitting a readonly replica for that sounds like a prudent idea.
,
Feb 25 2017
,
Feb 25 2017
Will shuqianz handle the change to tko/parse or should I? If me, what's the official name for the slave DB?
,
Mar 1 2017
Ping on this bug.
,
Mar 1 2017
So the --afe option will allow us to hit a different rpc server (right? I'm pretty sure your script is making afe rpcs, not direct db queries?) If so I don't think that will actually help us hit a slave db. We'd need to make the rpc server smart enough to route read-only requests to the slave db (or, allow the rpc client to pass in a hint about which backend db they want to hit). +dshi for comment.
,
Mar 2 2017
Where's the RPC server code? Is it autotest/files/frontend?
,
Mar 2 2017
The rpc code is in frontend/afe/rpc_interface.py We need a new afe role with some special shadow config setting for it to point to slave db.
,
Mar 2 2017
Are there AFE slave replicas already? Are there consistency issues with operations that might interleave reads and writes and having those operations hit different database instances?
,
Mar 2 2017
This change looks like it reverts functionality which previously did this. https://chromium-review.googlesource.com/223393
,
Mar 7 2017
,
Mar 10 2017
dshi@: Can you respond to c#9?
,
Mar 10 2017
No, secondary AFE role to point to secondary db server is not implemented yet. We need to create a new role for it (most likely just deploying new shadow config file). The delay between master and slave db should be minimum, like <1min
,
Jun 28 2017
De-prioritizing since suite report is making use of new index.
,
May 11 2018
,
May 28 2018
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by davidri...@chromium.org
, Feb 25 2017