Don't wrap every exception thrown by the connector with VectorStoreOperationException #11512

Activity

1. Adam Sitnik (adamsitnik)

   added

   BuildFeatures planned for next Build conferenceFeatures planned for next Build conference

   msft.ext.vectordataRelated to Microsoft.Extensions.VectorDataRelated to Microsoft.Extensions.VectorData

   on Apr 11, 2025
2. Mark Wallace (markwallace-microsoft)

   added

   triage

   and removed

   triage

   on Apr 11, 2025
3. Mark Wallace (markwallace-microsoft)

   added this to Semantic Kernelon Apr 14, 2025
4. Mark Wallace (markwallace-microsoft)

   moved this to Backlog: Planned in Semantic Kernelon Apr 14, 2025
5. Mark Wallace (markwallace-microsoft)

   assigned

   westey (westey-m)

   on Apr 15, 2025
6. roji commented on Apr 15, 2025

   

   Shay Rojansky (roji)

   on Apr 15, 2025

   Member

   More actions

   While doing this, also ensure that we're indeed wrapping the relevant connector-specific exceptions everywhere, as discussed in #10883.
7. westey (westey-m)

   moved this from Backlog: Planned to Sprint: Planned in Semantic Kernelon Apr 15, 2025
8. westey (westey-m)

   moved this from Sprint: Planned to Sprint: In Review in Semantic Kernelon Apr 23, 2025
9. westey (westey-m)

   moved this from Sprint: In Review to Sprint: Planned in Semantic Kernelon Apr 23, 2025
10. westey (westey-m)

    moved this from Sprint: Planned to Sprint: In Progress in Semantic Kernelon Apr 23, 2025
11. stephentoub commented on May 1, 2025

    

    Stephen Toub (stephentoub)

    on May 1, 2025

    Last edited by stephentoub

    Member

    More actions

    TaskCanceledException

    TaskCanceledException is a mistake I regret.

    It derives from OperationCanceledException, which is the one everyone should pay attention to. If TaskCanceledException is being referenced anywhere, it's likely a bug to be fixed, replaced by OCE.

    So, for the purposes of this issue, OCE shouldn't be wrapped, either.

    👍React with 👍3Reacted by Shay Rojansky, westey and Adam Sitnik
12. Shay Rojansky (roji)

    moved this from Sprint: In Progress to Sprint: In Review in Semantic Kernelon May 2, 2025
13. westey (westey-m)

    added a commit that references this issue on May 2, 2025

    .Net: MEVD: Add consistent error handling (#11845)

    ...

    fb341a7
14. Shay Rojansky (roji)

    moved this from Sprint: In Review to Sprint: Done in Semantic Kernelon May 2, 2025
15. westey (westey-m)

    closed this as completedon May 2, 2025