[ 
https://issues.apache.org/jira/browse/COUCHDB-769?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14956784#comment-14956784
 ] 

ASF GitHub Bot commented on COUCHDB-769:
----------------------------------------

Github user kxepal commented on a diff in the pull request:

    https://github.com/apache/couchdb-couch/pull/120#discussion_r41984693
  
    --- Diff: src/couch_att.erl ---
    @@ -438,17 +438,28 @@ digest_from_json(Props) ->
     
     
     to_json(Att, OutputData, DataToFollow, ShowEncoding) ->
    -    [Name, Data, DiskLen, AttLen, Enc, Type, RevPos, Md5] = fetch(
    +    [AttExternal,AttExternalSize, AttExternalMD5] = 
couch_att:fetch([att_external,att_external_size,att_external_md5],Att),
    +    [Name, Data, DiskLenTmp, AttLen, Enc, Type, RevPos, Md5] = fetch(
             [name, data, disk_len, att_len, encoding, type, revpos, md5], Att
         ),
    +    case AttExternal of
    +        "external" ->
    +            DiskLen = AttExternalSize,
    +            DigestProp = case AttExternalMD5 of
    +                <<>> -> [];
    +                Digest -> [{<<"digest">>, <<"md5-", Digest/binary>>}]
    +            end;
    +        _ ->
    +            DiskLen = DiskLenTmp,
    +            DigestProp = case base64:encode(Md5) of
    +                <<>> -> [];
    +                Digest -> [{<<"digest">>, <<"md5-", Digest/binary>>}]
    --- End diff --
    
    Can we not repeat ourself in Digest generation?


> Store large attachments external to the .couch file
> ---------------------------------------------------
>
>                 Key: COUCHDB-769
>                 URL: https://issues.apache.org/jira/browse/COUCHDB-769
>             Project: CouchDB
>          Issue Type: New Feature
>          Components: Database Core
>            Reporter: Robert Newson
>            Assignee: Adam Kocoloski
>         Attachments: external_attachments_alpha.patch
>
>
> For attachment-heavy applications storing the attachments in separate files 
> significantly eases compaction problems.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to