Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Make document addition number visible #178

Merged
merged 1 commit into from
Apr 29, 2021
Merged

Conversation

MarinPostma
Copy link
Contributor

The nb_document needs to be public to allow the conversion of update statuses to the legacy meilisearch format.

Copy link
Member

@Kerollmops Kerollmops left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops, it seems like something I forgot to made public.

@Kerollmops Kerollmops merged commit c112877 into main Apr 29, 2021
@Kerollmops Kerollmops deleted the visible-document-nb branch April 29, 2021 19:54
@curquiza curquiza changed the title make document addition number visible Make document addition number visible Apr 29, 2021
bors bot added a commit to meilisearch/transplant that referenced this pull request Jun 2, 2021
173: convert UpdateStatus to legacy meilisearch format r=MarinPostma a=MarinPostma

Returns the update statuses with the same format as legacy meilisearch.

The number of documents in a document addition/deletion is not known before processing, so it is only returned when the update is `processed`.

close #78 

associated milli PR: meilisearch/milli#178


Co-authored-by: Marin Postma <postma.marin@protonmail.com>
bors bot added a commit to meilisearch/transplant that referenced this pull request Jun 14, 2021
144: Concurrent update run loop (refactor) r=MarinPostma a=MarinPostma

This PR allows multiple request to the update store to be performed concurently (i.e, one can list updates while an updates in being written to the update store).


173: Convert UpdateStatus to legacy meilisearch format r=MarinPostma a=MarinPostma

Returns the update statuses with the same format as legacy meilisearch.

The number of documents in a document addition/deletion is not known before processing, so it is only returned when the update is `processed`.

close #78 

associated milli PR: meilisearch/milli#178


Co-authored-by: marin postma <postma.marin@protonmail.com>
Co-authored-by: Marin Postma <postma.marin@protonmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants