Skip to content

revert(analytics): drop the call-count column and its migration machinery - #390

Merged
prajjwalkumar17 merged 2 commits into
mainfrom
revert-387-feat/audit-call-counts
Aug 31, 2026
Merged

revert(analytics): drop the call-count column and its migration machinery#390
prajjwalkumar17 merged 2 commits into
mainfrom
revert-387-feat/audit-call-counts

Conversation

@prajjwalkumar17

@prajjwalkumar17 prajjwalkumar17 commented Aug 31, 2026

Copy link
Copy Markdown
Member

Reverts #387

Closes #402

@prajjwalkumar17 prajjwalkumar17 self-assigned this Aug 31, 2026
Copilot AI lite review requested due to automatic review settings August 31, 2026 11:01

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@prajjwalkumar17 prajjwalkumar17 changed the title Revert "feat(analytics): count evaluation calls in the Decision Audit, not per-entry events" revert(analytics): drop the call-count column and its migration machinery Aug 31, 2026
.is_some_and(|fallback| !fallback.is_empty()) =>
{
let call_payment_id = uniform_payment_id(&payload.requests);
let mut entry_outcomes = Vec::with_capacity(payload.requests.len());
};

let call_payment_id = uniform_payment_id(&payload.requests);
let mut entry_outcomes: Vec<Value> = Vec::with_capacity(payload.requests.len());
@prajjwalkumar17
prajjwalkumar17 merged commit 7472efa into main Aug 31, 2026
18 of 20 checks passed
@prajjwalkumar17
prajjwalkumar17 deleted the revert-387-feat/audit-call-counts branch August 31, 2026 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] One batch evaluate call floods the Decision Audit with per-entry events

3 participants