Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Add deleteAll to the orm #2519

Merged
merged 2 commits into from
Oct 17, 2022
Merged

Add deleteAll to the orm #2519

merged 2 commits into from
Oct 17, 2022

Conversation

chkeita
Copy link
Contributor

@chkeita chkeita commented Oct 14, 2022

Summary of the Pull Request

Introduce a DeleteAll operation to speedup the deletion of multiple entities.

The new method is able to delete 13404 Nodemessges in 4s

@chkeita chkeita requested review from Porges, stishkin and tevoinea and removed request for Porges October 14, 2022 20:39
@codecov-commenter
Copy link

codecov-commenter commented Oct 14, 2022

Codecov Report

Merging #2519 (e2a2eed) into main (8a4b46e) will increase coverage by 0.03%.
The diff coverage is 50.94%.

@@            Coverage Diff             @@
##             main    #2519      +/-   ##
==========================================
+ Coverage   29.95%   29.99%   +0.03%     
==========================================
  Files         289      289              
  Lines       35432    35480      +48     
==========================================
+ Hits        10615    10641      +26     
- Misses      24817    24839      +22     
Impacted Files Coverage Δ
src/ApiService/ApiService/onefuzzlib/Utils.cs 50.00% <42.85%> (-16.67%) ⬇️
src/ApiService/ApiService/onefuzzlib/orm/Orm.cs 65.62% <52.77%> (-3.73%) ⬇️
...ice/ApiService/onefuzzlib/NodeMessageOperations.cs 60.86% <66.66%> (+8.86%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@chkeita chkeita merged commit d95342b into microsoft:main Oct 17, 2022
@AdamL-Microsoft AdamL-Microsoft mentioned this pull request Oct 20, 2022
@chkeita chkeita mentioned this pull request Nov 8, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Nov 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants