[Telemetry] Make server startup faster and use less RAM.

[Telemetry] Make server startup faster and use less RAM.

This avoids reading in the large .svn directories on the bots.

BUG=None
TEST=intl1 cycler on windows
NOTRY=True


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=190508

[dtu, 2013-03-25 22:30:35]

lgtmnits

https://chromiumcodereview.appspot.com/12767015/diff/1/tools/telemetry/teleme...
File tools/telemetry/telemetry/core/memory_cache_http_server.py (right):

https://chromiumcodereview.appspot.com/12767015/diff/1/tools/telemetry/teleme...
tools/telemetry/telemetry/core/memory_cache_http_server.py:57: files = [f for f
in files if not f[0] == '.']
if f[0] != '.'

https://chromiumcodereview.appspot.com/12767015/diff/1/tools/telemetry/teleme...
tools/telemetry/telemetry/core/memory_cache_http_server.py:58: dirs[:] = [d for
d in dirs if not d[0] == '.']
Just dirs?

[tonyg, 2013-03-25 22:42:39]

https://chromiumcodereview.appspot.com/12767015/diff/1/tools/telemetry/teleme...
File tools/telemetry/telemetry/core/memory_cache_http_server.py (right):

https://chromiumcodereview.appspot.com/12767015/diff/1/tools/telemetry/teleme...
tools/telemetry/telemetry/core/memory_cache_http_server.py:57: files = [f for f
in files if not f[0] == '.']
On 2013/03/25 22:30:35, Dave Tu wrote:
> if f[0] != '.'

Done.

https://chromiumcodereview.appspot.com/12767015/diff/1/tools/telemetry/teleme...
tools/telemetry/telemetry/core/memory_cache_http_server.py:58: dirs[:] = [d for
d in dirs if not d[0] == '.']
On 2013/03/25 22:30:35, Dave Tu wrote:
> Just dirs?

I copied this syntax from stack overflow and then was surprised to see that it
didn't work with the [:] removed. After some googling, it appears like the [:]
syntax is to employ the mutating slice method which alters the list in place
instead of creating a new list.

[nduca, 2013-03-25 22:44:50]

Nice ty

[commit-bot: I haz the power, 2013-03-25 22:48:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tonyg@chromium.org/12767015/6001

[dtu, 2013-03-25 22:59:26]

> I copied this syntax from stack overflow and then was surprised to see that it
> didn't work with the [:] removed. After some googling, it appears like the [:]
> syntax is to employ the mutating slice method which alters the list in place
> instead of creating a new list.


Oh, so the dirs returned by os.walk() is the one that it recurses on internally,
so modifying it changes the list of directories generated. That is... a little
hacky but also kind of elegant.

[commit-bot: I haz the power, 2013-03-25 23:00:23]

Change committed as 190508