Skip to content

Commit 1ff6f59

Browse files
committed
chore(controlplane): drop private-issue references from public comments
Removes the PFM-5839 references in the migration header and the WorkflowRunRepo.List comment. The PFM tracker is private; the public codebase shouldn't link to it. The technical context remains in place. Assisted-by: Claude Code Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev> Chainloop-Trace-Sessions: f00cda5d-4707-40cf-800e-54da06ada048
1 parent 440a496 commit 1ff6f59

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

app/controlplane/pkg/data/ent/migrate/migrations/20260516210119.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
-- atlas:txmode none
22

33
-- Denormalize organization_id onto workflow_runs so org-scoped list/aggregate
4-
-- queries become sargable without joining workflows. See PFM-5839.
4+
-- queries become sargable without joining workflows.
55
--
66
-- Why a trigger?
77
--

app/controlplane/pkg/data/workflowrun.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -370,9 +370,9 @@ func (r *WorkflowRunRepo) List(ctx context.Context, orgID uuid.UUID, filters *bi
370370

371371
// Org-scoped query uses the denormalized organization_id column for a
372372
// sargable range scan via the (organization_id, created_at DESC) index.
373-
// See PFM-5839 — the prior HasWorkflowWith form compiled to a correlated
374-
// EXISTS that let the planner pick a bad ORDER BY created_at DESC plan.
375-
// The deleted_at filter still goes via the workflows edge: it's a per-row
373+
// The prior HasWorkflowWith form compiled to a correlated EXISTS that
374+
// let the planner pick a bad ORDER BY created_at DESC plan. The
375+
// deleted_at filter still goes via the workflows edge: it's a per-row
376376
// PK lookup applied after the index narrows the candidate set, so it
377377
// doesn't reintroduce the planner ambiguity.
378378
wfPredicates := []predicate.Workflow{workflow.DeletedAtIsNil()}

0 commit comments

Comments
 (0)