-
Notifications
You must be signed in to change notification settings - Fork 49
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
removed previous schema preservation for nodes, edges #209
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #209 +/- ##
======================================
Coverage 9.99% 10.00%
======================================
Files 406 406
Lines 30146 30137 -9
Branches 654 654
======================================
Hits 3014 3014
+ Misses 26798 26789 -9
Partials 334 334
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
* removed previous schema preservation for nodes, edges --------- Co-authored-by: Michael Chin <chnmch@amazon.com>
Issue #147:
Description of changes:
The previous schema preservation logic was removed to avoid retaining old values causing incorrect rendering of nodes and edges totals from the cached previous load.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.