milvus-io / milvus

A cloud-native vector database, storage for next generation AI applications
https://milvus.io
Apache License 2.0
29.26k stars 2.81k forks source link

[Bug]: Upsert should be allowed for autoid collection #25397

Open yhmo opened 1 year ago

yhmo commented 1 year ago

Is there an existing issue for this?

Environment

- Milvus version:2.3.0
- Deployment mode(standalone or cluster):
- MQ type(rocksmq, pulsar or kafka):    
- SDK version(e.g. pymilvus v2.0.0rc2):
- OS(Ubuntu or CentOS): 
- CPU/Memory: 
- GPU: 
- Others:

Current Behavior

Create a collection with auto-id primary key. insert some data try call upsert() to update some rows by pk

error thrown: pymilvus.exceptions.UpsertAutoIDTrueException: <UpsertAutoIDTrueException: (code=1, message=Upsert don't support autoid == true)>

Expected Behavior

Upsert should be allowed for autoid collection

Steps To Reproduce

No response

Milvus Log

No response

Anything else?

No response

xiaofan-luan commented 1 year ago

/assign @smellthemoon

xiaofan-luan commented 4 months ago

@smellthemoon how is this pr works?

ed-cho commented 2 months ago

@xiaofan-luan this would be very useful. Is the PR planned to be merged?