get_multiple doesn't document result order

khonsulabs / bonsaidb

A developer-friendly document database that grows with you, written in Rust

https://bonsaidb.io/

Apache License 2.0

1.02k stars 37 forks source link

get_multiple doesn't document result order #315

Open ecton opened 6 months ago

ecton commented 6 months ago

The implementation of get_multiple does not document that the returned list's order is not guaranteed. Additionally, we might want to add another function such as get_multiple_ordered that performs the additional step of sorting the results to match the input request, minus any missing items.

Originally inspired by a question by @martpie in https://github.com/khonsulabs/bonsaidb/issues/237#issuecomment-1987342741

martpie commented 6 months ago

I can maybe work on a PR if you don't have the bandwidth :)

ecton commented 6 months ago

I'm not currently focused on this project but don't mind tackling this once I get back to BonsaiDb. But I'm also more than happy to accept a PR!