Milvus vector store: Pass databaseName in DeleteParam for non-default databases

When using a non-default database, MilvusVectorStore's doDelete method wasn't
passing the specified databaseName in DeleteParam, causing searches to fall back
to the default database where collections couldn't be found. Added explicit
databaseName parameter to fix this issue.

- Add databaseName to DeleteParam builder
- Upgrade milvus-sdk from 2.3.4 to 2.3.5
This commit is contained in:
leevh0908
2024-10-29 23:57:12 +08:00
committed by Soby Chacko
parent cf17a6dccb
commit 3fc03233c0
2 changed files with 2 additions and 1 deletions

View File

@@ -200,7 +200,7 @@
<oracle.version>23.4.0.24.05</oracle.version>
<postgresql.version>42.7.2</postgresql.version>
<elasticsearch-java.version>8.13.3</elasticsearch-java.version>
<milvus.version>2.3.4</milvus.version>
<milvus.version>2.3.5</milvus.version>
<gemfire.testcontainers.version>2.3.0</gemfire.testcontainers.version>
<pinecone.version>0.8.0</pinecone.version>
<fastjson.version>2.0.46</fastjson.version>

View File

@@ -197,6 +197,7 @@ public class MilvusVectorStore extends AbstractObservationVectorStore implements
idList.stream().map(id -> "'" + id + "'").collect(Collectors.joining(",")));
R<MutationResult> status = this.milvusClient.delete(DeleteParam.newBuilder()
.withDatabaseName(this.config.databaseName)
.withCollectionName(this.config.collectionName)
.withExpr(deleteExpression)
.build());