brawer / osmdiffs / 34938322400

93%

push

github

fix: give every osm.prune/osm.assemble stage a real staleness check (#809)

Almost every intermediate stage in prune.rs and assemble.rs (plus
import_osm's own top-level shortcut) only checked whether its cached
output files existed, never whether the inputs they were built from
had changed since -- so reusing a workdir against an updated planet
PBF could silently keep serving output built from the old one. That's
also why CoordTable/GraphTable/U64Set/BlobTable/GeometryTable's
modified() accessors were dead code: no staleness check ever reached
them.

Adds BlobReader<File>::modified() (the raw planet file's mtime) and
pipeline::earliest_modified() (the output-side counterpart to the
existing last_modified()), and wires real input-vs-output mtime
comparisons into prune_relations_pass_1/2, prune_ways, prune_nodes,
assemble_nodes, assemble_ways, assemble_leaf_relations,
assemble_super_relations, and import_osm's shortcut (falling back to
the old existence-only check there if the planet file itself has since
been deleted, since there's then nothing to compare against).

Fixes #704.

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>

106 of 186 new or added lines in 4 files covered. (56.99%)

10490 of 11311 relevant lines covered (92.74%)

1369.89 hits per line