-
Notifications
You must be signed in to change notification settings - Fork 1
feat(kv): standalone kv store #49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Greptile OverviewGreptile SummaryAdds a new standalone KV store package ( Key changes:
Critical issues:
Confidence Score: 2/5
Important Files ChangedFile Analysis
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Additional Comments (1)
-
.github/workflows/release.yaml, line 52-54 (link)logic: tests not running for the new
@tigrisdata/kvpackage - should addnpm run test --workspace=@tigrisdata/kvto match the pattern of other packages
10 files reviewed, 2 comments
4c42d49 to
5377613
Compare
5377613 to
00392ec
Compare
00392ec to
997197e
Compare
|
Is this the keyv integration? |
@ovaistariq This is a standalone implementation. You don't need |
|
Closing this PR as features like TTL are better handled at backend. Will revise it when backend adds support for it |
Note
Introduces
@tigrisdata/kvpackage (TTL, namespaces, batch ops, iterators, events) and updates workflows/scripts to build and test it across the monorepo.@tigrisdata/kvKVclass over Tigris Storage withget/set/delete,getMany/setMany/deleteMany,has/hasMany,clear, asynciterator,entries/keys/values.namespaceprefixes, and error events.README.md,tsconfig.json,tsup.config.ts,vitest.config.ts, and entry points insrc/.packages/kvto workspaces; new scriptbuild:kv.npm run testacross all packages and includekv; removes release dry-run andNPM_TOKENenv from release.Written by Cursor Bugbot for commit 997197e. This will update automatically on new commits. Configure here.