Bug: Upsert method of SQL Server connectors returns only inserted keys, instead of all keys

[eavanvalkenburg]

Describe the bug
A clear and concise description of what the bug is.
The UpsertMany method of the dotnet SQL connector is inconsistent with other connectors, it returns only inserted keys, instead of all handled keys.

To Reproduce
Steps to reproduce the behavior:

1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Platform

• Language: [e.g. C#, Python]
• Source: [e.g. NuGet package version 0.1.0, pip package version 0.1.0, main branch of repository]
• AI model: [e.g. OpenAI:GPT-4o-mini(2024-07-18)]
• IDE: [e.g. Visual Studio, VS Code]
• OS: [e.g. Windows, Mac]

Additional context
Add any other context about the problem here.

[adamsitnik]

Another great catch @eavanvalkenburg ! We need to discuss #10692 (cc @roji @westey-m )

[adamsitnik]

I can't reproduce the bug. One of our integration test ensures that the keys returned from UpsertBatchAsync contain all keys (not updated):

semantic-kernel/dotnet/src/VectorDataIntegrationTests/VectorDataIntegrationTests/CRUD/BatchConformanceTests.cs

Lines 100 to 120 in a0dce99

	SimpleModel<TKey>[] inserted = Enumerable.Range(0, 10).Select(i => new SimpleModel<TKey>()
	{
	Id = fixture.GenerateNextKey<TKey>(),
	Number = 100 + i,
	Text = i.ToString(),
	Floats = Enumerable.Range(0, SimpleModel<TKey>.DimensionCount).Select(j => (float)(i + j)).ToArray()
	}).ToArray();
	await fixture.Collection.UpsertBatchAsync(inserted).ToArrayAsync();
	SimpleModel<TKey>[] updated = inserted.Select(i => new SimpleModel<TKey>()
	{
	Id = i.Id,
	Text = i.Text + "updated",
	Number = i.Number + 200,
	Floats = i.Floats
	}).ToArray();
	var keys = await fixture.Collection.UpsertBatchAsync(updated).ToArrayAsync();
	Assert.Equal(
	updated.Select(r => r.Id).OrderBy(id => id).ToArray(),
	keys.OrderBy(id => id).ToArray());

I'll write a test for mixed mode (insert some new records and update other records at the same time) and see how it goes