facebook / akd

An implementation of an auditable key directory
Apache License 2.0
246 stars 36 forks source link

[akd] Fix prefetch bug #440

Closed artem-shu closed 4 months ago

artem-shu commented 4 months ago

My original diff added a bug by moving the pre-fetch down to the code-block where version-range was in scope. This means that the update proof was run before the pre-load, resulting in extra fetches.

https://github.com/facebook/akd/pull/436

This PR should address that my moving the pre-fetch block back to its original position, and moving the range-selection to a separate earlier block.

From initial canary, the latency appears to be improved:

Screenshot 2024-06-26 at 4 01 21 PM