server/{db,auth}: add a dedicated table for tracking reputation effects #3259

buck54321 · 2025-04-30T13:57:18Z

Loading reputation history from the orders and matches tables used some gnarly queries that were getting incredibly slow as the table sizes grew. This was causing extremely long connect times that often timed out when the server connection flapped and everybody tried to connect at once. This PR adds a dedicated table for reputation effects that stays pruned, and a rolling upgrade process.

JoeGruffins

Only partial review. I'm testing on simnet and I can't seem to bring reputation down. I make some matches and then don't continue them but score never decreases. Strikes should be going negative here I think.

server log

2025-05-02 19:35:19.991 [INF] AUTH: Match failure for user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f: '\x04' (badness -11), strikes 0, bond tier 11 => trading tier 11
2025-05-02 19:35:19.991 [INF] SWAP: Sending a 'revoke_match' notification to each client for match abdb2b5052c4bb0a2734a1c2bbe0b8e7e8e7c243e1e63444bf2fd85f12b1be8d
2025-05-02 19:35:19.991 [DBG] AUTH: Send requested for disconnected user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:19.991 [DBG] SWAP: Failed to send 'revoke_match' notification to user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f, match abdb2b5052c4bb0a2734a1c2bbe0b8e7e8e7c243e1e63444bf2fd85f12b1be8d: user not connected: 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:19.991 [DBG] SWAP: failMatch: swap ab99a57c3e0c5b1cdd3867d8bc7c56fb413359c4b8131d2192af06867eccfb7f failing at MakerSwapCast (4), user fault = true
2025-05-02 19:35:19.993 [DBG] AUTH: Recorded order 94586971590ba92767d366ac55c436b36a72a5ecb7938f01ec0e97abf97f33ef that has finished processing: user=ba6e444e88845e954d4f7b28bfee4f9f6e254612ebbe93f193cb26b241c5485e, time=1746182119992, target=<nil>
2025-05-02 19:35:19.994 [INF] AUTH: Match failure for user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f: '\x04' (badness -11), strikes 0, bond tier 11 => trading tier 11
2025-05-02 19:35:19.994 [INF] SWAP: Sending a 'revoke_match' notification to each client for match ab99a57c3e0c5b1cdd3867d8bc7c56fb413359c4b8131d2192af06867eccfb7f
2025-05-02 19:35:19.994 [DBG] AUTH: Send requested for disconnected user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:19.994 [DBG] SWAP: Failed to send 'revoke_match' notification to user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f, match ab99a57c3e0c5b1cdd3867d8bc7c56fb413359c4b8131d2192af06867eccfb7f: user not connected: 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:19.994 [DBG] SWAP: failMatch: swap f70107549c60c2cfef20f5f6e1ff75ceb41a8d6c17eb103f023af6401c827322 failing at MakerSwapCast (4), user fault = true
2025-05-02 19:35:19.995 [DBG] AUTH: Recorded order 59c482fe4704ad0f04e1cd510e90a883016b80cd14d2430cb59837a1cdabf51d that has finished processing: user=ba6e444e88845e954d4f7b28bfee4f9f6e254612ebbe93f193cb26b241c5485e, time=1746182119994, target=<nil>
2025-05-02 19:35:19.996 [INF] AUTH: Match failure for user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f: '\x04' (badness -11), strikes 0, bond tier 11 => trading tier 11
2025-05-02 19:35:19.996 [INF] SWAP: Sending a 'revoke_match' notification to each client for match f70107549c60c2cfef20f5f6e1ff75ceb41a8d6c17eb103f023af6401c827322
2025-05-02 19:35:19.996 [DBG] AUTH: Send requested for disconnected user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:19.996 [DBG] SWAP: Failed to send 'revoke_match' notification to user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f, match f70107549c60c2cfef20f5f6e1ff75ceb41a8d6c17eb103f023af6401c827322: user not connected: 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:19.996 [DBG] SWAP: failMatch: swap 005a3c9ca592594572302879226567030e25254d79b331cfef773a31038b4665 failing at MakerSwapCast (4), user fault = true
2025-05-02 19:35:19.997 [DBG] AUTH: Recorded order 8c6fa51df1502a45f3f65e909eaf97b0bd2c4e1b3c9926de4d59b08d7d3736e8 that has finished processing: user=ba6e444e88845e954d4f7b28bfee4f9f6e254612ebbe93f193cb26b241c5485e, time=1746182119996, target=<nil>
2025-05-02 19:35:19.998 [INF] AUTH: Match failure for user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f: '\x04' (badness -11), strikes 0, bond tier 11 => trading tier 11
2025-05-02 19:35:19.998 [INF] SWAP: Sending a 'revoke_match' notification to each client for match 005a3c9ca592594572302879226567030e25254d79b331cfef773a31038b4665
2025-05-02 19:35:19.998 [DBG] AUTH: Send requested for disconnected user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:19.998 [DBG] SWAP: Failed to send 'revoke_match' notification to user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f, match 005a3c9ca592594572302879226567030e25254d79b331cfef773a31038b4665: user not connected: 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:19.998 [DBG] SWAP: failMatch: swap d0d8642563039321f518704cf8e6635d41ab4e4a2bcc99c0006aad136af34a47 failing at MakerSwapCast (4), user fault = true
2025-05-02 19:35:19.999 [DBG] AUTH: Recorded order 1a04639e3710b0d2c67b87bc46e5228d35f9783ed9ebf151e8d865f74bc19df0 that has finished processing: user=ba6e444e88845e954d4f7b28bfee4f9f6e254612ebbe93f193cb26b241c5485e, time=1746182119998, target=<nil>
2025-05-02 19:35:20.000 [INF] AUTH: Match failure for user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f: '\x04' (badness -11), strikes 0, bond tier 11 => trading tier 11
2025-05-02 19:35:20.000 [INF] SWAP: Sending a 'revoke_match' notification to each client for match d0d8642563039321f518704cf8e6635d41ab4e4a2bcc99c0006aad136af34a47
2025-05-02 19:35:20.000 [DBG] AUTH: Send requested for disconnected user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:20.000 [DBG] SWAP: Failed to send 'revoke_match' notification to user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f, match d0d8642563039321f518704cf8e6635d41ab4e4a2bcc99c0006aad136af34a47: user not connected: 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:20.000 [DBG] SWAP: failMatch: swap f7052880bec0a3c1e0b0de4aaa9e0215bf4184fd231dfd53f79781216078acc6 failing at MakerSwapCast (4), user fault = true
2025-05-02 19:35:20.001 [DBG] AUTH: Recorded order b449193d947f059fc26ffa11d95527767385becbc9e3f549cce4e4ccbb9dd4c5 that has finished processing: user=ba6e444e88845e954d4f7b28bfee4f9f6e254612ebbe93f193cb26b241c5485e, time=1746182120001, target=<nil>
2025-05-02 19:35:20.002 [INF] AUTH: Match failure for user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f: '\x04' (badness -11), strikes 0, bond tier 11 => trading tier 11
2025-05-02 19:35:20.002 [INF] SWAP: Sending a 'revoke_match' notification to each client for match f7052880bec0a3c1e0b0de4aaa9e0215bf4184fd231dfd53f79781216078acc6
2025-05-02 19:35:20.002 [DBG] AUTH: Send requested for disconnected user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:20.002 [DBG] SWAP: Failed to send 'revoke_match' notification to user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f, match f7052880bec0a3c1e0b0de4aaa9e0215bf4184fd231dfd53f79781216078acc6: user not connected: 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:20.002 [DBG] SWAP: failMatch: swap 8a9be5a5ee13aff5762171b699ca6747a42934373fde99fc006ad5c2daa685f9 failing at MakerSwapCast (4), user fault = true
2025-05-02 19:35:20.003 [DBG] AUTH: Recorded order 4706a52c6d316a2abed1f79d7fba5dacfdad90dc64926bdefa282eed6cd1dea0 that has finished processing: user=ba6e444e88845e954d4f7b28bfee4f9f6e254612ebbe93f193cb26b241c5485e, time=1746182120003, target=<nil>
2025-05-02 19:35:20.004 [INF] AUTH: Match failure for user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f: '\x04' (badness -11), strikes 0, bond tier 11 => trading tier 11
2025-05-02 19:35:20.004 [INF] SWAP: Sending a 'revoke_match' notification to each client for match 8a9be5a5ee13aff5762171b699ca6747a42934373fde99fc006ad5c2daa685f9
2025-05-02 19:35:20.004 [DBG] AUTH: Send requested for disconnected user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:20.004 [DBG] SWAP: Failed to send 'revoke_match' notification to user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f, match 8a9be5a5ee13aff5762171b699ca6747a42934373fde99fc006ad5c2daa685f9: user not connected: 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:20.004 [DBG] SWAP: failMatch: swap 2d2f3b9c5ad4022cdad2b42bd54c69372ef3aee13c0ce3f03208099828ac3bc0 failing at MakerSwapCast (4), user fault = true
2025-05-02 19:35:20.005 [DBG] AUTH: Recorded order c53b1e1fe298a5eeb1e3d133dda937b8ec1260ac97680b07ef301958f3f78b6f that has finished processing: user=ba6e444e88845e954d4f7b28bfee4f9f6e254612ebbe93f193cb26b241c5485e, time=1746182120005, target=<nil>
2025-05-02 19:35:20.006 [INF] AUTH: Match failure for user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f: '\x04' (badness -11), strikes 0, bond tier 11 => trading tier 11
2025-05-02 19:35:20.006 [INF] SWAP: Sending a 'revoke_match' notification to each client for match 2d2f3b9c5ad4022cdad2b42bd54c69372ef3aee13c0ce3f03208099828ac3bc0
2025-05-02 19:35:20.006 [DBG] AUTH: Send requested for disconnected user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:20.006 [DBG] SWAP: Failed to send 'revoke_match' notification to user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f, match 2d2f3b9c5ad4022cdad2b42bd54c69372ef3aee13c0ce3f03208099828ac3bc0: user not connected: 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:20.006 [DBG] SWAP: failMatch: swap e84eba96df6f0c068a9d75cf81cf61741294e55a0f87ac3679f3bab8a33c23fa failing at MakerSwapCast (4), user fault = true
2025-05-02 19:35:20.007 [DBG] AUTH: Recorded order f3e61417341a426447e534f40aabfc16b4b9cc33888a4133aba89e40db828208 that has finished processing: user=ba6e444e88845e954d4f7b28bfee4f9f6e254612ebbe93f193cb26b241c5485e, time=1746182120006, target=<nil>
2025-05-02 19:35:20.009 [INF] AUTH: Match failure for user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f: '\x04' (badness -11), strikes 0, bond tier 11 => trading tier 11
2025-05-02 19:35:20.009 [INF] SWAP: Sending a 'revoke_match' notification to each client for match e84eba96df6f0c068a9d75cf81cf61741294e55a0f87ac3679f3bab8a33c23fa
2025-05-02 19:35:20.009 [DBG] AUTH: Send requested for disconnected user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:20.009 [DBG] SWAP: Failed to send 'revoke_match' notification to user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f, match e84eba96df6f0c068a9d75cf81cf61741294e55a0f87ac3679f3bab8a33c23fa: user not connected: 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f
2025-05-02 19:35:20.009 [DBG] SWAP: failMatch: swap d58f4308f60554d34b2a7e7d1a1da81552991b70246e70512c342790871f682f failing at MakerSwapCast (4), user fault = true
2025-05-02 19:35:20.044 [DBG] AUTH: Recorded order bb24bb73dd38b7a68c6d7db70bedaa61ad1bc1a5a95ca17c9aac53c8cad1d2c6 that has finished processing: user=ba6e444e88845e954d4f7b28bfee4f9f6e254612ebbe93f193cb26b241c5485e, time=1746182120017, target=<nil>
2025-05-02 19:35:20.046 [INF] AUTH: Match failure for user 58c371400e97f6209a5f4e47113a992f9ed08dcff9aa7d9d80b50af13d56b20f: '\x04' (badness -11), strikes 0, bond tier 11 => trading tier 11

JoeGruffins · 2025-05-02T07:36:02Z

dex/order/order.go

+		if t.Quantity%lotSize != 0 || t.Remaining()%lotSize != 0 {
+			return fmt.Errorf("market sell order fails lot size requirement %d %% %d = %d", t.Quantity, lotSize, t.Quantity%lotSize)
+		}


This is just moved, but error leaves out possible t.Remaining()%lotSize != 0 being false.

server/auth/auth.go

martonp

Haven't tested.. will test after the issue @JoeGruffins found is resolved.

server/db/driver/pg/pg.go

martonp · 2025-05-02T15:01:44Z

server/db/driver/pg/reputation.go

+			})
+		}
+	}


Suggested change

})

}

}

})

}

if len(pings) >= pingSz && len(matches) >= matchSz && len(orders) >= orderSz {

break

}

}

martonp · 2025-05-02T15:05:44Z

server/db/driver/pg/reputation.go

+		case db.OutcomeClassPreimage:
+			pimgs = append(pimgs, &db.PreimageOutcome{
+				DBID:    dbID,
+				OrderID: link,
+				Miss:    outcome == db.OutcomePreimageMiss,
+			})
+		case db.OutcomeClassMatch:
+			var mid order.MatchID
+			copy(mid[:], link[:])
+			matches = append(matches, &db.MatchResult{
+				DBID:         dbID,
+				MatchID:      mid,
+				MatchOutcome: outcome,
+			})
+		case db.OutcomeClassOrder:
+			orders = append(orders, &db.OrderOutcome{
+				DBID:     dbID,
+				OrderID:  link,
+				Canceled: outcome == db.OutcomeOrderCanceled,
+			})
+		}


No need to append these if we already have enough for the class, and then you don't need to shorten the slices below.

martonp · 2025-05-02T15:09:31Z

server/db/driver/pg/internal/reputation.go

+	CreatePointsTable = `CREATE TABLE IF NOT EXISTS %s (
+		id BIGSERIAL PRIMARY KEY,
+		account BYTEA,
+		link BYTEA,             -- Order ID or Match ID


Suggested change

link BYTEA, -- Order ID or Match ID

link BYTEA UNIQUE, -- Order ID or Match ID

martonp · 2025-05-02T15:14:08Z

server/db/driver/pg/reputation.go

+		if o.Miss {
+			outcome = db.OutcomePreimageMiss
+		}
+		if err := stmt.QueryRowContext(ctx, user, o.OrderID, db.OutcomeClassPreimage, outcome).Scan(&o.DBID); err != nil {


Suggested change

if err := stmt.QueryRowContext(ctx, user, o.OrderID, db.OutcomeClassPreimage, outcome).Scan(&o.DBID); err != nil {

if err := a.queries.insertPoints.QueryRowContext(ctx, user, o.OrderID, db.OutcomeClassPreimage, outcome).Scan(&o.DBID); err != nil {

and also below

martonp · 2025-05-02T15:16:45Z

server/db/interface.go

+	OutcomeOrderComplete
+	OutcomeOrderCanceled


May be confusing.. could do OutcomeCancelledEarly and OutcomeNotCancelledEarly

martonp · 2025-05-02T15:38:20Z

server/auth/auth.go

@@ -1711,7 +1765,7 @@ func (auth *AuthManager) handleConnect(conn comms.Link, msg *msgjson.Message) *m
 	return nil
 }

-func (auth *AuthManager) loadRecentFinishedOrders(aid account.AccountID, N int) (*latestOrders, error) {
+func (auth *AuthManager) loadRecentFinishedOrders(aid account.AccountID, N int) (*latestOutcomes[*db.OrderOutcome], error) {


loadRecentFinishedOrders is unused now.

JoeGruffins · 2025-05-08T06:55:58Z

Reputation is going down as expected now.

JoeGruffins · 2025-05-08T07:50:44Z

On simnet, seeing an error on server whenever connecting with a new user. Seems to be fine after that though.

2025-05-08 16:47:04.261 [ERR] AUTH: failed to load user score: error getting current user reputation version: sql: no rows in result set
2025-05-08 16:47:04.262 [DBG] AUTH: Activating new bond 07ada0694c7da80dfa39d282fd5071be86f7edf489780fcf7872f49c7ef5086c:0 (dcr) committing 5000000000 for user a8db099a3436f30019446c56d2e0ead09158b15df1d9d6e0ec39adde12d2c376
2025-05-08 16:47:04.262 [INF] AUTH: Creating new user account a8db099a3436f30019446c56d2e0ead09158b15df1d9d6e0ec39adde12d2c376 from 127.0.0.1, posted first bond in 07ada0694c7da80dfa39d282fd5071be86f7edf489780fcf7872f49c7ef5086c:0 (dcr)
2025-05-08 16:47:04.262 [INF] AUTH: Bond accepted: acct a8db099a3436f30019446c56d2e0ead09158b15df1d9d6e0ec39adde12d2c376 from 127.0.0.1 locked 5000000000 in 07ada0694c7da80dfa39d282fd5071be86f7edf489780fcf7872f49c7ef5086c:0. Bond total 100, tier 100

JoeGruffins · 2025-05-08T08:42:25Z

server/db/driver/pg/reputation.go

+		if o.Miss {
+			outcome = db.OutcomePreimageMiss
+		}
+		if err := stmt.QueryRowContext(ctx, user, o.OrderID, db.OutcomeClassPreimage, outcome).Scan(&o.DBID); err != nil {


I don't know if we are using this pattern elsewhere, but if you return inside the if, this err is in a different scope than the defer err so the tx will not be rolled back.

https://go.dev/play/p/knpoiH3mcmw

Suggested change

if err := stmt.QueryRowContext(ctx, user, o.OrderID, db.OutcomeClassPreimage, outcome).Scan(&o.DBID); err != nil {

if err = stmt.QueryRowContext(ctx, user, o.OrderID, db.OutcomeClassPreimage, outcome).Scan(&o.DBID); err != nil {

Good catch!

JoeGruffins · 2025-05-08T08:43:00Z

server/db/driver/pg/reputation.go

+		}
+	}
+	for _, o := range matches {
+		if err := stmt.QueryRowContext(ctx, user, o.MatchID, db.OutcomeClassMatch, o.MatchOutcome).Scan(&o.DBID); err != nil {


Suggested change

if err := stmt.QueryRowContext(ctx, user, o.MatchID, db.OutcomeClassMatch, o.MatchOutcome).Scan(&o.DBID); err != nil {

if err = stmt.QueryRowContext(ctx, user, o.MatchID, db.OutcomeClassMatch, o.MatchOutcome).Scan(&o.DBID); err != nil {

JoeGruffins · 2025-05-08T08:43:12Z

server/db/driver/pg/reputation.go

+		if o.Canceled {
+			outcome = db.OutcomeOrderCanceled
+		}
+		if err := stmt.QueryRowContext(ctx, user, o.OrderID, db.OutcomeClassOrder, outcome).Scan(&o.DBID); err != nil {


Suggested change

if err := stmt.QueryRowContext(ctx, user, o.OrderID, db.OutcomeClassOrder, outcome).Scan(&o.DBID); err != nil {

if err = stmt.QueryRowContext(ctx, user, o.OrderID, db.OutcomeClassOrder, outcome).Scan(&o.DBID); err != nil {

JoeGruffins · 2025-05-08T08:49:11Z

server/auth/auth_test.go

+	// if score != wantScore {
+	// 	t.Errorf("wrong score. got %d, want %d", score, wantScore)
+	// }


Should we still check this?

…effects Loading reputation history from the orders and matches tables used some gnarly queries that were getting incredibly slow as the table sizes grew. This was causing extremly long connect times that often timed when the server connection flapped and everybody tried to connect at once. This PR adds a dedicated table for reputation effects that stays pruned, and a rolling upgrade process.

…ement

JoeGruffins · 2025-06-09T07:23:02Z

Tested forgiving a user, looks fine. Merging soon...

buck54321 force-pushed the v1-server-rep branch from c774678 to 5df63ad Compare April 30, 2025 20:27

JoeGruffins reviewed May 2, 2025

View reviewed changes

martonp reviewed May 2, 2025

View reviewed changes

JoeGruffins reviewed May 8, 2025

View reviewed changes

JoeGruffins approved these changes May 9, 2025

View reviewed changes

buck54321 added 3 commits May 12, 2025 17:56

match Outcome a stringer. Fix db outcome add ordering, fix index stat…

1b390da

…ement

handle new user rep query. properly scope errors under transaction

77a93ff

buck54321 force-pushed the v1-server-rep branch from 982d60d to 87c504c Compare May 12, 2025 23:02

add forgive_user admin api

c1e5774

buck54321 force-pushed the v1-server-rep branch from 87c504c to c1e5774 Compare May 13, 2025 05:11

-			})
-		}
-	}
+			})
+		}
+		if len(pings) >= pingSz && len(matches) >= matchSz && len(orders) >= orderSz {
+		    break
+		}
+	}

	link BYTEA, -- Order ID or Match ID
	link BYTEA UNIQUE, -- Order ID or Match ID

	if err := stmt.QueryRowContext(ctx, user, o.OrderID, db.OutcomeClassPreimage, outcome).Scan(&o.DBID); err != nil {
	if err := a.queries.insertPoints.QueryRowContext(ctx, user, o.OrderID, db.OutcomeClassPreimage, outcome).Scan(&o.DBID); err != nil {

	if err := stmt.QueryRowContext(ctx, user, o.OrderID, db.OutcomeClassPreimage, outcome).Scan(&o.DBID); err != nil {
	if err = stmt.QueryRowContext(ctx, user, o.OrderID, db.OutcomeClassPreimage, outcome).Scan(&o.DBID); err != nil {

server/{db,auth}: add a dedicated table for tracking reputation effects #3259

Are you sure you want to change the base?

server/{db,auth}: add a dedicated table for tracking reputation effects #3259

Uh oh!

Conversation

buck54321 commented Apr 30, 2025

Uh oh!

JoeGruffins left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

martonp left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

JoeGruffins commented May 8, 2025

Uh oh!

JoeGruffins commented May 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

JoeGruffins commented Jun 9, 2025

Uh oh!

Uh oh!

martonp left a comment •

edited

Loading

JoeGruffins commented May 8, 2025 •

edited

Loading