1
86%
master: 86%

Ran 29 Nov 2018 04:37PM UTC

Files 79

Run time 55s

Badge

Embed ▾

Committed 29 Nov 2018 04:34PM UTC coverage: 72.513% (-0.2%) from 72.69%

Job # 810.1

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Remove memcache usage from regions.py (and refactor some other methods) (#217)

Part of #176

The main change here is that we don't cache the scraper package and
scraper class name in memcache. Instead, we only parse the region manifest
into a dict once, keep it in memory indefinitely, and get the scraper
package and class name from there.

This has the side effect of making Region object construction much cheaper
as we don't have to parse the full yaml file each time. Because of this,
I moved all of the region specific methods to be Region class methods.
This required some changes throughout the code base, and I may have gotten
a little carried away, so if you prefer any (or all) of them as non-class
methods I am happy to revert them.

In the future where each region has it's own manifest, instead of one large
`region_manifest`, I imagine we will only store the parsed config for a
region once it is needed. If this grows too large, we could put a
replacement policy in place.

Coverage Stats

2456 of 3387 relevant lines covered (72.51%)

0.73 hits per line

Recidiviz / recidiviz-data / 810 / 1
86%
master: 86%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 810.1

Recidiviz / recidiviz-data / 810 / 1 86% master: 86%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 810.1

Recidiviz / recidiviz-data / 810 / 1
86%
master: 86%

README BADGES
x