accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chris Carrino (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-1968) The FileDataIngest example creates MD5 hash keys, but the javadoc says it creates SHA1 hash keys.
Date Thu, 05 Dec 2013 11:52:35 GMT

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

Chris Carrino commented on ACCUMULO-1968:
-----------------------------------------

I just realized another documentation page is affected as well:  http://accumulo.apache.org/1.5/examples/filedata.html.
 The "SHA1" text should probably be changed to "MD5", unless the code itself is changed to
generate SHA1 keys. 

> The FileDataIngest example creates MD5 hash keys, but the javadoc says it creates SHA1
hash keys.
> -------------------------------------------------------------------------------------------------
>
>                 Key: ACCUMULO-1968
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1968
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.4.4, 1.5.0
>            Reporter: Chris Carrino
>            Priority: Trivial
>              Labels: documentation
>             Fix For: 1.7.0
>
>   Original Estimate: 2m
>  Remaining Estimate: 2m
>
> In file org.apache.accumulo.examples.simple.filedata.FileDataIngest the javadoc comment
says "Takes a list of files and archives them into Accumulo keyed on the SHA1 hashes of the
files."  But the code on line 64 creates an MD5 hash using MessageDigest.getInstance("MD5").
> So either the "SHA1" in javadoc should be changed to "MD5", or the "MD5" in the MessageDigest.getInstance()
method argument should be changed to "SHA1".



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Mime
View raw message