Support more map containers
- Dominant language
- C++
- Stars
- 282
- Forks
- 49
- PR merge metrics
- No merged PRs in 30d
Description
I'd like to use `response>>` or `response>` as my response type for `HGETALL`, but the adapter is hardcoded to only support the standard library associative containers:
https://github.com/boostorg/redis/blob/6f17663165a8db3cbac8bf3065355b420032fc21/include/boost/redis/adapter/detail/adapters.hpp#L487-L530
It's also missing the new ones from C++23 (ie. `flat_map`). It would ideally support anything with a "map-like" interface.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start in include/boost/redis/adapter/detail/adapters.hpp around lines 487-530 and inspect how HGETALL responses are restricted to standard associative containers. Define the supported map-like shape from the requested vector>, absl::flat_hash_map, and C++23 flat_map examples; done means these response types are accepted instead of only the currently hardcoded containers.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp, redis
- Domain
- backend-api-design
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 56/100