geo-engine / geoengine / 42035167676

88%

main: 88%

push

github

fix(backend): drop abandoned GDAL reads and unused broker slots (#1401)

* fix(backend): drop queued GDAL reads that no caller waits for

The detached leader task of a deduplicated tile read always held the
result channel, so the broker's check for cancelled requests never fired:
tiles of aborted queries, e.g. after panning a map, were still read before
the tiles the client still waits for. The leader now gives up once all
callers are gone, so the broker drops the request if it is still queued.
Callers that join while the leader gives up are handled under the
in-flight map lock.

Callers can now give up between the broker's scan for the next request
and its dispatch. The request is therefore taken before the worker, and a
dataset without waiting callers is skipped instead of panicking.

* fix(backend): remove unused dataset slots from the GDAL broker

The registry kept one entry for every dataset ever read, e.g. every STAC
scene file, so it grew for the whole runtime of the server. A slot is now
removed once nothing of its dataset is queued or read, and created again
by the next request.

168 of 174 new or added lines in 1 file covered. (96.55%)

138674 of 157404 relevant lines covered (88.1%)

533752.93 hits per line