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

Masatake Iwasaki commented on HADOOP-17144:
-------------------------------------------

{quote}
we do have a test case similar to this scenario in 
TestCompressorDecompressor#testCompressorDecompressorWithExeedBufferLimit , 
modified the lz4 constructors to use default buffer size 
{quote}

[~hemanthboyina] Interestingly the test case does not cover the intended 
scenario. I filed HADOOP-17270 for that. The 004 patch fails on the updated 
test case. Reverting the change below fixed the failure.

{noformat}
@@ -120,7 +121,7 @@ public synchronized void setInput(byte[] b, int off, int 
len) {
    * consumed.
    */
   synchronized void setInputFromSavedData() {
-    compressedDirectBufLen = Math.min(userBufLen, directBufferSize);
+    compressedDirectBufLen = userBufLen;

     // Reinitialize lz4's input direct buffer
     compressedDirectBuf.rewind();
{noformat}


> Update Hadoop's lz4 to v1.9.2
> -----------------------------
>
>                 Key: HADOOP-17144
>                 URL: https://issues.apache.org/jira/browse/HADOOP-17144
>             Project: Hadoop Common
>          Issue Type: Improvement
>            Reporter: Hemanth Boyina
>            Assignee: Hemanth Boyina
>            Priority: Major
>         Attachments: HADOOP-17144.001.patch, HADOOP-17144.002.patch, 
> HADOOP-17144.003.patch, HADOOP-17144.004.patch
>
>
> Update hadoop's native lz4 to v1.9.2 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to