Fixes bug in historical indexing update cache

ponder-sh / ponder

A backend framework for crypto apps

https://ponder.sh

MIT License

609 stars 89 forks source link

Fixes bug in historical indexing update cache #939

Closed kyscott18 closed 3 months ago

kyscott18 commented 3 months ago

The "update" branch of the flush function uses onConflict to overwrite existing database rows, but when there are no rows besides "id", nothing needs to be updated. Previously this code path was causing an SQL syntax error, now it just exits early.

kyscott18 commented 3 months ago

LGTM - I assume that test was failing before the change?

yes