meilisearch/MeiliSearch
+
( 23)
-
This PR is a reimplementation of https://github.com/meilisearch/meilisearch/pull/2803, on the new engine. Thanks for your idea and initial PR `@MixusMinimax;` sorry I couldn’t update/merge your PR. Way too many changes happened on the engine in the meantime.
While updating the test suite, I also noticed an issue with the indexed_documents value of failed tasks and had to update it. I also named a bunch of snapshots that had no name, sorry 😬
I also named a bunch of snapshots that had no name sorry 😬
Sorry, This PR is mainly to fix the problems caused by my previously provided PR #3164. It causes multiple ndjson data deserialization failures
`@ivanionut` for your information, I'm sorry I should have better checked before accepting the PR, this is my bad
* reformat, sorry @kero
+ cargo fmt, oops, sorry for the bad history :(
P.S: sorry for this monstrous PR :'(
`@Kerollmops` This should be the last PR for the geosearch and error handling, sorry for doing it in so many steps :grimacing:
Sorry for that, here is the fix
Sorry I left this update in the code (I'm confused because no issues was open to update `facetsDistribution`), there might have been a confusion with `fieldsDistribution` that has been renamed into `fieldDistribution`. Sorry!
@bidoubiwa @curquiza @eskombro, this was a misunderstanding from our side. Doing this would in fact be an error, and would prevent us to do this: https://github.com/meilisearch/MeiliSearch/issues/945#issuecomment-685526678, which is what we are really after. We are resetting this to its default behaviour before it goes in prodution. Sorry for the confusion.
|