Skip to content

duplicate#34

Open
sskumar18 wants to merge 1 commit intovcon-dev:mainfrom
sskumar18:fix/duplicate
Open

duplicate#34
sskumar18 wants to merge 1 commit intovcon-dev:mainfrom
sskumar18:fix/duplicate

Conversation

@sskumar18
Copy link
Contributor

┌─────────────────────────────────────────────────────────┐
│  PASS 1 (lines 745-768) - "candidate" pre-filter        │
│                                                          │
│  Query parties WHERE name ILIKE '%alice%'               │
│  → gets vcon_ids: [101, 202, 303]                       │
│  → stores in candidateVconIds = Set{101, 202, 303}      │
└─────────────────────────────────────────────────────────┘
                         ↓
┌─────────────────────────────────────────────────────────┐
│  MAIN QUERY (lines 771-800)                             │
│                                                          │
│  SELECT uuid FROM vcons                                 │
│    WHERE id IN (101, 202, 303)   ← candidateVconIds     │
│    AND subject ILIKE '%meeting%'                        │
│  → returns uuids: ['abc', 'def']                        │
└─────────────────────────────────────────────────────────┘
                         ↓
┌─────────────────────────────────────────────────────────┐
│  PASS 2 (lines 826-855) - ❌ the duplicate              │
│                                                          │
│  Query parties WHERE name ILIKE '%alice%'  ← AGAIN!    │
│  → gets same vcon_ids: [101, 202, 303]                  │
│  → then fetches vcons WHERE id IN (101, 202, 303)       │
│  → then filters ['abc', 'def'] against that list        │
│  → result is still ['abc', 'def'] — nothing changed!    │
└─────────────────────────────────────────────────────────┘

@cursor
Copy link

cursor bot commented Feb 25, 2026

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on March 17.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

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.

1 participant