.NET: fix: filter filesystem checkpoint index by session#6132
.NET: fix: filter filesystem checkpoint index by session#6132he-yufeng wants to merge 2 commits into
Conversation
| this.CheckDisposed(); | ||
|
|
||
| return new(this.CheckpointIndex); | ||
| return new(this.CheckpointIndex.Where(checkpoint => checkpoint.SessionId == sessionId).ToArray()); |
There was a problem hiding this comment.
Please also include filtering by withParent
fc2cb82 to
665462f
Compare
|
Updated in 04e1cba to include the parent filter as well. The filesystem index now persists the parent checkpoint id, reloads it with the index, and applies withParent in RetrieveIndexAsync. I also added coverage for both the live store and a reopened store so the persisted index path is covered. Validation:
Note: this machine has only .NET SDK 10.0.203, so direct dotnet test hits the Microsoft.Testing.Platform/VSTest entrypoint error. I ran the generated MTP test app directly after build. |
334acbe to
9bf2a5b
Compare
|
Rebased this branch onto current Validation on Windows: |
9bf2a5b to
576da57
Compare
|
Rebased onto the latest Validation:
|
576da57 to
c681aa3
Compare
|
Rebased onto the latest The requested Current local check against fetched Validation on Windows: |
c681aa3 to
fae6a63
Compare
|
Rebased onto current Validation on Windows: |
Summary
Fixes #5942.
To verify
Note: direct dotnet test is blocked locally by the Microsoft.Testing.Platform / .NET 10 VSTest entrypoint error, so I ran the generated test runner executable instead.