diff options
| author | Patrick Cloke <clokep@users.noreply.github.com> | 2022-11-22 09:41:09 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-11-22 09:41:09 -0500 |
| commit | 6d7523ef1484ec56f4a6dffdd2ea3d8736b4cc98 (patch) | |
| tree | 89bc9f7a62a3b08d7a955622810813c225e50523 /tests/rest/client/test_relations.py | |
| parent | Batch fetch bundled annotations (#14491) (diff) | |
| download | synapse-6d7523ef1484ec56f4a6dffdd2ea3d8736b4cc98.tar.xz | |
Batch fetch bundled references (#14508)
Avoid an n+1 query problem and fetch the bundled aggregations for m.reference relations in a single query instead of a query per event. This applies similar logic for as was previously done for edits in 8b309adb436c162510ed1402f33b8741d71fc058 (#11660; threads in b65acead428653b988351ae8d7b22127a22039cd (#11752); and annotations in 1799a54a545618782840a60950ef4b64da9ee24d (#14491).
Diffstat (limited to 'tests/rest/client/test_relations.py')
| -rw-r--r-- | tests/rest/client/test_relations.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/rest/client/test_relations.py b/tests/rest/client/test_relations.py index 2d2b683548..b86f341ff5 100644 --- a/tests/rest/client/test_relations.py +++ b/tests/rest/client/test_relations.py @@ -1108,7 +1108,7 @@ class BundledAggregationsTestCase(BaseRelationsTestCase): # The "user" sent the root event and is making queries for the bundled # aggregations: they have participated. - self._test_bundled_aggregations(RelationTypes.THREAD, _gen_assert(True), 8) + self._test_bundled_aggregations(RelationTypes.THREAD, _gen_assert(True), 7) # The "user2" sent replies in the thread and is making queries for the # bundled aggregations: they have participated. # @@ -1170,7 +1170,7 @@ class BundledAggregationsTestCase(BaseRelationsTestCase): bundled_aggregations["latest_event"].get("unsigned"), ) - self._test_bundled_aggregations(RelationTypes.THREAD, assert_thread, 8) + self._test_bundled_aggregations(RelationTypes.THREAD, assert_thread, 7) def test_nested_thread(self) -> None: """ |