-
-
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
FEAT: Explicit Polymorphic Relationship Support [emberjs/rfcs#793] #7955
Conversation
Asset Size Report for 5255537 Modern Builds 🛑 The size of the library EmberData has increased by +2.06 KB (+441.0 B compressed) which exceeds the failure threshold of 75 bytes.Warnings
Changeset
Full Asset Analysis (Modern)
Modern Builds (No Rollup) 🛑 The size of the library EmberData has increased by +2.01 KB (+413.0 B compressed) which exceeds the failure threshold of 75 bytes.Warnings
Changeset
Full Asset Analysis (Modern)
|
41d1c6f
to
53cf8e1
Compare
Performance Report for 5255537 Scenario - basic-record-materialization: ☑️ Performance is stable
Scenario - relationship-materialization-simple: ☑️ Performance is stable
Scenario - relationship-materialization-complex: ☑️ Performance is stable
Scenario - unload: ☑️ Performance is stable
Scenario - unload-all: ☑️ Performance is stable
Scenario - destroy: ☑️ Performance is stable
Scenario - add-children: ☑️ Performance is stable
Scenario - unused-relationships: ☑️ Performance is stable
|
118137e
to
0e07474
Compare
Commit v Release Performance Report for 5255537 Scenario - basic-record-materialization: ✅ Performance improved
Scenario - relationship-materialization-simple: ✅ Performance improved
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
|
c920915
to
4dd2342
Compare
Implements emberjs/rfcs#793
Note, these are mostly covered ad-hoc by other portions of the test suite, but we should take the opportunity to write clear scoped tests:
resolves #8150 by introducing the
sync
API to the cache which currently takes one operation:MergeIdentifier
allowing it to recombine data as desired. The current implementation of merging could probably be enhanced quite a bit.