mirror of
https://github.com/meilisearch/MeiliSearch
synced 2025-01-09 13:04:30 +01:00
baeefa4817
5166: fix list indexes r=dureuill a=irevoire # Pull Request ### Smol benchmark on a meilisearch with 1009 indexes: **Before** this PR on my computer, it was taking 5.5s to call the `GET /indexes` route on a cold computer where all the indexes were closed. **After** this PR it takes 0.009s to call the route on the first 20 indexes, and 0.176 for the last 20 indexes (retrieving the first or last indexes on main has no impact on performances). If my computations are right, that's between 61111.1% and 3125% faster on this test 😂 ## Related issue Fixes https://github.com/meilisearch/meilisearch/issues/4694 ## What does this PR do? - Add the primary key to the cache we already have in the index-mapper - Provide a new route to retrieve the paginated indexes straight from the cache without opening them - Fix a bug where the cache was not computed when loading a dump and was forcing us to open the indexes to compute their stats on the fly ## Is it breaking? Since the field I added is an `Option` I think we should consider it as non-breaking and let it update itself automatically on the next operation of this index. I also tested to run my patch over a DB generated on release-v1.12.0 and it works. The importing a dump also works. Co-authored-by: Tamo <tamo@meilisearch.com>