-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
When updating the primaryKey
, if the previous primaryKey
value has already been used for a document, the API returns an index_primary_key_already_exists error -> ASYNC
#3385
Comments
Not sure we actually want to « fix » this. |
I'm not sure to get the relation with auto-batching |
Basically, if an index doesn't have a pk (or doesn't exists yet) when we need to select which tasks will be batched together, we have nothing to rely on to know which task will correctly set the primary key, and thus, we can't autobatch any two tasks that don't share the same pk. Maybe that's the kind of thing we like, though, I don't really know what to think of it |
Maybe you need to inform @gmourier about this because I'm pretty sure that returning a |
Yeah, I already told him about it, but he doesn't have the time rn; I guess we'll see next week what we want to do with it. |
We can wait for next week, no problem |
It has been decided to merge it because, most of the time it shouldn't be an issue |
3412: When adding documents, trying to update the primary-key now throw an error r=Kerollmops a=irevoire 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 😬 Fixes #3385 Fixes #3411 Co-authored-by: Tamo <tamo@meilisearch.com>
Closed by #3412 |
No description provided.
The text was updated successfully, but these errors were encountered: