-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Closed as not planned
Labels
.NETIssue or Pull requests regarding .NET codeIssue or Pull requests regarding .NET codemsft.ext.vectordataRelated to Microsoft.Extensions.VectorDataRelated to Microsoft.Extensions.VectorData
Description
Important exceptions like “the collection doesn’t exists” should be agnostic of the storage type, to avoid the need to catch several exception types.
For instance, this code would be preferable and wouldn’t need changes when swapping storage type:
Try { … }
catch (Microsoft.Extensions.VectorData.CollectionNotFoundException e)
{ … }as opposed to:
Try { … }
catch (Exception e) when (e is Qdrant.CollectionNotFoundException or Postgres.CollectionNotFoundException or AzureAiSearch.CollectionNotFoundException or Milvus.CollectionNotFoundException or ...)
{ … }Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
.NETIssue or Pull requests regarding .NET codeIssue or Pull requests regarding .NET codemsft.ext.vectordataRelated to Microsoft.Extensions.VectorDataRelated to Microsoft.Extensions.VectorData