Signature Of KeyValue::Batch::Get-Keys

[adamgfraser]

The current signature is:

get-keys: func(bucket: bucket) -> keys

Unlike almost all the other functions in the interface, this one returns a raw value versus a result. This seems strange since getting the keys could fail with a variety of database related errors (e.g. can't connect to the database).

I would have thought the signature would be:

get-keys: func(bucket: bucket) -> result<keys, error>

[Mossaka]

Thanks for raising this issue! Yes, returning a raw value does seem strange to me. I think the new signature looks much better.