7
66%
master: 66%

Ran 31 Oct 2019 05:39PM UTC

Files 108

Run time 7s

Badge

Embed ▾

Committed 31 Oct 2019 05:25PM UTC coverage: 61.863% (-0.2%) from 62.03%

Job # RUN="coverage" CONTAINER="netaccess"

Build Type

push

travis-ci

Committed by

cpu

Commit Message

SA: remove JOIN from GetAuthorizations2 and filter in code (#4512)

Previously we used a JOIN on the orderToAuthz2 table in order to make sure
we only returned authorizations created using the ACME v2 API. Each time an
order is created a pivot row (order ID + authz ID) is added to the
orderToAuthz2 table. If a large number of orders are created that all contain
the same authorization, due to reuse, then the JOINd query would return a full
authorization row for each entry in the orderToAuthz2 table with the authorization
ID.

Instead we now filter out these authorizations by doing a second query against
the orderToAuthz2 table. Using this query still requires examining a large number
of rows, but because we don't need to construct a temporary table for the JOIN
and fill it with all the full authorization rows we should save resources.

Fixes #4500.

Coverage Stats

11767 of 19021 relevant lines covered (61.86%)

0.69 hits per line

letsencrypt / boulder / 11425 / 7
66%
master: 66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 11425.7 (RUN="coverage" CONTAINER="netaccess")

letsencrypt / boulder / 11425 / 7 66% master: 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 11425.7 (RUN="coverage" CONTAINER="netaccess")

letsencrypt / boulder / 11425 / 7
66%
master: 66%

README BADGES
x