dataset-viewer
refactor(libcommon): remove `Indexer` in favor of local caching in the `rows` endpoint
#3251
Merged

refactor(libcommon): remove `Indexer` in favor of local caching in the `rows` endpoint #3251

kszucs merged 12 commits into main from no-indexer
kszucs
kszucs refactor(libcommon): remove the effectively unused arguments of `Inde…
c7879be5
kszucs style: remove unnecessarry imports
6602e5cb
kszucs refactor(libcommon): remove `Indexer`
1d1acadc
kszucs refactor(services): directly create `RowsIndex` instead of `Indexer`
d352ebb3
kszucs kszucs force pushed from 91d2078d to d352ebb3 92 days ago
kszucs test(libcommon): fix `test_rows_index_query_with_empty_dataset` to us…
78c529fb
kszucs chore: missing import and mypy types
472c0915
kszucs style: fix import order
81c610a9
kszucs kszucs requested a review from lhoestq lhoestq 89 days ago
kszucs
kszucs commented on 2025-11-03
kszucs fix(libcommon): cache the latest instance of `RowsIndex`
0c03093e
kszucs kszucs force pushed from d5d722ad to 0c03093e 89 days ago
kszucs test(libcommon): add a test for caching the latest RowsIndex instance
6309b73d
kszucs kszucs force pushed from 83da75d1 to 6309b73d 89 days ago
kszucs fix(libcommon): only cache RowsIndex when serving from the rows endpoint
bd385e5c
kszucs test(libcommon): remove previously added test case for caching RowInd…
67bad31b
kszucs kszucs changed the title refactor(libcommon): remove `Indexer` refactor(libcommon): remove `Indexer` in favor of local caching in the `rows` endpoint 89 days ago
kszucs chore: missing type annotations
c366f7fe
lhoestq
lhoestq approved these changes on 2025-11-04
kszucs kszucs merged 51ff4634 into main 88 days ago
kszucs kszucs deleted the no-indexer branch 88 days ago

Login to write a write a comment.

Login via GitHub

Reviewers
Assignees
No one assigned
Labels
Milestone