mirror of
https://github.com/matrix-org/dendrite
synced 2024-12-25 21:13:45 +01:00
f64c8822bc
* Refactor federation sender, again
* Clean up better
* Missing operators
* Try to get overflowed events from database
* Fix queries
* Log less
* Comments
* nil PDUs/EDUs shouldn't happen but guard against them for safety
* Tweak logging
* Fix transaction coalescing
* Update comments
* Check nils more
* Remove channels as they add extra complexity and possibly will deadlock
* Don't hold lock while sending transaction
* Less spam about sleeping queues
* Comments
* Bug-fixing
* Don't try to rehydrate twice
* Don't queue in memory for blacklisted destinations
* Don't queue in memory for blacklisted destinations
* Fix a couple of bugs
* Check for duplicates when pulling things out of the database
* Durable transactions, some more refactoring
* Revert "Durable transactions, some more refactoring"
This reverts commit 5daf924eaa
.
* Fix deadlock
159 lines
4.7 KiB
Go
159 lines
4.7 KiB
Go
// Copyright 2020 The Matrix.org Foundation C.I.C.
|
|
//
|
|
// 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.
|
|
|
|
package shared
|
|
|
|
import (
|
|
"context"
|
|
"database/sql"
|
|
"encoding/json"
|
|
"errors"
|
|
"fmt"
|
|
|
|
"github.com/matrix-org/gomatrixserverlib"
|
|
)
|
|
|
|
// AssociatePDUWithDestination creates an association that the
|
|
// destination queues will use to determine which JSON blobs to send
|
|
// to which servers.
|
|
func (d *Database) AssociatePDUWithDestination(
|
|
ctx context.Context,
|
|
transactionID gomatrixserverlib.TransactionID,
|
|
serverName gomatrixserverlib.ServerName,
|
|
receipt *Receipt,
|
|
) error {
|
|
return d.Writer.Do(d.DB, nil, func(txn *sql.Tx) error {
|
|
if err := d.FederationSenderQueuePDUs.InsertQueuePDU(
|
|
ctx, // context
|
|
txn, // SQL transaction
|
|
transactionID, // transaction ID
|
|
serverName, // destination server name
|
|
receipt.nid, // NID from the federationsender_queue_json table
|
|
); err != nil {
|
|
return fmt.Errorf("InsertQueuePDU: %w", err)
|
|
}
|
|
return nil
|
|
})
|
|
}
|
|
|
|
// GetNextTransactionPDUs retrieves events from the database for
|
|
// the next pending transaction, up to the limit specified.
|
|
func (d *Database) GetPendingPDUs(
|
|
ctx context.Context,
|
|
serverName gomatrixserverlib.ServerName,
|
|
limit int,
|
|
) (
|
|
events map[*Receipt]*gomatrixserverlib.HeaderedEvent,
|
|
err error,
|
|
) {
|
|
// Strictly speaking this doesn't need to be using the writer
|
|
// since we are only performing selects, but since we don't have
|
|
// a guarantee of transactional isolation, it's actually useful
|
|
// to know in SQLite mode that nothing else is trying to modify
|
|
// the database.
|
|
events = make(map[*Receipt]*gomatrixserverlib.HeaderedEvent)
|
|
err = d.Writer.Do(d.DB, nil, func(txn *sql.Tx) error {
|
|
nids, err := d.FederationSenderQueuePDUs.SelectQueuePDUs(ctx, txn, serverName, limit)
|
|
if err != nil {
|
|
return fmt.Errorf("SelectQueuePDUs: %w", err)
|
|
}
|
|
|
|
retrieve := make([]int64, 0, len(nids))
|
|
for _, nid := range nids {
|
|
if event, ok := d.Cache.GetFederationSenderQueuedPDU(nid); ok {
|
|
events[&Receipt{nid}] = event
|
|
} else {
|
|
retrieve = append(retrieve, nid)
|
|
}
|
|
}
|
|
|
|
blobs, err := d.FederationSenderQueueJSON.SelectQueueJSON(ctx, txn, retrieve)
|
|
if err != nil {
|
|
return fmt.Errorf("SelectQueueJSON: %w", err)
|
|
}
|
|
|
|
for nid, blob := range blobs {
|
|
var event gomatrixserverlib.HeaderedEvent
|
|
if err := json.Unmarshal(blob, &event); err != nil {
|
|
return fmt.Errorf("json.Unmarshal: %w", err)
|
|
}
|
|
events[&Receipt{nid}] = &event
|
|
d.Cache.StoreFederationSenderQueuedPDU(nid, &event)
|
|
}
|
|
|
|
return nil
|
|
})
|
|
return
|
|
}
|
|
|
|
// CleanTransactionPDUs cleans up all associated events for a
|
|
// given transaction. This is done when the transaction was sent
|
|
// successfully.
|
|
func (d *Database) CleanPDUs(
|
|
ctx context.Context,
|
|
serverName gomatrixserverlib.ServerName,
|
|
receipts []*Receipt,
|
|
) error {
|
|
if len(receipts) == 0 {
|
|
return errors.New("expected receipt")
|
|
}
|
|
|
|
nids := make([]int64, len(receipts))
|
|
for i := range receipts {
|
|
nids[i] = receipts[i].nid
|
|
}
|
|
|
|
return d.Writer.Do(d.DB, nil, func(txn *sql.Tx) error {
|
|
if err := d.FederationSenderQueuePDUs.DeleteQueuePDUs(ctx, txn, serverName, nids); err != nil {
|
|
return err
|
|
}
|
|
|
|
var deleteNIDs []int64
|
|
for _, nid := range nids {
|
|
count, err := d.FederationSenderQueuePDUs.SelectQueuePDUReferenceJSONCount(ctx, txn, nid)
|
|
if err != nil {
|
|
return fmt.Errorf("SelectQueuePDUReferenceJSONCount: %w", err)
|
|
}
|
|
if count == 0 {
|
|
deleteNIDs = append(deleteNIDs, nid)
|
|
d.Cache.EvictFederationSenderQueuedPDU(nid)
|
|
}
|
|
}
|
|
|
|
if len(deleteNIDs) > 0 {
|
|
if err := d.FederationSenderQueueJSON.DeleteQueueJSON(ctx, txn, deleteNIDs); err != nil {
|
|
return fmt.Errorf("DeleteQueueJSON: %w", err)
|
|
}
|
|
}
|
|
|
|
return nil
|
|
})
|
|
}
|
|
|
|
// GetPendingPDUCount returns the number of PDUs waiting to be
|
|
// sent for a given servername.
|
|
func (d *Database) GetPendingPDUCount(
|
|
ctx context.Context,
|
|
serverName gomatrixserverlib.ServerName,
|
|
) (int64, error) {
|
|
return d.FederationSenderQueuePDUs.SelectQueuePDUCount(ctx, nil, serverName)
|
|
}
|
|
|
|
// GetPendingServerNames returns the server names that have PDUs
|
|
// waiting to be sent.
|
|
func (d *Database) GetPendingPDUServerNames(
|
|
ctx context.Context,
|
|
) ([]gomatrixserverlib.ServerName, error) {
|
|
return d.FederationSenderQueuePDUs.SelectQueuePDUServerNames(ctx, nil)
|
|
}
|