rust code review

[dcherian]

Low priority things to look at

• Storage trait knows too much about Icechunk. It shouldn't know about ObjectIds, or tables ideally. One thing to think about is how to do efficient caching. If it's only bytes, we would have to do parsing multiple times, so that doesn't work.
• Review error enum variants. Some are probably unused.
• Review performance of all ok_or calls

[mpiannucci]

I also think the consolidated store knows too much about storage, it's restricting on people adding their own storage backends outside the library.

An example is if I wanted to add a special storage implementation for a WASM implementation I wouldn't want the core of the crate to have code about that backend generally