Skip to content
This repository was archived by the owner on May 25, 2021. It is now read-only.

Conversation

gilv
Copy link

@gilv gilv commented Oct 14, 2015

Initial implementation that allows CouchDB to store attachments outside of the database file.
This implementation supports OpenStack Swift and SoftLayer Object store

…mplementation, supports OpenStack Swift and SoftLayer Object store
DiskLen = DiskLenTmp,
DigestProp = case base64:encode(Md5) of
<<>> -> [];
Digest -> [{<<"digest">>, <<"md5-", Digest/binary>>}]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we not repeat ourself in Digest generation?

@kxepal
Copy link
Member

kxepal commented Oct 14, 2015

Please fix style: 80 chars line length, commas/spaces, etc.

@gilv
Copy link
Author

gilv commented Jan 4, 2016

@kxepal @rnewson
I provided another patch, implementing most of the comments.
Here is what is not implemented in this patch:

  1. Remark on the AttFun. Not yet implemented. Original remark: “AttFun could be actually an attachment data or some fold function? Something went wrong if you have to do this.”
  2. usage of couch_util:to_hex
  3. still no unitests

@gilv
Copy link
Author

gilv commented Jan 29, 2016

@kxepal Can you please review the recent code?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants