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

Todd Lipcon commented on HADOOP-7657:
-------------------------------------

Actually, two small comments:

- Can you please indicate in lz4.c which SVN revision or release it is copied 
from on the main repository? This will help if we want to update it in the 
future.
- Can you please add text like the following to LICENSE.txt in the 
hadoop-common project root?
{code}
For src/main/native/src/org/apache/hadoop/io/compress/lz4/lz4.c:

/*
   LZ4 - Fast LZ compression algorithm
   Copyright (C) 2011, Yann Collet.
   BSD License

   Redistribution and use in source and binary forms, with or without
   modification, are permitted provided that the following conditions are
   met:
  
       * Redistributions of source code must retain the above copyright
   notice, this list of conditions and the following disclaimer.
       * Redistributions in binary form must reproduce the above
   copyright notice, this list of conditions and the following disclaimer
   in the documentation and/or other materials provided with the
   distribution.
  
   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
   A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
   OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
   SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
   LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
   DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
   THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
{code}
so that we comply correctly with the restriction that binary distributions will 
include the license text and copyright?
                
> Add support for LZ4 compression
> -------------------------------
>
>                 Key: HADOOP-7657
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7657
>             Project: Hadoop Common
>          Issue Type: Improvement
>            Reporter: Mr Bsd
>              Labels: compression
>         Attachments: HADOOP-7657.lz4.v1.patch
>
>
> According to several benchmark sites, LZ4 seems to overtake other fast 
> compression algorithms, especially in the decompression speed area. The 
> interface is also trivial to integrate 
> (http://code.google.com/p/lz4/source/browse/trunk/lz4.h) and there is no 
> license issue.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to