-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
perf: reduce notifications on relationship remote state updates #9655
Conversation
351af8b
to
fb5fae0
Compare
fb5fae0
to
f2f7448
Compare
Performance Report for 212d278 Scenario - basic-record-materialization: ☑️ Performance is stable
Scenario - relationship-materialization-simple: ☑️ Performance is stable
Scenario - relationship-materialization-complex: ✅ Performance improved
Scenario - unload: ☑️ Performance is stable
Scenario - unload-all:
|
Commit v Release Performance Report for 212d278 Scenario - basic-record-materialization: ✅ Performance improved
Scenario - relationship-materialization-simple: ☑️ Performance is stable
Scenario - relationship-materialization-complex: ✅ Performance improved
Scenario - unload: ✅ Performance improved
Scenario - unload-all: ✅ Performance improved
Scenario - destroy: ✅ Performance improved
Scenario - add-children: ✅ Performance improved
Scenario - unused-relationships: ✅ Performance improved
|
* perf: reduce notifications on relationship remote state updates * fix local replace semantics * fix rollback/sort-order * cleanup notifications mroe
* perf: reduce notifications on relationship remote state updates * fix local replace semantics * fix rollback/sort-order * cleanup notifications mroe
If this works, this would fix one issue we noticed in 4.13-canary