-
Notifications
You must be signed in to change notification settings - Fork 595
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: use TakeSnapshotIfChanged to avoid taking redundant snapshot #6092
Conversation
ead548c
to
e2dfd41
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6092 +/- ##
========================================
- Coverage 74.6% 6.1% -68.5%
========================================
Files 194 106 -88
Lines 18861 9731 -9130
========================================
- Hits 14072 595 -13477
- Misses 3814 9023 +5209
+ Partials 975 113 -862 ☔ View full report in Codecov by Sentry. |
d43abef
to
aa7d6d7
Compare
aa7d6d7
to
1dddde1
Compare
What this PR does / why we need it:
Replaces usage of
TakeSnapshot
withTakeSnapshotIfChanged
that was introduced inWhich issue this PR fixes:
Closes #6080