Revert "Label the transaction counter with account_id (cardinality experiment)"
This reverts commit a1f1d80. Phase 5 step 4 is done: the burst minted
68 series across 60 account_id values.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5.5
parent
a1f1d80bba
commit
d54ea2ba41
@@ -155,11 +155,7 @@ async def submit_transaction(body: TransactionRequest):
|
|||||||
finally:
|
finally:
|
||||||
transaction_duration.record(time.perf_counter() - start, {"kind": body.kind})
|
transaction_duration.record(time.perf_counter() - start, {"kind": body.kind})
|
||||||
currency = body.currency if body.currency in KNOWN_CURRENCIES else "other"
|
currency = body.currency if body.currency in KNOWN_CURRENCIES else "other"
|
||||||
# Phase 5 step 4: deliberate cardinality explosion. Revert after the experiment.
|
transactions_total.add(1, {"kind": body.kind, "status": status, "currency": currency})
|
||||||
transactions_total.add(
|
|
||||||
1,
|
|
||||||
{"kind": body.kind, "status": status, "currency": currency, "account_id": body.account_id},
|
|
||||||
)
|
|
||||||
return JSONResponse(status_code=201, content={**result, "request_id": request_id})
|
return JSONResponse(status_code=201, content={**result, "request_id": request_id})
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user