-
-
Notifications
You must be signed in to change notification settings - Fork 29
feat: add getMany() method #99
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
base: main
Are you sure you want to change the base?
Conversation
…ance L1/L2 interaction with detailed event emission.
…apters - Introduced `getMany` method in `CacheProvider` interface for batch retrieval of cache values. - Implemented `getMany` in `BentoCache`, `Cache`, `KnexAdapter`, `OrchidAdapter`, and `DynamoDbDriver` to support fetching multiple keys efficiently. - Enhanced error handling and logging in DynamoDB driver for unprocessed keys. - Added comprehensive tests for `getMany` functionality, covering various scenarios including defaults, expired items, and concurrent operations. - Updated typings to ensure correct type inference for `getMany` method across different cache implementations.
🦋 Changeset detectedLatest commit: ba7c3e7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
…ny` `defaultValue` handling with new tests.
…econd-level TTLs and adding a delay.
|
Just was looking for a way to get multiple keys from the database, since in my project there are pretty large T1 cache misses. |
|
Hi @Julien-R44, just wanted to check if there are any updates on this PR. Appreciate your time and feedback! |
|
Hey, quick question, was this AI-generated? Just asking because it looks like it might be. |
|
Yes, I did use AI assistance while drafting parts of the code, but I’ve reviewed and understand those sections. I completely understand the need for extra review time, so please take whatever time you need. |
closes #78