feat: support for key enumeration in the postgres store#359
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
WalkthroughThis PR extends the PostgreSQL key-value store with collection-level key enumeration. A new 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
No issues found across 1 file
Requires human review: This pull request implements a new feature for key enumeration in the PostgreSQL store; because it introduces new database query logic and extends the store's interface, a human review is necessary.
Re-trigger cubic
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Auto-approved: This change adds a new read-only method to list keys in a PostgreSQL collection, following the same patterns as existing methods, with comprehensive tests covering edge cases, and does not alter any existing business logic or database schema.
Re-trigger cubic
|
I'm happy to merge this but keep in mind that the AsyncKeyValue protocol does not expose key enumeration so this has limited utility in common use-cases |
No description provided.