mbr / simplekv

A simple key-value store for binary data.
http://simplekv.readthedocs.io
MIT License
152 stars 50 forks source link

Add checksum feature for Azure blob storage. #77

Closed amerkel2 closed 6 years ago

amerkel2 commented 6 years ago

Add the option of setting content_md5 in the content settings of an Azure blob during upload. This is helpful for checking if two blobs are identical without having to download them.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-9.3%) to 80.0% when pulling 5d6201aaa28d6c675613be21dd9bf620b8ca23d3 on amerkel2:master into 207f2a45c38fdb1c91b617229f7bdbf77f730f99 on mbr:master.

amerkel2 commented 6 years ago

Coverage decrease is because Azure tests are not executed in PRs.

criemen commented 6 years ago

Reverted because it is not working on python 3. Before merging again, I can push the branch to the mbr/simplekv-repo so travis tests are executed for azure.

amerkel2 commented 6 years ago

I've opened https://github.com/mbr/simplekv/pull/78 with fixes that enable Python3 compatibility. Can you push that one to mbr/simplekv-repo?

criemen commented 6 years ago

See https://travis-ci.org/mbr/simplekv/jobs/400004927, tests are working 👍