2018-04-16 13:41:37 -04:00
|
|
|
|
# Copyright 2018 New Vector Ltd
|
|
|
|
|
#
|
|
|
|
|
# Licensed under the Apache License, Version 2.0 (the 'License');
|
|
|
|
|
# you may not use this file except in compliance with the License.
|
|
|
|
|
# You may obtain a copy of the License at
|
|
|
|
|
#
|
|
|
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
|
#
|
|
|
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
|
|
|
# distributed under the License is distributed on an 'AS IS' BASIS,
|
|
|
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
|
|
|
# See the License for the specific language governing permissions and
|
|
|
|
|
# limitations under the License.
|
|
|
|
|
|
2022-09-23 15:01:29 -04:00
|
|
|
|
import datetime
|
|
|
|
|
from typing import Dict, List, Tuple, Union
|
2022-01-24 07:20:01 -05:00
|
|
|
|
|
2021-01-11 11:09:22 -05:00
|
|
|
|
import attr
|
|
|
|
|
from parameterized import parameterized
|
|
|
|
|
|
2022-09-23 15:01:29 -04:00
|
|
|
|
from twisted.test.proto_helpers import MemoryReactor
|
|
|
|
|
|
|
|
|
|
from synapse.api.constants import EventTypes
|
2022-01-24 07:20:01 -05:00
|
|
|
|
from synapse.api.room_versions import (
|
|
|
|
|
KNOWN_ROOM_VERSIONS,
|
|
|
|
|
EventFormatVersions,
|
|
|
|
|
RoomVersion,
|
|
|
|
|
)
|
2021-01-11 11:09:22 -05:00
|
|
|
|
from synapse.events import _EventInternalMetadata
|
2022-09-23 15:01:29 -04:00
|
|
|
|
from synapse.server import HomeServer
|
|
|
|
|
from synapse.storage.database import LoggingTransaction
|
|
|
|
|
from synapse.types import JsonDict
|
|
|
|
|
from synapse.util import Clock, json_encoder
|
2021-01-11 11:09:22 -05:00
|
|
|
|
|
2018-04-16 13:41:37 -04:00
|
|
|
|
import tests.unittest
|
|
|
|
|
import tests.utils
|
|
|
|
|
|
|
|
|
|
|
2022-09-23 15:01:29 -04:00
|
|
|
|
@attr.s(auto_attribs=True, frozen=True, slots=True)
|
|
|
|
|
class _BackfillSetupInfo:
|
|
|
|
|
room_id: str
|
|
|
|
|
depth_map: Dict[str, int]
|
|
|
|
|
|
|
|
|
|
|
2020-03-18 12:46:41 -04:00
|
|
|
|
class EventFederationWorkerStoreTestCase(tests.unittest.HomeserverTestCase):
|
2022-09-23 15:01:29 -04:00
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
2022-02-23 06:04:02 -05:00
|
|
|
|
self.store = hs.get_datastores().main
|
2018-04-16 13:41:37 -04:00
|
|
|
|
|
2020-01-03 11:30:51 -05:00
|
|
|
|
def test_get_prev_events_for_room(self):
|
2018-04-16 13:41:37 -04:00
|
|
|
|
room_id = "@ROOM:local"
|
|
|
|
|
|
|
|
|
|
# add a bunch of events and hashes to act as forward extremities
|
|
|
|
|
def insert_event(txn, i):
|
|
|
|
|
event_id = "$event_%i:local" % i
|
|
|
|
|
|
|
|
|
|
txn.execute(
|
2018-08-10 09:54:09 -04:00
|
|
|
|
(
|
2018-04-16 13:41:37 -04:00
|
|
|
|
"INSERT INTO events ("
|
|
|
|
|
" room_id, event_id, type, depth, topological_ordering,"
|
2018-09-03 12:21:48 -04:00
|
|
|
|
" content, processed, outlier, stream_ordering) "
|
|
|
|
|
"VALUES (?, ?, 'm.test', ?, ?, 'test', ?, ?, ?)"
|
2018-04-16 13:41:37 -04:00
|
|
|
|
),
|
2018-09-03 12:21:48 -04:00
|
|
|
|
(room_id, event_id, i, i, True, False, i),
|
2018-04-16 13:41:37 -04:00
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
txn.execute(
|
2018-08-10 09:54:09 -04:00
|
|
|
|
(
|
2018-04-16 13:41:37 -04:00
|
|
|
|
"INSERT INTO event_forward_extremities (room_id, event_id) "
|
|
|
|
|
"VALUES (?, ?)"
|
|
|
|
|
),
|
|
|
|
|
(room_id, event_id),
|
|
|
|
|
)
|
|
|
|
|
|
2020-01-03 11:30:51 -05:00
|
|
|
|
for i in range(0, 20):
|
2020-08-05 16:38:57 -04:00
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.db_pool.runInteraction("insert", insert_event, i)
|
|
|
|
|
)
|
2018-04-16 13:41:37 -04:00
|
|
|
|
|
2020-01-03 11:30:51 -05:00
|
|
|
|
# this should get the last ten
|
2020-03-18 12:46:41 -04:00
|
|
|
|
r = self.get_success(self.store.get_prev_events_for_room(room_id))
|
2018-04-16 13:41:37 -04:00
|
|
|
|
self.assertEqual(10, len(r))
|
2020-01-03 11:30:51 -05:00
|
|
|
|
for i in range(0, 10):
|
|
|
|
|
self.assertEqual("$event_%i:local" % (19 - i), r[i])
|
2019-09-26 06:47:53 -04:00
|
|
|
|
|
|
|
|
|
def test_get_rooms_with_many_extremities(self):
|
|
|
|
|
room1 = "#room1"
|
|
|
|
|
room2 = "#room2"
|
|
|
|
|
room3 = "#room3"
|
|
|
|
|
|
|
|
|
|
def insert_event(txn, i, room_id):
|
|
|
|
|
event_id = "$event_%i:local" % i
|
|
|
|
|
txn.execute(
|
|
|
|
|
(
|
|
|
|
|
"INSERT INTO event_forward_extremities (room_id, event_id) "
|
|
|
|
|
"VALUES (?, ?)"
|
|
|
|
|
),
|
|
|
|
|
(room_id, event_id),
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
for i in range(0, 20):
|
2020-03-18 12:46:41 -04:00
|
|
|
|
self.get_success(
|
2020-08-05 16:38:57 -04:00
|
|
|
|
self.store.db_pool.runInteraction("insert", insert_event, i, room1)
|
2020-03-18 12:46:41 -04:00
|
|
|
|
)
|
|
|
|
|
self.get_success(
|
2020-08-05 16:38:57 -04:00
|
|
|
|
self.store.db_pool.runInteraction("insert", insert_event, i, room2)
|
2020-03-18 12:46:41 -04:00
|
|
|
|
)
|
|
|
|
|
self.get_success(
|
2020-08-05 16:38:57 -04:00
|
|
|
|
self.store.db_pool.runInteraction("insert", insert_event, i, room3)
|
2020-03-18 12:46:41 -04:00
|
|
|
|
)
|
2019-09-26 06:47:53 -04:00
|
|
|
|
|
|
|
|
|
# Test simple case
|
2020-03-18 12:46:41 -04:00
|
|
|
|
r = self.get_success(self.store.get_rooms_with_many_extremities(5, 5, []))
|
2019-09-26 06:47:53 -04:00
|
|
|
|
self.assertEqual(len(r), 3)
|
|
|
|
|
|
|
|
|
|
# Does filter work?
|
|
|
|
|
|
2020-03-18 12:46:41 -04:00
|
|
|
|
r = self.get_success(self.store.get_rooms_with_many_extremities(5, 5, [room1]))
|
2019-09-26 06:47:53 -04:00
|
|
|
|
self.assertTrue(room2 in r)
|
|
|
|
|
self.assertTrue(room3 in r)
|
|
|
|
|
self.assertEqual(len(r), 2)
|
|
|
|
|
|
2020-03-18 12:46:41 -04:00
|
|
|
|
r = self.get_success(
|
|
|
|
|
self.store.get_rooms_with_many_extremities(5, 5, [room1, room2])
|
|
|
|
|
)
|
2019-09-26 06:47:53 -04:00
|
|
|
|
self.assertEqual(r, [room3])
|
|
|
|
|
|
|
|
|
|
# Does filter and limit work?
|
|
|
|
|
|
2020-03-18 12:46:41 -04:00
|
|
|
|
r = self.get_success(self.store.get_rooms_with_many_extremities(5, 1, [room1]))
|
2019-09-26 06:47:53 -04:00
|
|
|
|
self.assertTrue(r == [room2] or r == [room3])
|
2020-03-18 12:46:41 -04:00
|
|
|
|
|
2021-03-10 09:57:59 -05:00
|
|
|
|
def _setup_auth_chain(self, use_chain_cover_index: bool) -> str:
|
2020-03-18 12:46:41 -04:00
|
|
|
|
room_id = "@ROOM:local"
|
|
|
|
|
|
|
|
|
|
# The silly auth graph we use to test the auth difference algorithm,
|
|
|
|
|
# where the top are the most recent events.
|
|
|
|
|
#
|
|
|
|
|
# A B
|
|
|
|
|
# \ /
|
|
|
|
|
# D E
|
|
|
|
|
# \ |
|
|
|
|
|
# ` F C
|
|
|
|
|
# | /|
|
|
|
|
|
# G ´ |
|
|
|
|
|
# | \ |
|
|
|
|
|
# H I
|
|
|
|
|
# | |
|
|
|
|
|
# K J
|
|
|
|
|
|
|
|
|
|
auth_graph = {
|
|
|
|
|
"a": ["e"],
|
|
|
|
|
"b": ["e"],
|
|
|
|
|
"c": ["g", "i"],
|
|
|
|
|
"d": ["f"],
|
|
|
|
|
"e": ["f"],
|
|
|
|
|
"f": ["g"],
|
|
|
|
|
"g": ["h", "i"],
|
|
|
|
|
"h": ["k"],
|
|
|
|
|
"i": ["j"],
|
|
|
|
|
"k": [],
|
|
|
|
|
"j": [],
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
depth_map = {
|
|
|
|
|
"a": 7,
|
|
|
|
|
"b": 7,
|
|
|
|
|
"c": 4,
|
|
|
|
|
"d": 6,
|
|
|
|
|
"e": 6,
|
|
|
|
|
"f": 5,
|
|
|
|
|
"g": 3,
|
|
|
|
|
"h": 2,
|
|
|
|
|
"i": 2,
|
|
|
|
|
"k": 1,
|
|
|
|
|
"j": 1,
|
|
|
|
|
}
|
|
|
|
|
|
2021-03-10 09:57:59 -05:00
|
|
|
|
# Mark the room as maybe having a cover index.
|
2021-01-11 11:09:22 -05:00
|
|
|
|
|
|
|
|
|
def store_room(txn):
|
|
|
|
|
self.store.db_pool.simple_insert_txn(
|
|
|
|
|
txn,
|
|
|
|
|
"rooms",
|
|
|
|
|
{
|
|
|
|
|
"room_id": room_id,
|
|
|
|
|
"creator": "room_creator_user_id",
|
|
|
|
|
"is_public": True,
|
|
|
|
|
"room_version": "6",
|
|
|
|
|
"has_auth_chain_index": use_chain_cover_index,
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
self.get_success(self.store.db_pool.runInteraction("store_room", store_room))
|
|
|
|
|
|
2020-03-18 12:46:41 -04:00
|
|
|
|
# We rudely fiddle with the appropriate tables directly, as that's much
|
|
|
|
|
# easier than constructing events properly.
|
|
|
|
|
|
2021-01-11 11:09:22 -05:00
|
|
|
|
def insert_event(txn):
|
|
|
|
|
stream_ordering = 0
|
|
|
|
|
|
|
|
|
|
for event_id in auth_graph:
|
|
|
|
|
stream_ordering += 1
|
|
|
|
|
depth = depth_map[event_id]
|
|
|
|
|
|
|
|
|
|
self.store.db_pool.simple_insert_txn(
|
|
|
|
|
txn,
|
|
|
|
|
table="events",
|
|
|
|
|
values={
|
|
|
|
|
"event_id": event_id,
|
|
|
|
|
"room_id": room_id,
|
|
|
|
|
"depth": depth,
|
|
|
|
|
"topological_ordering": depth,
|
|
|
|
|
"type": "m.test",
|
|
|
|
|
"processed": True,
|
|
|
|
|
"outlier": False,
|
|
|
|
|
"stream_ordering": stream_ordering,
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
self.hs.datastores.persist_events._persist_event_auth_chain_txn(
|
|
|
|
|
txn,
|
|
|
|
|
[
|
|
|
|
|
FakeEvent(event_id, room_id, auth_graph[event_id])
|
|
|
|
|
for event_id in auth_graph
|
|
|
|
|
],
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.db_pool.runInteraction(
|
|
|
|
|
"insert",
|
|
|
|
|
insert_event,
|
|
|
|
|
)
|
2021-02-16 17:32:34 -05:00
|
|
|
|
)
|
2021-01-11 11:09:22 -05:00
|
|
|
|
|
2021-03-10 09:57:59 -05:00
|
|
|
|
return room_id
|
|
|
|
|
|
|
|
|
|
@parameterized.expand([(True,), (False,)])
|
|
|
|
|
def test_auth_chain_ids(self, use_chain_cover_index: bool):
|
|
|
|
|
room_id = self._setup_auth_chain(use_chain_cover_index)
|
|
|
|
|
|
|
|
|
|
# a and b have the same auth chain.
|
|
|
|
|
auth_chain_ids = self.get_success(self.store.get_auth_chain_ids(room_id, ["a"]))
|
|
|
|
|
self.assertCountEqual(auth_chain_ids, ["e", "f", "g", "h", "i", "j", "k"])
|
|
|
|
|
auth_chain_ids = self.get_success(self.store.get_auth_chain_ids(room_id, ["b"]))
|
|
|
|
|
self.assertCountEqual(auth_chain_ids, ["e", "f", "g", "h", "i", "j", "k"])
|
|
|
|
|
auth_chain_ids = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_ids(room_id, ["a", "b"])
|
|
|
|
|
)
|
|
|
|
|
self.assertCountEqual(auth_chain_ids, ["e", "f", "g", "h", "i", "j", "k"])
|
|
|
|
|
|
|
|
|
|
auth_chain_ids = self.get_success(self.store.get_auth_chain_ids(room_id, ["c"]))
|
|
|
|
|
self.assertCountEqual(auth_chain_ids, ["g", "h", "i", "j", "k"])
|
|
|
|
|
|
|
|
|
|
# d and e have the same auth chain.
|
|
|
|
|
auth_chain_ids = self.get_success(self.store.get_auth_chain_ids(room_id, ["d"]))
|
|
|
|
|
self.assertCountEqual(auth_chain_ids, ["f", "g", "h", "i", "j", "k"])
|
|
|
|
|
auth_chain_ids = self.get_success(self.store.get_auth_chain_ids(room_id, ["e"]))
|
|
|
|
|
self.assertCountEqual(auth_chain_ids, ["f", "g", "h", "i", "j", "k"])
|
|
|
|
|
|
|
|
|
|
auth_chain_ids = self.get_success(self.store.get_auth_chain_ids(room_id, ["f"]))
|
|
|
|
|
self.assertCountEqual(auth_chain_ids, ["g", "h", "i", "j", "k"])
|
|
|
|
|
|
|
|
|
|
auth_chain_ids = self.get_success(self.store.get_auth_chain_ids(room_id, ["g"]))
|
|
|
|
|
self.assertCountEqual(auth_chain_ids, ["h", "i", "j", "k"])
|
|
|
|
|
|
|
|
|
|
auth_chain_ids = self.get_success(self.store.get_auth_chain_ids(room_id, ["h"]))
|
2022-02-12 05:44:16 -05:00
|
|
|
|
self.assertEqual(auth_chain_ids, {"k"})
|
2021-03-10 09:57:59 -05:00
|
|
|
|
|
|
|
|
|
auth_chain_ids = self.get_success(self.store.get_auth_chain_ids(room_id, ["i"]))
|
2022-02-12 05:44:16 -05:00
|
|
|
|
self.assertEqual(auth_chain_ids, {"j"})
|
2021-03-10 09:57:59 -05:00
|
|
|
|
|
|
|
|
|
# j and k have no parents.
|
|
|
|
|
auth_chain_ids = self.get_success(self.store.get_auth_chain_ids(room_id, ["j"]))
|
2022-02-12 05:44:16 -05:00
|
|
|
|
self.assertEqual(auth_chain_ids, set())
|
2021-03-10 09:57:59 -05:00
|
|
|
|
auth_chain_ids = self.get_success(self.store.get_auth_chain_ids(room_id, ["k"]))
|
2022-02-12 05:44:16 -05:00
|
|
|
|
self.assertEqual(auth_chain_ids, set())
|
2021-03-10 09:57:59 -05:00
|
|
|
|
|
|
|
|
|
# More complex input sequences.
|
|
|
|
|
auth_chain_ids = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_ids(room_id, ["b", "c", "d"])
|
|
|
|
|
)
|
|
|
|
|
self.assertCountEqual(auth_chain_ids, ["e", "f", "g", "h", "i", "j", "k"])
|
|
|
|
|
|
|
|
|
|
auth_chain_ids = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_ids(room_id, ["h", "i"])
|
|
|
|
|
)
|
|
|
|
|
self.assertCountEqual(auth_chain_ids, ["k", "j"])
|
|
|
|
|
|
|
|
|
|
# e gets returned even though include_given is false, but it is in the
|
|
|
|
|
# auth chain of b.
|
|
|
|
|
auth_chain_ids = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_ids(room_id, ["b", "e"])
|
|
|
|
|
)
|
|
|
|
|
self.assertCountEqual(auth_chain_ids, ["e", "f", "g", "h", "i", "j", "k"])
|
|
|
|
|
|
|
|
|
|
# Test include_given.
|
|
|
|
|
auth_chain_ids = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_ids(room_id, ["i"], include_given=True)
|
|
|
|
|
)
|
|
|
|
|
self.assertCountEqual(auth_chain_ids, ["i", "j"])
|
|
|
|
|
|
|
|
|
|
@parameterized.expand([(True,), (False,)])
|
|
|
|
|
def test_auth_difference(self, use_chain_cover_index: bool):
|
|
|
|
|
room_id = self._setup_auth_chain(use_chain_cover_index)
|
|
|
|
|
|
2021-01-11 11:09:22 -05:00
|
|
|
|
# Now actually test that various combinations give the right result:
|
|
|
|
|
|
|
|
|
|
difference = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a"}, {"b"}])
|
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b"})
|
|
|
|
|
|
|
|
|
|
difference = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a"}, {"b"}, {"c"}])
|
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b", "c", "e", "f"})
|
|
|
|
|
|
|
|
|
|
difference = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a", "c"}, {"b"}])
|
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b", "c"})
|
|
|
|
|
|
|
|
|
|
difference = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a", "c"}, {"b", "c"}])
|
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b"})
|
|
|
|
|
|
|
|
|
|
difference = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a"}, {"b"}, {"d"}])
|
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b", "d", "e"})
|
|
|
|
|
|
|
|
|
|
difference = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a"}, {"b"}, {"c"}, {"d"}])
|
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b", "c", "d", "e", "f"})
|
|
|
|
|
|
|
|
|
|
difference = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a"}, {"b"}, {"e"}])
|
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b"})
|
|
|
|
|
|
|
|
|
|
difference = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a"}])
|
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, set())
|
|
|
|
|
|
|
|
|
|
def test_auth_difference_partial_cover(self):
|
|
|
|
|
"""Test that we correctly handle rooms where not all events have a chain
|
|
|
|
|
cover calculated. This can happen in some obscure edge cases, including
|
|
|
|
|
during the background update that calculates the chain cover for old
|
|
|
|
|
rooms.
|
|
|
|
|
"""
|
|
|
|
|
|
|
|
|
|
room_id = "@ROOM:local"
|
|
|
|
|
|
|
|
|
|
# The silly auth graph we use to test the auth difference algorithm,
|
|
|
|
|
# where the top are the most recent events.
|
|
|
|
|
#
|
|
|
|
|
# A B
|
|
|
|
|
# \ /
|
|
|
|
|
# D E
|
|
|
|
|
# \ |
|
|
|
|
|
# ` F C
|
|
|
|
|
# | /|
|
|
|
|
|
# G ´ |
|
|
|
|
|
# | \ |
|
|
|
|
|
# H I
|
|
|
|
|
# | |
|
|
|
|
|
# K J
|
|
|
|
|
|
|
|
|
|
auth_graph = {
|
|
|
|
|
"a": ["e"],
|
|
|
|
|
"b": ["e"],
|
|
|
|
|
"c": ["g", "i"],
|
|
|
|
|
"d": ["f"],
|
|
|
|
|
"e": ["f"],
|
|
|
|
|
"f": ["g"],
|
|
|
|
|
"g": ["h", "i"],
|
|
|
|
|
"h": ["k"],
|
|
|
|
|
"i": ["j"],
|
|
|
|
|
"k": [],
|
|
|
|
|
"j": [],
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
depth_map = {
|
|
|
|
|
"a": 7,
|
|
|
|
|
"b": 7,
|
|
|
|
|
"c": 4,
|
|
|
|
|
"d": 6,
|
|
|
|
|
"e": 6,
|
|
|
|
|
"f": 5,
|
|
|
|
|
"g": 3,
|
|
|
|
|
"h": 2,
|
|
|
|
|
"i": 2,
|
|
|
|
|
"k": 1,
|
|
|
|
|
"j": 1,
|
|
|
|
|
}
|
2020-03-18 12:46:41 -04:00
|
|
|
|
|
2021-01-11 11:09:22 -05:00
|
|
|
|
# We rudely fiddle with the appropriate tables directly, as that's much
|
|
|
|
|
# easier than constructing events properly.
|
2020-03-18 12:46:41 -04:00
|
|
|
|
|
2021-01-11 11:09:22 -05:00
|
|
|
|
def insert_event(txn):
|
|
|
|
|
# First insert the room and mark it as having a chain cover.
|
2020-08-05 16:38:57 -04:00
|
|
|
|
self.store.db_pool.simple_insert_txn(
|
2020-03-18 12:46:41 -04:00
|
|
|
|
txn,
|
2021-01-11 11:09:22 -05:00
|
|
|
|
"rooms",
|
|
|
|
|
{
|
2020-03-18 12:46:41 -04:00
|
|
|
|
"room_id": room_id,
|
2021-01-11 11:09:22 -05:00
|
|
|
|
"creator": "room_creator_user_id",
|
|
|
|
|
"is_public": True,
|
|
|
|
|
"room_version": "6",
|
|
|
|
|
"has_auth_chain_index": True,
|
2020-03-18 12:46:41 -04:00
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
|
2021-01-11 11:09:22 -05:00
|
|
|
|
stream_ordering = 0
|
|
|
|
|
|
|
|
|
|
for event_id in auth_graph:
|
|
|
|
|
stream_ordering += 1
|
|
|
|
|
depth = depth_map[event_id]
|
|
|
|
|
|
|
|
|
|
self.store.db_pool.simple_insert_txn(
|
|
|
|
|
txn,
|
|
|
|
|
table="events",
|
|
|
|
|
values={
|
|
|
|
|
"event_id": event_id,
|
|
|
|
|
"room_id": room_id,
|
|
|
|
|
"depth": depth,
|
|
|
|
|
"topological_ordering": depth,
|
|
|
|
|
"type": "m.test",
|
|
|
|
|
"processed": True,
|
|
|
|
|
"outlier": False,
|
|
|
|
|
"stream_ordering": stream_ordering,
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
# Insert all events apart from 'B'
|
|
|
|
|
self.hs.datastores.persist_events._persist_event_auth_chain_txn(
|
2020-03-18 12:46:41 -04:00
|
|
|
|
txn,
|
2021-01-11 11:09:22 -05:00
|
|
|
|
[
|
|
|
|
|
FakeEvent(event_id, room_id, auth_graph[event_id])
|
|
|
|
|
for event_id in auth_graph
|
|
|
|
|
if event_id != "b"
|
2020-03-18 12:46:41 -04:00
|
|
|
|
],
|
|
|
|
|
)
|
|
|
|
|
|
2021-01-11 11:09:22 -05:00
|
|
|
|
# Now we insert the event 'B' without a chain cover, by temporarily
|
|
|
|
|
# pretending the room doesn't have a chain cover.
|
|
|
|
|
|
|
|
|
|
self.store.db_pool.simple_update_txn(
|
|
|
|
|
txn,
|
|
|
|
|
table="rooms",
|
|
|
|
|
keyvalues={"room_id": room_id},
|
|
|
|
|
updatevalues={"has_auth_chain_index": False},
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
self.hs.datastores.persist_events._persist_event_auth_chain_txn(
|
|
|
|
|
txn,
|
|
|
|
|
[FakeEvent("b", room_id, auth_graph["b"])],
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
self.store.db_pool.simple_update_txn(
|
|
|
|
|
txn,
|
|
|
|
|
table="rooms",
|
|
|
|
|
keyvalues={"room_id": room_id},
|
|
|
|
|
updatevalues={"has_auth_chain_index": True},
|
2020-03-18 12:46:41 -04:00
|
|
|
|
)
|
|
|
|
|
|
2021-01-11 11:09:22 -05:00
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.db_pool.runInteraction(
|
|
|
|
|
"insert",
|
|
|
|
|
insert_event,
|
|
|
|
|
)
|
2021-02-16 17:32:34 -05:00
|
|
|
|
)
|
2021-01-11 11:09:22 -05:00
|
|
|
|
|
2020-03-18 12:46:41 -04:00
|
|
|
|
# Now actually test that various combinations give the right result:
|
|
|
|
|
|
|
|
|
|
difference = self.get_success(
|
2020-12-04 10:52:49 -05:00
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a"}, {"b"}])
|
2020-03-18 12:46:41 -04:00
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b"})
|
|
|
|
|
|
|
|
|
|
difference = self.get_success(
|
2020-12-04 10:52:49 -05:00
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a"}, {"b"}, {"c"}])
|
2020-03-18 12:46:41 -04:00
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b", "c", "e", "f"})
|
|
|
|
|
|
|
|
|
|
difference = self.get_success(
|
2020-12-04 10:52:49 -05:00
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a", "c"}, {"b"}])
|
2020-03-18 12:46:41 -04:00
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b", "c"})
|
|
|
|
|
|
2020-12-02 10:22:37 -05:00
|
|
|
|
difference = self.get_success(
|
2020-12-04 10:52:49 -05:00
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a", "c"}, {"b", "c"}])
|
2020-12-02 10:22:37 -05:00
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b"})
|
|
|
|
|
|
2020-03-18 12:46:41 -04:00
|
|
|
|
difference = self.get_success(
|
2020-12-04 10:52:49 -05:00
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a"}, {"b"}, {"d"}])
|
2020-03-18 12:46:41 -04:00
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b", "d", "e"})
|
|
|
|
|
|
|
|
|
|
difference = self.get_success(
|
2020-12-04 10:52:49 -05:00
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a"}, {"b"}, {"c"}, {"d"}])
|
2020-03-18 12:46:41 -04:00
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b", "c", "d", "e", "f"})
|
|
|
|
|
|
|
|
|
|
difference = self.get_success(
|
2020-12-04 10:52:49 -05:00
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a"}, {"b"}, {"e"}])
|
2020-03-18 12:46:41 -04:00
|
|
|
|
)
|
|
|
|
|
self.assertSetEqual(difference, {"a", "b"})
|
|
|
|
|
|
2020-12-04 10:52:49 -05:00
|
|
|
|
difference = self.get_success(
|
|
|
|
|
self.store.get_auth_chain_difference(room_id, [{"a"}])
|
|
|
|
|
)
|
2020-03-18 12:46:41 -04:00
|
|
|
|
self.assertSetEqual(difference, set())
|
2021-01-11 11:09:22 -05:00
|
|
|
|
|
2022-01-24 07:20:01 -05:00
|
|
|
|
@parameterized.expand(
|
|
|
|
|
[(room_version,) for room_version in KNOWN_ROOM_VERSIONS.values()]
|
|
|
|
|
)
|
|
|
|
|
def test_prune_inbound_federation_queue(self, room_version: RoomVersion):
|
|
|
|
|
"""Test that pruning of inbound federation queues work"""
|
2021-08-02 09:37:25 -04:00
|
|
|
|
|
|
|
|
|
room_id = "some_room_id"
|
|
|
|
|
|
2022-01-24 07:20:01 -05:00
|
|
|
|
def prev_event_format(prev_event_id: str) -> Union[Tuple[str, dict], str]:
|
|
|
|
|
"""Account for differences in prev_events format across room versions"""
|
2022-09-07 06:08:20 -04:00
|
|
|
|
if room_version.event_format == EventFormatVersions.ROOM_V1_V2:
|
2022-01-24 07:20:01 -05:00
|
|
|
|
return prev_event_id, {}
|
|
|
|
|
|
|
|
|
|
return prev_event_id
|
|
|
|
|
|
2021-08-02 09:37:25 -04:00
|
|
|
|
# Insert a bunch of events that all reference the previous one.
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.db_pool.simple_insert_many(
|
|
|
|
|
table="federation_inbound_events_staging",
|
2022-01-13 19:44:18 -05:00
|
|
|
|
keys=(
|
|
|
|
|
"origin",
|
|
|
|
|
"room_id",
|
|
|
|
|
"received_ts",
|
|
|
|
|
"event_id",
|
|
|
|
|
"event_json",
|
|
|
|
|
"internal_metadata",
|
|
|
|
|
),
|
2021-08-02 09:37:25 -04:00
|
|
|
|
values=[
|
2022-01-13 19:44:18 -05:00
|
|
|
|
(
|
|
|
|
|
"some_origin",
|
|
|
|
|
room_id,
|
|
|
|
|
0,
|
|
|
|
|
f"$fake_event_id_{i + 1}",
|
2022-01-24 07:20:01 -05:00
|
|
|
|
json_encoder.encode(
|
|
|
|
|
{"prev_events": [prev_event_format(f"$fake_event_id_{i}")]}
|
|
|
|
|
),
|
2022-01-13 19:44:18 -05:00
|
|
|
|
"{}",
|
|
|
|
|
)
|
2021-08-02 09:37:25 -04:00
|
|
|
|
for i in range(500)
|
|
|
|
|
],
|
|
|
|
|
desc="test_prune_inbound_federation_queue",
|
|
|
|
|
)
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
# Calling prune once should return True, i.e. a prune happen. The second
|
|
|
|
|
# time it shouldn't.
|
|
|
|
|
pruned = self.get_success(
|
2022-01-24 07:20:01 -05:00
|
|
|
|
self.store.prune_staged_events_in_room(room_id, room_version)
|
2021-08-02 09:37:25 -04:00
|
|
|
|
)
|
|
|
|
|
self.assertTrue(pruned)
|
|
|
|
|
|
|
|
|
|
pruned = self.get_success(
|
2022-01-24 07:20:01 -05:00
|
|
|
|
self.store.prune_staged_events_in_room(room_id, room_version)
|
2021-08-02 09:37:25 -04:00
|
|
|
|
)
|
|
|
|
|
self.assertFalse(pruned)
|
|
|
|
|
|
|
|
|
|
# Assert that we only have a single event left in the queue, and that it
|
|
|
|
|
# is the last one.
|
|
|
|
|
count = self.get_success(
|
|
|
|
|
self.store.db_pool.simple_select_one_onecol(
|
|
|
|
|
table="federation_inbound_events_staging",
|
|
|
|
|
keyvalues={"room_id": room_id},
|
2021-12-14 07:34:30 -05:00
|
|
|
|
retcol="COUNT(*)",
|
2021-08-02 09:37:25 -04:00
|
|
|
|
desc="test_prune_inbound_federation_queue",
|
|
|
|
|
)
|
|
|
|
|
)
|
|
|
|
|
self.assertEqual(count, 1)
|
|
|
|
|
|
2022-09-23 15:01:29 -04:00
|
|
|
|
next_staged_event_info = self.get_success(
|
2021-08-02 09:37:25 -04:00
|
|
|
|
self.store.get_next_staged_event_id_for_room(room_id)
|
|
|
|
|
)
|
2022-09-23 15:01:29 -04:00
|
|
|
|
assert next_staged_event_info
|
|
|
|
|
_, event_id = next_staged_event_info
|
2021-08-02 09:37:25 -04:00
|
|
|
|
self.assertEqual(event_id, "$fake_event_id_500")
|
|
|
|
|
|
2022-09-23 15:01:29 -04:00
|
|
|
|
def _setup_room_for_backfill_tests(self) -> _BackfillSetupInfo:
|
|
|
|
|
"""
|
|
|
|
|
Sets up a room with various events and backward extremities to test
|
|
|
|
|
backfill functions against.
|
|
|
|
|
|
|
|
|
|
Returns:
|
|
|
|
|
_BackfillSetupInfo including the `room_id` to test against and
|
|
|
|
|
`depth_map` of events in the room
|
|
|
|
|
"""
|
|
|
|
|
room_id = "!backfill-room-test:some-host"
|
|
|
|
|
|
|
|
|
|
# The silly graph we use to test grabbing backward extremities,
|
|
|
|
|
# where the top is the oldest events.
|
|
|
|
|
# 1 (oldest)
|
|
|
|
|
# |
|
|
|
|
|
# 2 ⹁
|
|
|
|
|
# | \
|
|
|
|
|
# | [b1, b2, b3]
|
|
|
|
|
# | |
|
|
|
|
|
# | A
|
|
|
|
|
# | /
|
|
|
|
|
# 3 {
|
|
|
|
|
# | \
|
|
|
|
|
# | [b4, b5, b6]
|
|
|
|
|
# | |
|
|
|
|
|
# | B
|
|
|
|
|
# | /
|
|
|
|
|
# 4 ´
|
|
|
|
|
# |
|
|
|
|
|
# 5 (newest)
|
|
|
|
|
|
|
|
|
|
event_graph: Dict[str, List[str]] = {
|
|
|
|
|
"1": [],
|
|
|
|
|
"2": ["1"],
|
|
|
|
|
"3": ["2", "A"],
|
|
|
|
|
"4": ["3", "B"],
|
|
|
|
|
"5": ["4"],
|
|
|
|
|
"A": ["b1", "b2", "b3"],
|
|
|
|
|
"b1": ["2"],
|
|
|
|
|
"b2": ["2"],
|
|
|
|
|
"b3": ["2"],
|
|
|
|
|
"B": ["b4", "b5", "b6"],
|
|
|
|
|
"b4": ["3"],
|
|
|
|
|
"b5": ["3"],
|
|
|
|
|
"b6": ["3"],
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
depth_map: Dict[str, int] = {
|
|
|
|
|
"1": 1,
|
|
|
|
|
"2": 2,
|
|
|
|
|
"b1": 3,
|
|
|
|
|
"b2": 3,
|
|
|
|
|
"b3": 3,
|
|
|
|
|
"A": 4,
|
|
|
|
|
"3": 5,
|
|
|
|
|
"b4": 6,
|
|
|
|
|
"b5": 6,
|
|
|
|
|
"b6": 6,
|
|
|
|
|
"B": 7,
|
|
|
|
|
"4": 8,
|
|
|
|
|
"5": 9,
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
# The events we have persisted on our server.
|
|
|
|
|
# The rest are events in the room but not backfilled tet.
|
|
|
|
|
our_server_events = {"5", "4", "B", "3", "A"}
|
|
|
|
|
|
|
|
|
|
complete_event_dict_map: Dict[str, JsonDict] = {}
|
|
|
|
|
stream_ordering = 0
|
|
|
|
|
for (event_id, prev_event_ids) in event_graph.items():
|
|
|
|
|
depth = depth_map[event_id]
|
|
|
|
|
|
|
|
|
|
complete_event_dict_map[event_id] = {
|
|
|
|
|
"event_id": event_id,
|
|
|
|
|
"type": "test_regular_type",
|
|
|
|
|
"room_id": room_id,
|
|
|
|
|
"sender": "@sender",
|
|
|
|
|
"prev_event_ids": prev_event_ids,
|
|
|
|
|
"auth_event_ids": [],
|
|
|
|
|
"origin_server_ts": stream_ordering,
|
|
|
|
|
"depth": depth,
|
|
|
|
|
"stream_ordering": stream_ordering,
|
|
|
|
|
"content": {"body": "event" + event_id},
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
stream_ordering += 1
|
|
|
|
|
|
|
|
|
|
def populate_db(txn: LoggingTransaction):
|
|
|
|
|
# Insert the room to satisfy the foreign key constraint of
|
|
|
|
|
# `event_failed_pull_attempts`
|
|
|
|
|
self.store.db_pool.simple_insert_txn(
|
|
|
|
|
txn,
|
|
|
|
|
"rooms",
|
|
|
|
|
{
|
|
|
|
|
"room_id": room_id,
|
|
|
|
|
"creator": "room_creator_user_id",
|
|
|
|
|
"is_public": True,
|
|
|
|
|
"room_version": "6",
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
# Insert our server events
|
|
|
|
|
for event_id in our_server_events:
|
|
|
|
|
event_dict = complete_event_dict_map[event_id]
|
|
|
|
|
|
|
|
|
|
self.store.db_pool.simple_insert_txn(
|
|
|
|
|
txn,
|
|
|
|
|
table="events",
|
|
|
|
|
values={
|
|
|
|
|
"event_id": event_dict.get("event_id"),
|
|
|
|
|
"type": event_dict.get("type"),
|
|
|
|
|
"room_id": event_dict.get("room_id"),
|
|
|
|
|
"depth": event_dict.get("depth"),
|
|
|
|
|
"topological_ordering": event_dict.get("depth"),
|
|
|
|
|
"stream_ordering": event_dict.get("stream_ordering"),
|
|
|
|
|
"processed": True,
|
|
|
|
|
"outlier": False,
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
# Insert the event edges
|
|
|
|
|
for event_id in our_server_events:
|
|
|
|
|
for prev_event_id in event_graph[event_id]:
|
|
|
|
|
self.store.db_pool.simple_insert_txn(
|
|
|
|
|
txn,
|
|
|
|
|
table="event_edges",
|
|
|
|
|
values={
|
|
|
|
|
"event_id": event_id,
|
|
|
|
|
"prev_event_id": prev_event_id,
|
|
|
|
|
"room_id": room_id,
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
# Insert the backward extremities
|
|
|
|
|
prev_events_of_our_events = {
|
|
|
|
|
prev_event_id
|
|
|
|
|
for our_server_event in our_server_events
|
|
|
|
|
for prev_event_id in complete_event_dict_map[our_server_event][
|
|
|
|
|
"prev_event_ids"
|
|
|
|
|
]
|
|
|
|
|
}
|
|
|
|
|
backward_extremities = prev_events_of_our_events - our_server_events
|
|
|
|
|
for backward_extremity in backward_extremities:
|
|
|
|
|
self.store.db_pool.simple_insert_txn(
|
|
|
|
|
txn,
|
|
|
|
|
table="event_backward_extremities",
|
|
|
|
|
values={
|
|
|
|
|
"event_id": backward_extremity,
|
|
|
|
|
"room_id": room_id,
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.db_pool.runInteraction(
|
|
|
|
|
"_setup_room_for_backfill_tests_populate_db",
|
|
|
|
|
populate_db,
|
|
|
|
|
)
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
return _BackfillSetupInfo(room_id=room_id, depth_map=depth_map)
|
|
|
|
|
|
|
|
|
|
def test_get_backfill_points_in_room(self):
|
|
|
|
|
"""
|
2022-09-28 16:26:16 -04:00
|
|
|
|
Test to make sure only backfill points that are older and come before
|
|
|
|
|
the `current_depth` are returned.
|
2022-09-23 15:01:29 -04:00
|
|
|
|
"""
|
|
|
|
|
setup_info = self._setup_room_for_backfill_tests()
|
|
|
|
|
room_id = setup_info.room_id
|
2022-09-28 16:26:16 -04:00
|
|
|
|
depth_map = setup_info.depth_map
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
2022-09-28 16:26:16 -04:00
|
|
|
|
# Try at "B"
|
2022-09-23 15:01:29 -04:00
|
|
|
|
backfill_points = self.get_success(
|
2022-09-28 16:26:16 -04:00
|
|
|
|
self.store.get_backfill_points_in_room(room_id, depth_map["B"], limit=100)
|
2022-09-23 15:01:29 -04:00
|
|
|
|
)
|
|
|
|
|
backfill_event_ids = [backfill_point[0] for backfill_point in backfill_points]
|
2022-09-30 06:54:53 -04:00
|
|
|
|
self.assertEqual(backfill_event_ids, ["b6", "b5", "b4", "2", "b3", "b2", "b1"])
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
2022-09-28 16:26:16 -04:00
|
|
|
|
# Try at "A"
|
|
|
|
|
backfill_points = self.get_success(
|
|
|
|
|
self.store.get_backfill_points_in_room(room_id, depth_map["A"], limit=100)
|
|
|
|
|
)
|
|
|
|
|
backfill_event_ids = [backfill_point[0] for backfill_point in backfill_points]
|
|
|
|
|
# Event "2" has a depth of 2 but is not included here because we only
|
|
|
|
|
# know the approximate depth of 5 from our event "3".
|
|
|
|
|
self.assertListEqual(backfill_event_ids, ["b3", "b2", "b1"])
|
|
|
|
|
|
2022-09-23 15:01:29 -04:00
|
|
|
|
def test_get_backfill_points_in_room_excludes_events_we_have_attempted(
|
|
|
|
|
self,
|
|
|
|
|
):
|
|
|
|
|
"""
|
|
|
|
|
Test to make sure that events we have attempted to backfill (and within
|
|
|
|
|
backoff timeout duration) do not show up as an event to backfill again.
|
|
|
|
|
"""
|
|
|
|
|
setup_info = self._setup_room_for_backfill_tests()
|
|
|
|
|
room_id = setup_info.room_id
|
2022-09-28 16:26:16 -04:00
|
|
|
|
depth_map = setup_info.depth_map
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
|
|
|
|
# Record some attempts to backfill these events which will make
|
|
|
|
|
# `get_backfill_points_in_room` exclude them because we
|
|
|
|
|
# haven't passed the backoff interval.
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(room_id, "b5", "fake cause")
|
|
|
|
|
)
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(room_id, "b4", "fake cause")
|
|
|
|
|
)
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(room_id, "b3", "fake cause")
|
|
|
|
|
)
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(room_id, "b2", "fake cause")
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
# No time has passed since we attempted to backfill ^
|
|
|
|
|
|
2022-09-28 16:26:16 -04:00
|
|
|
|
# Try at "B"
|
2022-09-23 15:01:29 -04:00
|
|
|
|
backfill_points = self.get_success(
|
2022-09-28 16:26:16 -04:00
|
|
|
|
self.store.get_backfill_points_in_room(room_id, depth_map["B"], limit=100)
|
2022-09-23 15:01:29 -04:00
|
|
|
|
)
|
|
|
|
|
backfill_event_ids = [backfill_point[0] for backfill_point in backfill_points]
|
|
|
|
|
# Only the backfill points that we didn't record earlier exist here.
|
2022-09-30 06:54:53 -04:00
|
|
|
|
self.assertEqual(backfill_event_ids, ["b6", "2", "b1"])
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
|
|
|
|
def test_get_backfill_points_in_room_attempted_event_retry_after_backoff_duration(
|
|
|
|
|
self,
|
|
|
|
|
):
|
|
|
|
|
"""
|
|
|
|
|
Test to make sure after we fake attempt to backfill event "b3" many times,
|
|
|
|
|
we can see retry and see the "b3" again after the backoff timeout duration
|
|
|
|
|
has exceeded.
|
|
|
|
|
"""
|
|
|
|
|
setup_info = self._setup_room_for_backfill_tests()
|
|
|
|
|
room_id = setup_info.room_id
|
2022-09-28 16:26:16 -04:00
|
|
|
|
depth_map = setup_info.depth_map
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
|
|
|
|
# Record some attempts to backfill these events which will make
|
|
|
|
|
# `get_backfill_points_in_room` exclude them because we
|
|
|
|
|
# haven't passed the backoff interval.
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(room_id, "b3", "fake cause")
|
|
|
|
|
)
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(room_id, "b1", "fake cause")
|
|
|
|
|
)
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(room_id, "b1", "fake cause")
|
|
|
|
|
)
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(room_id, "b1", "fake cause")
|
|
|
|
|
)
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(room_id, "b1", "fake cause")
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
# Now advance time by 2 hours and we should only be able to see "b3"
|
|
|
|
|
# because we have waited long enough for the single attempt (2^1 hours)
|
|
|
|
|
# but we still shouldn't see "b1" because we haven't waited long enough
|
|
|
|
|
# for this many attempts. We didn't do anything to "b2" so it should be
|
|
|
|
|
# visible regardless.
|
|
|
|
|
self.reactor.advance(datetime.timedelta(hours=2).total_seconds())
|
|
|
|
|
|
2022-09-28 16:26:16 -04:00
|
|
|
|
# Try at "A" and make sure that "b1" is not in the list because we've
|
2022-09-23 15:01:29 -04:00
|
|
|
|
# already attempted many times
|
|
|
|
|
backfill_points = self.get_success(
|
2022-09-28 16:26:16 -04:00
|
|
|
|
self.store.get_backfill_points_in_room(room_id, depth_map["A"], limit=100)
|
2022-09-23 15:01:29 -04:00
|
|
|
|
)
|
|
|
|
|
backfill_event_ids = [backfill_point[0] for backfill_point in backfill_points]
|
2022-09-30 06:54:53 -04:00
|
|
|
|
self.assertEqual(backfill_event_ids, ["b3", "b2"])
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
|
|
|
|
# Now advance time by 20 hours (above 2^4 because we made 4 attemps) and
|
|
|
|
|
# see if we can now backfill it
|
|
|
|
|
self.reactor.advance(datetime.timedelta(hours=20).total_seconds())
|
|
|
|
|
|
2022-09-28 16:26:16 -04:00
|
|
|
|
# Try at "A" again after we advanced enough time and we should see "b3" again
|
2022-09-23 15:01:29 -04:00
|
|
|
|
backfill_points = self.get_success(
|
2022-09-28 16:26:16 -04:00
|
|
|
|
self.store.get_backfill_points_in_room(room_id, depth_map["A"], limit=100)
|
2022-09-23 15:01:29 -04:00
|
|
|
|
)
|
|
|
|
|
backfill_event_ids = [backfill_point[0] for backfill_point in backfill_points]
|
2022-09-30 06:54:53 -04:00
|
|
|
|
self.assertEqual(backfill_event_ids, ["b3", "b2", "b1"])
|
|
|
|
|
|
|
|
|
|
def test_get_backfill_points_in_room_works_after_many_failed_pull_attempts_that_could_naively_overflow(
|
|
|
|
|
self,
|
|
|
|
|
) -> None:
|
|
|
|
|
"""
|
|
|
|
|
A test that reproduces #13929 (Postgres only).
|
|
|
|
|
|
|
|
|
|
Test to make sure we can still get backfill points after many failed pull
|
|
|
|
|
attempts that cause us to backoff to the limit. Even if the backoff formula
|
|
|
|
|
would tell us to wait for more seconds than can be expressed in a 32 bit
|
|
|
|
|
signed int.
|
|
|
|
|
"""
|
|
|
|
|
setup_info = self._setup_room_for_backfill_tests()
|
|
|
|
|
room_id = setup_info.room_id
|
|
|
|
|
depth_map = setup_info.depth_map
|
|
|
|
|
|
|
|
|
|
# Pretend that we have tried and failed 10 times to backfill event b1.
|
|
|
|
|
for _ in range(10):
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(room_id, "b1", "fake cause")
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
# If the backoff periods grow without limit:
|
|
|
|
|
# After the first failed attempt, we would have backed off for 1 << 1 = 2 hours.
|
|
|
|
|
# After the second failed attempt we would have backed off for 1 << 2 = 4 hours,
|
|
|
|
|
# so after the 10th failed attempt we should backoff for 1 << 10 == 1024 hours.
|
|
|
|
|
# Wait 1100 hours just so we have a nice round number.
|
|
|
|
|
self.reactor.advance(datetime.timedelta(hours=1100).total_seconds())
|
|
|
|
|
|
|
|
|
|
# 1024 hours in milliseconds is 1024 * 3600000, which exceeds the largest 32 bit
|
|
|
|
|
# signed integer. The bug we're reproducing is that this overflow causes an
|
|
|
|
|
# error in postgres preventing us from fetching a set of backwards extremities
|
|
|
|
|
# to retry fetching.
|
|
|
|
|
backfill_points = self.get_success(
|
|
|
|
|
self.store.get_backfill_points_in_room(room_id, depth_map["A"], limit=100)
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
# We should aim to fetch all backoff points: b1's latest backoff period has
|
|
|
|
|
# expired, and we haven't tried the rest.
|
|
|
|
|
backfill_event_ids = [backfill_point[0] for backfill_point in backfill_points]
|
|
|
|
|
self.assertEqual(backfill_event_ids, ["b3", "b2", "b1"])
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
|
|
|
|
def _setup_room_for_insertion_backfill_tests(self) -> _BackfillSetupInfo:
|
|
|
|
|
"""
|
|
|
|
|
Sets up a room with various insertion event backward extremities to test
|
|
|
|
|
backfill functions against.
|
|
|
|
|
|
|
|
|
|
Returns:
|
|
|
|
|
_BackfillSetupInfo including the `room_id` to test against and
|
|
|
|
|
`depth_map` of events in the room
|
|
|
|
|
"""
|
|
|
|
|
room_id = "!backfill-room-test:some-host"
|
|
|
|
|
|
|
|
|
|
depth_map: Dict[str, int] = {
|
|
|
|
|
"1": 1,
|
|
|
|
|
"2": 2,
|
|
|
|
|
"insertion_eventA": 3,
|
|
|
|
|
"3": 4,
|
|
|
|
|
"insertion_eventB": 5,
|
|
|
|
|
"4": 6,
|
|
|
|
|
"5": 7,
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
def populate_db(txn: LoggingTransaction):
|
|
|
|
|
# Insert the room to satisfy the foreign key constraint of
|
|
|
|
|
# `event_failed_pull_attempts`
|
|
|
|
|
self.store.db_pool.simple_insert_txn(
|
|
|
|
|
txn,
|
|
|
|
|
"rooms",
|
|
|
|
|
{
|
|
|
|
|
"room_id": room_id,
|
|
|
|
|
"creator": "room_creator_user_id",
|
|
|
|
|
"is_public": True,
|
|
|
|
|
"room_version": "6",
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
# Insert our server events
|
|
|
|
|
stream_ordering = 0
|
|
|
|
|
for event_id, depth in depth_map.items():
|
|
|
|
|
self.store.db_pool.simple_insert_txn(
|
|
|
|
|
txn,
|
|
|
|
|
table="events",
|
|
|
|
|
values={
|
|
|
|
|
"event_id": event_id,
|
|
|
|
|
"type": EventTypes.MSC2716_INSERTION
|
|
|
|
|
if event_id.startswith("insertion_event")
|
|
|
|
|
else "test_regular_type",
|
|
|
|
|
"room_id": room_id,
|
|
|
|
|
"depth": depth,
|
|
|
|
|
"topological_ordering": depth,
|
|
|
|
|
"stream_ordering": stream_ordering,
|
|
|
|
|
"processed": True,
|
|
|
|
|
"outlier": False,
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
if event_id.startswith("insertion_event"):
|
|
|
|
|
self.store.db_pool.simple_insert_txn(
|
|
|
|
|
txn,
|
|
|
|
|
table="insertion_event_extremities",
|
|
|
|
|
values={
|
|
|
|
|
"event_id": event_id,
|
|
|
|
|
"room_id": room_id,
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
stream_ordering += 1
|
|
|
|
|
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.db_pool.runInteraction(
|
|
|
|
|
"_setup_room_for_insertion_backfill_tests_populate_db",
|
|
|
|
|
populate_db,
|
|
|
|
|
)
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
return _BackfillSetupInfo(room_id=room_id, depth_map=depth_map)
|
|
|
|
|
|
|
|
|
|
def test_get_insertion_event_backward_extremities_in_room(self):
|
|
|
|
|
"""
|
2022-09-28 16:26:16 -04:00
|
|
|
|
Test to make sure only insertion event backward extremities that are
|
|
|
|
|
older and come before the `current_depth` are returned.
|
2022-09-23 15:01:29 -04:00
|
|
|
|
"""
|
|
|
|
|
setup_info = self._setup_room_for_insertion_backfill_tests()
|
|
|
|
|
room_id = setup_info.room_id
|
2022-09-28 16:26:16 -04:00
|
|
|
|
depth_map = setup_info.depth_map
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
2022-09-28 16:26:16 -04:00
|
|
|
|
# Try at "insertion_eventB"
|
2022-09-23 15:01:29 -04:00
|
|
|
|
backfill_points = self.get_success(
|
2022-09-28 16:26:16 -04:00
|
|
|
|
self.store.get_insertion_event_backward_extremities_in_room(
|
|
|
|
|
room_id, depth_map["insertion_eventB"], limit=100
|
|
|
|
|
)
|
2022-09-23 15:01:29 -04:00
|
|
|
|
)
|
|
|
|
|
backfill_event_ids = [backfill_point[0] for backfill_point in backfill_points]
|
2022-09-30 06:54:53 -04:00
|
|
|
|
self.assertEqual(backfill_event_ids, ["insertion_eventB", "insertion_eventA"])
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
2022-09-28 16:26:16 -04:00
|
|
|
|
# Try at "insertion_eventA"
|
|
|
|
|
backfill_points = self.get_success(
|
|
|
|
|
self.store.get_insertion_event_backward_extremities_in_room(
|
|
|
|
|
room_id, depth_map["insertion_eventA"], limit=100
|
|
|
|
|
)
|
|
|
|
|
)
|
|
|
|
|
backfill_event_ids = [backfill_point[0] for backfill_point in backfill_points]
|
|
|
|
|
# Event "2" has a depth of 2 but is not included here because we only
|
|
|
|
|
# know the approximate depth of 5 from our event "3".
|
|
|
|
|
self.assertListEqual(backfill_event_ids, ["insertion_eventA"])
|
|
|
|
|
|
2022-09-23 15:01:29 -04:00
|
|
|
|
def test_get_insertion_event_backward_extremities_in_room_excludes_events_we_have_attempted(
|
|
|
|
|
self,
|
|
|
|
|
):
|
|
|
|
|
"""
|
|
|
|
|
Test to make sure that insertion events we have attempted to backfill
|
|
|
|
|
(and within backoff timeout duration) do not show up as an event to
|
|
|
|
|
backfill again.
|
|
|
|
|
"""
|
|
|
|
|
setup_info = self._setup_room_for_insertion_backfill_tests()
|
|
|
|
|
room_id = setup_info.room_id
|
2022-09-28 16:26:16 -04:00
|
|
|
|
depth_map = setup_info.depth_map
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
|
|
|
|
# Record some attempts to backfill these events which will make
|
|
|
|
|
# `get_insertion_event_backward_extremities_in_room` exclude them
|
|
|
|
|
# because we haven't passed the backoff interval.
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(
|
|
|
|
|
room_id, "insertion_eventA", "fake cause"
|
|
|
|
|
)
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
# No time has passed since we attempted to backfill ^
|
|
|
|
|
|
2022-09-28 16:26:16 -04:00
|
|
|
|
# Try at "insertion_eventB"
|
2022-09-23 15:01:29 -04:00
|
|
|
|
backfill_points = self.get_success(
|
2022-09-28 16:26:16 -04:00
|
|
|
|
self.store.get_insertion_event_backward_extremities_in_room(
|
|
|
|
|
room_id, depth_map["insertion_eventB"], limit=100
|
|
|
|
|
)
|
2022-09-23 15:01:29 -04:00
|
|
|
|
)
|
|
|
|
|
backfill_event_ids = [backfill_point[0] for backfill_point in backfill_points]
|
|
|
|
|
# Only the backfill points that we didn't record earlier exist here.
|
2022-09-30 06:54:53 -04:00
|
|
|
|
self.assertEqual(backfill_event_ids, ["insertion_eventB"])
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
|
|
|
|
def test_get_insertion_event_backward_extremities_in_room_attempted_event_retry_after_backoff_duration(
|
|
|
|
|
self,
|
|
|
|
|
):
|
|
|
|
|
"""
|
|
|
|
|
Test to make sure after we fake attempt to backfill event
|
|
|
|
|
"insertion_eventA" many times, we can see retry and see the
|
|
|
|
|
"insertion_eventA" again after the backoff timeout duration has
|
|
|
|
|
exceeded.
|
|
|
|
|
"""
|
|
|
|
|
setup_info = self._setup_room_for_insertion_backfill_tests()
|
|
|
|
|
room_id = setup_info.room_id
|
2022-09-28 16:26:16 -04:00
|
|
|
|
depth_map = setup_info.depth_map
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
|
|
|
|
# Record some attempts to backfill these events which will make
|
|
|
|
|
# `get_backfill_points_in_room` exclude them because we
|
|
|
|
|
# haven't passed the backoff interval.
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(
|
|
|
|
|
room_id, "insertion_eventB", "fake cause"
|
|
|
|
|
)
|
|
|
|
|
)
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(
|
|
|
|
|
room_id, "insertion_eventA", "fake cause"
|
|
|
|
|
)
|
|
|
|
|
)
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(
|
|
|
|
|
room_id, "insertion_eventA", "fake cause"
|
|
|
|
|
)
|
|
|
|
|
)
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(
|
|
|
|
|
room_id, "insertion_eventA", "fake cause"
|
|
|
|
|
)
|
|
|
|
|
)
|
|
|
|
|
self.get_success(
|
|
|
|
|
self.store.record_event_failed_pull_attempt(
|
|
|
|
|
room_id, "insertion_eventA", "fake cause"
|
|
|
|
|
)
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
# Now advance time by 2 hours and we should only be able to see
|
|
|
|
|
# "insertion_eventB" because we have waited long enough for the single
|
|
|
|
|
# attempt (2^1 hours) but we still shouldn't see "insertion_eventA"
|
|
|
|
|
# because we haven't waited long enough for this many attempts.
|
|
|
|
|
self.reactor.advance(datetime.timedelta(hours=2).total_seconds())
|
|
|
|
|
|
2022-09-28 16:26:16 -04:00
|
|
|
|
# Try at "insertion_eventA" and make sure that "insertion_eventA" is not
|
|
|
|
|
# in the list because we've already attempted many times
|
2022-09-23 15:01:29 -04:00
|
|
|
|
backfill_points = self.get_success(
|
2022-09-28 16:26:16 -04:00
|
|
|
|
self.store.get_insertion_event_backward_extremities_in_room(
|
|
|
|
|
room_id, depth_map["insertion_eventA"], limit=100
|
|
|
|
|
)
|
2022-09-23 15:01:29 -04:00
|
|
|
|
)
|
|
|
|
|
backfill_event_ids = [backfill_point[0] for backfill_point in backfill_points]
|
2022-09-30 06:54:53 -04:00
|
|
|
|
self.assertEqual(backfill_event_ids, [])
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
|
|
|
|
# Now advance time by 20 hours (above 2^4 because we made 4 attemps) and
|
|
|
|
|
# see if we can now backfill it
|
|
|
|
|
self.reactor.advance(datetime.timedelta(hours=20).total_seconds())
|
|
|
|
|
|
|
|
|
|
# Try at "insertion_eventA" again after we advanced enough time and we
|
|
|
|
|
# should see "insertion_eventA" again
|
|
|
|
|
backfill_points = self.get_success(
|
2022-09-28 16:26:16 -04:00
|
|
|
|
self.store.get_insertion_event_backward_extremities_in_room(
|
|
|
|
|
room_id, depth_map["insertion_eventA"], limit=100
|
|
|
|
|
)
|
2022-09-23 15:01:29 -04:00
|
|
|
|
)
|
|
|
|
|
backfill_event_ids = [backfill_point[0] for backfill_point in backfill_points]
|
2022-09-30 06:54:53 -04:00
|
|
|
|
self.assertEqual(backfill_event_ids, ["insertion_eventA"])
|
2022-09-23 15:01:29 -04:00
|
|
|
|
|
2021-01-11 11:09:22 -05:00
|
|
|
|
|
|
|
|
|
@attr.s
|
|
|
|
|
class FakeEvent:
|
|
|
|
|
event_id = attr.ib()
|
|
|
|
|
room_id = attr.ib()
|
|
|
|
|
auth_events = attr.ib()
|
|
|
|
|
|
|
|
|
|
type = "foo"
|
|
|
|
|
state_key = "foo"
|
|
|
|
|
|
|
|
|
|
internal_metadata = _EventInternalMetadata({})
|
|
|
|
|
|
|
|
|
|
def auth_event_ids(self):
|
|
|
|
|
return self.auth_events
|
|
|
|
|
|
|
|
|
|
def is_state(self):
|
|
|
|
|
return True
|