Undo GapWriter algorithm changes

This commit is contained in:
Danila Fedorin 2021-09-20 18:41:01 -07:00
parent 9f6c48cf0c
commit 50c8b995c3

View file

@ -47,30 +47,28 @@ export class GapWriter {
} }
nonOverlappingEvents.push(...remainingEvents.slice(0, duplicateEventIndex)); nonOverlappingEvents.push(...remainingEvents.slice(0, duplicateEventIndex));
if (!expectedOverlappingEventId || duplicateEventId === expectedOverlappingEventId) { if (!expectedOverlappingEventId || duplicateEventId === expectedOverlappingEventId) {
// Only link fragment if this is the first overlapping fragment we discover. // TODO: check here that the neighbourEvent is at the correct edge of it's fragment
// TODO is this sufficient? Might we get "out of order" fragments from events? // get neighbour fragment to link it up later on
if (!neighbourFragmentEntry) { const neighbourEvent = await txn.timelineEvents.getByEventId(this._roomId, duplicateEventId);
// TODO: check here that the neighbourEvent is at the correct edge of it's fragment if (neighbourEvent.fragmentId === fragmentEntry.fragmentId) {
// get neighbour fragment to link it up later on log.log("hit #160, prevent fragment linking to itself", log.level.Warn);
const neighbourEvent = await txn.timelineEvents.getByEventId(this._roomId, duplicateEventId); } else {
const neighbourFragment = await txn.timelineFragments.get(this._roomId, neighbourEvent.fragmentId); const neighbourFragment = await txn.timelineFragments.get(this._roomId, neighbourEvent.fragmentId);
neighbourFragmentEntry = fragmentEntry.createNeighbourEntry(neighbourFragment); neighbourFragmentEntry = fragmentEntry.createNeighbourEntry(neighbourFragment);
} }
// trim overlapping events
remainingEvents = null;
} else {
// we've hit https://github.com/matrix-org/synapse/issues/7164,
// e.g. the event id we found is already in our store but it is not
// the adjacent fragment id. Ignore the event, but keep processing the ones after.
remainingEvents = remainingEvents.slice(duplicateEventIndex + 1);
} }
// If more events remain, or if this wasn't the expected overlapping event,
// we've hit https://github.com/matrix-org/synapse/issues/7164,
// e.g. the event id we found is already in our store but it is not
// the adjacent fragment id. Ignore the event, but keep processing the ones after.
remainingEvents = remainingEvents.slice(duplicateEventIndex + 1);
} else { } else {
nonOverlappingEvents.push(...remainingEvents); nonOverlappingEvents.push(...remainingEvents);
remainingEvents = null; remainingEvents = null;
} }
} }
if (neighbourFragmentEntry?.fragmentId === fragmentEntry.fragmentId) {
log.log("hit #160, prevent fragment linking to itself", log.level.Warn);
neighbourFragmentEntry = null;
}
return {nonOverlappingEvents, neighbourFragmentEntry}; return {nonOverlappingEvents, neighbourFragmentEntry};
} }