Code review comment for ~mdeslaur/ubuntu-cve-tracker:perf-part4

Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

> Re: the old releaase - I always thought they seemed redundant so I am happy to
> seem them go.
>
> One question - why not use `detailed=True` for the cache so that it is as
> complete as possible since we are generating it out-of-band then we may as
> well put as much in it as we can IMO.

Good point, I'll measure if there's a runtime search performance difference with a bigger dataset, and if it's negligible, I'll switch it to detailed.

« Back to merge proposal