Skip to content

Conversation

@adamsitnik
Copy link
Member

This PR fixes a bug reported offline by @jeffhandley

fail: Microsoft.Extensions.DataIngestion.IngestionPipeline[6]
      An error occurred while ingesting document 'Example_GPS_Watch.md'.
      System.InvalidCastException: Unable to cast object of type 'System.Guid' to type 'System.String'.
         at System.Linq.Enumerable.CastICollectionIterator`1.MoveNext()
         at Azure.Search.Documents.Models.IndexDocumentsBatch.Delete(String keyName, IEnumerable`1 keyValues)
         at Azure.Search.Documents.SearchClient.DeleteDocumentsAsync(String keyName, IEnumerable`1 keyValues, IndexDocumentsOptions options, CancellationToken cancellationToken)
         at Microsoft.Extensions.VectorData.VectorStoreErrorHandler.RunOperationAsync[TResult,TException](VectorStoreCollectionMetadata metadata, String operationName, Func`1 operation)
         at Microsoft.Extensions.DataIngestion.VectorStoreWriter`1.WriteAsync(IAsyncEnumerable`1 chunks, CancellationToken cancellationToken)
         at Microsoft.Extensions.DataIngestion.IngestionPipeline`1.IngestAsync(IngestionDocument document, Activity parentActivity, CancellationToken cancellationToken)
         at Microsoft.Extensions.DataIngestion.IngestionPipeline`1.ProcessAsync(IEnumerable`1 files, Activity rootActivity, CancellationToken cancellationToken)+MoveNext()

@adamsitnik adamsitnik requested a review from roji November 10, 2025 13:15
@adamsitnik adamsitnik requested a review from a team as a code owner November 10, 2025 13:15
@adamsitnik adamsitnik added the msft.ext.vectordata Related to Microsoft.Extensions.VectorData label Nov 10, 2025
@markwallace-microsoft markwallace-microsoft added the .NET Issue or Pull requests regarding .NET code label Nov 10, 2025
@github-actions github-actions bot changed the title [.NET] Azure AI Search: don't assume that TKey is always a string .Net: [.NET] Azure AI Search: don't assume that TKey is always a string Nov 10, 2025
@adamsitnik adamsitnik changed the title .Net: [.NET] Azure AI Search: don't assume that TKey is always a string .Net: Azure AI Search: don't assume that TKey is always a string Nov 10, 2025
@adamsitnik adamsitnik requested a review from westey-m November 10, 2025 14:05
@adamsitnik adamsitnik enabled auto-merge November 10, 2025 15:14
@roji roji changed the title .Net: Azure AI Search: don't assume that TKey is always a string Fixes to Get/Delete for Guid keys Nov 13, 2025
@adamsitnik adamsitnik added this pull request to the merge queue Nov 13, 2025
Merged via the queue into microsoft:main with commit de20575 Nov 13, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

msft.ext.vectordata Related to Microsoft.Extensions.VectorData .NET Issue or Pull requests regarding .NET code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants