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

Eric Yang commented on HADOOP-9863:
-----------------------------------

Sorry, what I said in previous comment is not right, JINT_MAX is 32 bit signed 
int.  This is defined in jni type spec.  Hence, the patch is correct as it is.  
+1 looks good.
                
> Snappy compression in branch1 cannot work on Big-Endian 64 bit platform w/o 
> backporting HADOOP-8686
> ---------------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-9863
>                 URL: https://issues.apache.org/jira/browse/HADOOP-9863
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: native
>    Affects Versions: 1.1.1, 1.1.2, 1.2.1
>            Reporter: Yu Li
>            Assignee: Yu Li
>              Labels: native, ppc64, snappy
>         Attachments: HADOOP-9863.patch
>
>
> w/o changes made in HADOOP-8686 on SnappyCompressor.c, snappy compression in 
> branch-1 hadoop on big endian 64 bit platform (ppc64 for example) will 
> generate incorrect .snappy (almost-empty) file because of type casting from 
> size_t to jint. Will include more detailed analysis in comments.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to