Remove deleted state logging lines from sync API (they are pointless)
MODIFY
syncapi/storage/postgres/output_room_events_table.go
syncapi/storage/postgres/output_room_events_table.go
@@ -30,7 +30,6 @@ import (
"github.com/lib/pq"
"github.com/matrix-org/dendrite/internal/sqlutil"
"github.com/matrix-org/gomatrixserverlib"
- log "github.com/sirupsen/logrus"
)
const outputRoomEventsSchema = `
@@ -273,16 +272,6 @@ func (s *outputRoomEventsStatements) SelectStateInRange(
if err := rows.Scan(&eventID, &streamPos, &eventBytes, &excludeFromSync, &addIDs, &delIDs, &historyVisibility); err != nil {
return nil, nil, err
}
- // Sanity check for deleted state and whine if we see it. We don't need to do anything
- // since it'll just mark the event as not being needed.
- if len(addIDs) < len(delIDs) {
- log.WithFields(log.Fields{
- "since": r.From,
- "current": r.To,
- "adds": len(addIDs),
- "dels": len(delIDs),
- }).Warn("StateBetween: ignoring deleted state")
- }
// TODO: Handle redacted events
var ev gomatrixserverlib.HeaderedEvent

MODIFY
syncapi/storage/sqlite3/output_room_events_table.go
syncapi/storage/sqlite3/output_room_events_table.go
@@ -31,7 +31,6 @@ import (
"github.com/matrix-org/dendrite/internal/sqlutil"
"github.com/matrix-org/gomatrixserverlib"
- log "github.com/sirupsen/logrus"
)
const outputRoomEventsSchema = `
@@ -228,17 +227,6 @@ func (s *outputRoomEventsStatements) SelectStateInRange(
return nil, nil, err
}
- // Sanity check for deleted state and whine if we see it. We don't need to do anything
- // since it'll just mark the event as not being needed.
- if len(addIDs) < len(delIDs) {
- log.WithFields(log.Fields{
- "since": r.From,
- "current": r.To,
- "adds": len(addIDsJSON),
- "dels": len(delIDsJSON),
- }).Warn("StateBetween: ignoring deleted state")
- }
-
// TODO: Handle redacted events
var ev gomatrixserverlib.HeaderedEvent
if err := ev.UnmarshalJSONWithEventID(eventBytes, eventID); err != nil {

Remove deleted state logging lines from sync API (they are pointless)