Commit 9eef2cce authored by Andrei Mihu's avatar Andrei Mihu
Browse files

Fix issue delivering persistent SendAll notifications to large numbers of users.

parent 71a620d4
Loading
Loading
Loading
Loading
+2 −1
Original line number Diff line number Diff line
@@ -6,7 +6,8 @@ The format is based on [keep a changelog](http://keepachangelog.com) and this pr
## [Unreleased]
### Fixed
- Correct cursor usage in group listings using only open/closed group state filter.
- Remap original Google IDs to "next generation player IDs" 
- Remap original Google IDs to "next generation player IDs".
- Fix issue delivering persistent SendAll notifications to large numbers of users.

## [3.16.0] - 2023-04-18
### Added
+1 −1
Original line number Diff line number Diff line
@@ -122,7 +122,7 @@ func NotificationSendAll(ctx context.Context, logger *zap.Logger, db *sql.DB, tr
			params := make([]interface{}, 0, 1)
			query := "SELECT id FROM users"
			if userIDStr != "" {
				query += " AND id > $1"
				query += " WHERE id > $1"
				params = append(params, userIDStr)
			}
			query += fmt.Sprintf(" ORDER BY id ASC LIMIT %d", limit)