Implementing ZipStore's __delitem__ via overwrite

[jakirkham]

Currently ZipStore's __delitem__ raises NotImplementedError, which is a bit awkward to workaround at higher levels since __delitem__ is usually implemented in other stores. The reason this isn't implemented in ZipStore is zip files lack the ability to delete members.

An alternative approach that might be worth considering is overwriting the existing members with an empty string (b""). We can then treat them as deleted in Zarr.

[I-am-Emmanuel]

I'm an outreachy applicant, can I contribute to this project?

[jakirkham]

Sure, please do Emmanuel 🙂

[jakirkham]

cc @MSanKeys963 @joshmoore (for awareness)

[KarlieKKY]

I'm an outreachy applicant, can I contribute to this project?

Hello, i'm also an outcheachy applicant, would you mind if we work on this issue together?

[jakirkham]

Thanks for your interest! 😀

Emmanuel opened PR ( #1184 ) already.

Would suggest picking a different issue 🙂

[jakirkham]

Also worth noting there is an upstream issue about adding this functionality in CPython ( python/cpython#51067 )