1527104 - use rkv::OwnedValue instead of internal OwnedValue in kvstore

Status: RESOLVED FIXED

(Toolkit :: Storage, enhancement, P3)

Description

[Myk Melez [:myk] [@mykmelez]]

The kvstore crate that landed in bug 1490496 uses an internal implementation of OwnedValue because the rkv crate's equivalent was insufficient at the time that kvstore was being developed.

rkv's OwnedValue has since evolved to support kvstore's use cases, and bug 1525392 is updating mozilla-central to the latest version of rkv; so we should replace kvstore's internal OwnedValue with rkv::OwnedValue.

Comment 1

[Myk Melez [:myk] [@mykmelez]]

Attachment: use rkv::OwnedValue instead of internal OwnedValue in kvstore

The kvstore crate that landed in bug 1490496 uses an internal implementation of OwnedValue because the rkv crate's equivalent was insufficient at the time that kvstore was being developed.

rkv's OwnedValue has since evolved to support kvstore's use cases, and bug 1525392 is updating mozilla-central to the latest version of rkv; so we should replace kvstore's internal OwnedValue with rkv::OwnedValue.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/977d3cf4a200
use rkv::OwnedValue instead of internal OwnedValue in kvstore r=nika

Comment 3

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/977d3cf4a200