[recipes] Fix UUID cursor in fingerprint dedup backfill#152
Open
demarant wants to merge 2 commits intoNateBJones-Projects:mainfrom
Open
[recipes] Fix UUID cursor in fingerprint dedup backfill#152demarant wants to merge 2 commits intoNateBJones-Projects:mainfrom
demarant wants to merge 2 commits intoNateBJones-Projects:mainfrom
Conversation
- Use created_at instead of id for cursor pagination - Fixes error when thoughts table has UUID primary key - Tested on real Open Brain DB with 1815 rows backfilled
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contribution Type
/recipes)/schemas)/dashboards)/integrations)/skills)What does this do?
Fixes the fingerprint dedup backfill scripts to work with UUID primary keys on the thoughts table. Changed cursor pagination from
id(integer) tocreated_at(timestamp), which works for any ID type and resolves the errorinvalid input syntax for type uuid: "0".Requirements
content_fingerprintcolumn exists on thoughts table)Checklist
README.mdwith prerequisites, step-by-step instructions, and expected outcomemetadata.jsonhas all required fieldscontent-fingerprint-dedupprimitive (declared in existing README)Test Results
Outcome: ✅ Fix verified working on real Open Brain database with UUID primary key.