Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-10027

*Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.8.0, 3.0.0-alpha1
    • Component/s: native
    • Labels:
      None
    • Target Version/s:
    • Hadoop Flags:
      Reviewed

      Description

      http://svn.apache.org/viewvc/hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c?view=markup

      This pattern appears in all the native compressors.
      // Get members of ZlibCompressor
      jobject clazz = (*env)->GetStaticObjectField(env, this,
      ZlibCompressor_clazz);

      The 2nd argument to GetStaticObjectField is supposed to be a jclass, not a jobject. Adding the JVM param -Xcheck:jni will cause "FATAL ERROR in native method: JNI received a class argument that is not a class" and a core dump such as the following.

      (gdb)
      #0 0x00007f02e4aef8a5 in raise () from /lib64/libc.so.6
      #1 0x00007f02e4af1085 in abort () from /lib64/libc.so.6
      #2 0x00007f02e45bd727 in os::abort(bool) () from /opt/jdk1.6.0_31/jre/lib/amd64/server/libjvm.so
      #3 0x00007f02e43cec63 in jniCheck::validate_class(JavaThread*, _jclass*, bool) () from /opt/jdk1.6.0_31/jre/lib/amd64/server/libjvm.so
      #4 0x00007f02e43ea669 in checked_jni_GetStaticObjectField () from /opt/jdk1.6.0_31/jre/lib/amd64/server/libjvm.so
      #5 0x00007f02d38eaf79 in Java_org_apache_hadoop_io_compress_zlib_ZlibCompressor_deflateBytesDirect () from /usr/lib/hadoop/lib/native/libhadoop.so.1.0.0

      In addition, that clazz object is only used for synchronization. In the case of the native method _deflateBytesDirect, the result is a class wide lock used to access the instance field uncompressed_direct_buf. Perhaps using the instance as the sync point is more appropriate?

      1. HADOOP-10027.5.patch
        37 kB
        Hui Zheng
      2. HADOOP-10027.4.patch
        35 kB
        Hui Zheng
      3. HADOOP-10027.3.patch
        34 kB
        Hui Zheng
      4. HADOOP-10027.2.patch
        7 kB
        Hui Zheng
      5. HADOOP-10027.1.patch
        2 kB
        Hui Zheng

        Issue Links

          Activity

          Hide
          spackler Eric Abbott added a comment -

          Of note, the object used for sync inside the C code is pulled from the java side with the malformed GetStaticObjectField call. 3 things can happen:

          1) core dump: We've seen some of these.
          2) clazz (or another static) object from java side returned: synch will be broader by ok
          3) Some object that is not static: uncompressed_direct_buf amongst others will be protected by two different monitor objects which could lead to corruption

          Show
          spackler Eric Abbott added a comment - Of note, the object used for sync inside the C code is pulled from the java side with the malformed GetStaticObjectField call. 3 things can happen: 1) core dump: We've seen some of these. 2) clazz (or another static) object from java side returned: synch will be broader by ok 3) Some object that is not static: uncompressed_direct_buf amongst others will be protected by two different monitor objects which could lead to corruption
          Hide
          huizane Hui Zheng added a comment -

          I take it that the problem happened in GetStaticObjectField method,however it would be better to use a instance wide lock instead of a class wide lock.
          But I have no idea why you need to synchronize the GetDirectBufferAddress method such as

          LOCK_CLASS(env, clazz, "ZlibCompressor");
          uncompressed_bytes = (*env)->GetDirectBufferAddress(env,
          uncompressed_direct_buf);
          UNLOCK_CLASS(env, clazz, "ZlibCompressor");

          I think if we need thread-safe,we should synchronize every method(which use compressedDirectBuf or uncompressedDirectBuf) of ZlibCompressor.

          Show
          huizane Hui Zheng added a comment - I take it that the problem happened in GetStaticObjectField method,however it would be better to use a instance wide lock instead of a class wide lock. But I have no idea why you need to synchronize the GetDirectBufferAddress method such as LOCK_CLASS(env, clazz, "ZlibCompressor"); uncompressed_bytes = (*env)->GetDirectBufferAddress(env, uncompressed_direct_buf); UNLOCK_CLASS(env, clazz, "ZlibCompressor"); I think if we need thread-safe,we should synchronize every method(which use compressedDirectBuf or uncompressedDirectBuf) of ZlibCompressor.
          Hide
          cmccabe Colin P. McCabe added a comment -

          Indeed, those lines make no sense at all. Java_org_apache_hadoop_io_compress_bzip2_Bzip2Compressor_deflateBytesDirect is already only ever called by a synchronized java method, so it should already be thread-safe. Even if we somehow did need synchronization, we would need it for the entire time we were using the buffer, not just for calling GetDirectBufferAddress.

          We should just delete those lines since there's no possible way they could help, and clearly they are causing crashing bugs.

          Show
          cmccabe Colin P. McCabe added a comment - Indeed, those lines make no sense at all. Java_org_apache_hadoop_io_compress_bzip2_Bzip2Compressor_deflateBytesDirect is already only ever called by a synchronized java method, so it should already be thread-safe. Even if we somehow did need synchronization, we would need it for the entire time we were using the buffer, not just for calling GetDirectBufferAddress . We should just delete those lines since there's no possible way they could help, and clearly they are causing crashing bugs.
          Hide
          chris.douglas Chris Douglas added a comment -

          IIRC the lock was working around a JVM bug. Arun C Murthy, do you remember the details?

          Show
          chris.douglas Chris Douglas added a comment - IIRC the lock was working around a JVM bug. Arun C Murthy , do you remember the details?
          Hide
          huizane Hui Zheng added a comment -

          Thank you for your comments.
          I understand that the lock is not used for atomic operations but used to avoid a JVM bug.
          So I think it is dangerous to delete these lines.
          If the JVM bug is fixed in the future,we'd better to delete them.

          Show
          huizane Hui Zheng added a comment - Thank you for your comments. I understand that the lock is not used for atomic operations but used to avoid a JVM bug. So I think it is dangerous to delete these lines. If the JVM bug is fixed in the future,we'd better to delete them.
          Hide
          huizane Hui Zheng added a comment -

          I attached a patch.

          Show
          huizane Hui Zheng added a comment - I attached a patch.
          Hide
          huizane Hui Zheng added a comment -

          It can be tested by TestZlibCompressorDecompressor with " -Xcheck:jni -Djava.library.path=XXX" JVM options.

          Show
          huizane Hui Zheng added a comment - It can be tested by TestZlibCompressorDecompressor with " -Xcheck:jni -Djava.library.path=XXX" JVM options.
          Hide
          hadoopqa Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12700996/HADOOP-10027.1.patch
          against trunk revision 71385f9.

          +1 @author. The patch does not contain any @author tags.

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          +1 core tests. The patch passed unit tests in hadoop-common-project/hadoop-common.

          Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/5783//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/5783//console

          This message is automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12700996/HADOOP-10027.1.patch against trunk revision 71385f9. +1 @author . The patch does not contain any @author tags. -1 tests included . The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-common-project/hadoop-common. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/5783//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/5783//console This message is automatically generated.
          Hide
          cmccabe Colin P. McCabe added a comment -

          Thanks for the additional info, Chris Douglas. If this is working around a JVM bug, we had better add a comment explaining which JVMs have the bug, and linking to the bug report. I also strongly suspect that even if this was an issue in the past, it no longer is with now that we don't support JVMs older than 1.7 (java 7).

          I apologize for being so skeptical, but my first instinct when our JNI code and the Sun / Oracle docs conflict is to believe that our JNI code is wrong. I've just seen so much bad JNI code I hope that we can clear this up.

          Show
          cmccabe Colin P. McCabe added a comment - Thanks for the additional info, Chris Douglas . If this is working around a JVM bug, we had better add a comment explaining which JVMs have the bug, and linking to the bug report. I also strongly suspect that even if this was an issue in the past, it no longer is with now that we don't support JVMs older than 1.7 (java 7). I apologize for being so skeptical, but my first instinct when our JNI code and the Sun / Oracle docs conflict is to believe that our JNI code is wrong. I've just seen so much bad JNI code I hope that we can clear this up.
          Hide
          chris.douglas Chris Douglas added a comment -

          This was a really long time ago (2008?); it's almost certainly not an issue anymore. IIRC it caused crashes or hangs, not silent corruption. It's probably safe to remove.

          Show
          chris.douglas Chris Douglas added a comment - This was a really long time ago (2008?); it's almost certainly not an issue anymore. IIRC it caused crashes or hangs, not silent corruption. It's probably safe to remove.
          Hide
          cnauroth Chris Nauroth added a comment -

          My best friend, the git blame log, tells me that this traces back to commit 9e22afd8704f62e312e749ebdb882455569de560, which was HADOOP-3604 from 2008. We should be able to use the information in that issue to trace back to specific JVM versions that exhibited this bug. Assuming those are all very old JVM versions, then I am +1 for the approach of removing this locking completely. Thanks for taking this on Ki Tei.

          Show
          cnauroth Chris Nauroth added a comment - My best friend, the git blame log, tells me that this traces back to commit 9e22afd8704f62e312e749ebdb882455569de560, which was HADOOP-3604 from 2008. We should be able to use the information in that issue to trace back to specific JVM versions that exhibited this bug. Assuming those are all very old JVM versions, then I am +1 for the approach of removing this locking completely. Thanks for taking this on Ki Tei .
          Hide
          huizane Hui Zheng added a comment -

          I updated the patch which delete these code and add a testcase for multithread.

          Show
          huizane Hui Zheng added a comment - I updated the patch which delete these code and add a testcase for multithread.
          Hide
          hadoopqa Hadoop QA added a comment -

          +1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12701338/HADOOP-10027.2.patch
          against trunk revision 4f75b15.

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 1 new or modified test files.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          +1 core tests. The patch passed unit tests in hadoop-common-project/hadoop-common.

          Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/5792//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/5792//console

          This message is automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - +1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12701338/HADOOP-10027.2.patch against trunk revision 4f75b15. +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-common-project/hadoop-common. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/5792//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/5792//console This message is automatically generated.
          Hide
          cnauroth Chris Nauroth added a comment -

          Hui Zheng, thank you for the patch. This looks like the right approach. I did a closer review of HADOOP-3604, and the linked Java bugs are quite old. I don't believe any realistic deployment would still be running on such old Java versions.

          This same code pattern is in all of the native compression codecs, likely due to copy-paste. To make this patch comprehensive, let's update all of them: bzip2, lz4, snappy and zlib.

          The new test is a good idea, but I think it needs some changes. As written, it starts 10 threads, but then JUnit will leave execution of testZlibCompressDecompressInMultiThreads before those threads really complete. If an exception is thrown from within the background thread, there is no reporting back to the main JUnit thread. Because of those 2 things, unexpected failures on the background threads wouldn't actually show up as JUnit failures. To fix this, I think you'll need to capture the Thread instances in array, join to all of them at the end of the test, and also work out a way to propagate possible exceptions out of those threads. There is a helper class at org.apache.hadoop.test.MultithreadedTestUtil that might help you implement this.

          Show
          cnauroth Chris Nauroth added a comment - Hui Zheng , thank you for the patch. This looks like the right approach. I did a closer review of HADOOP-3604 , and the linked Java bugs are quite old. I don't believe any realistic deployment would still be running on such old Java versions. This same code pattern is in all of the native compression codecs, likely due to copy-paste. To make this patch comprehensive, let's update all of them: bzip2, lz4, snappy and zlib. The new test is a good idea, but I think it needs some changes. As written, it starts 10 threads, but then JUnit will leave execution of testZlibCompressDecompressInMultiThreads before those threads really complete. If an exception is thrown from within the background thread, there is no reporting back to the main JUnit thread. Because of those 2 things, unexpected failures on the background threads wouldn't actually show up as JUnit failures. To fix this, I think you'll need to capture the Thread instances in array, join to all of them at the end of the test, and also work out a way to propagate possible exceptions out of those threads. There is a helper class at org.apache.hadoop.test.MultithreadedTestUtil that might help you implement this.
          Hide
          huizane Hui Zheng added a comment -

          Chris Nauroth
          Thank you for your help.
          I checked bzip2, lz4 and snappy. Yes,they should also be deleted.
          But I can't found any test code of bzip2. Is it being used.

          Show
          huizane Hui Zheng added a comment - Chris Nauroth Thank you for your help. I checked bzip2, lz4 and snappy. Yes,they should also be deleted. But I can't found any test code of bzip2. Is it being used.
          Hide
          cnauroth Chris Nauroth added a comment -

          That's interesting. It looks like we have a testing gap for coverage of bzip2. I recommend adding src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java with a test similar to what you're doing for zlib.

          Show
          cnauroth Chris Nauroth added a comment - That's interesting. It looks like we have a testing gap for coverage of bzip2. I recommend adding src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java with a test similar to what you're doing for zlib.
          Hide
          cmccabe Colin P. McCabe added a comment -

          Chris wrote: My best friend, the git blame log, tells me that this traces back to commit 9e22afd8704f62e312e749ebdb882455569de560, which was HADOOP-3604 from 2008. We should be able to use the information in that issue to trace back to specific JVM versions that exhibited this bug. Assuming those are all very old JVM versions, then I am +1 for the approach of removing this locking completely. Thanks for taking this on Hui Zheng.

          Thanks, Chris. Very good research. Indeed, the "fix version" on all those java bugs is somewhere between 1.3 and 1.5, inclusive. Since our min JDK version is 1.7 now (or possibly 1.6 if we backport this to 2.6.x), it should be an issue.

          I agree that the new testZlibCompressDecompressInMultiThreads function needs some work. We definitely should be joining those threads before the end of the test. Does it make sense to commit what we have now (minus the new unit test), and create a new JIRA for adding proper bzip2 test coverage? That might keep the scope more manageable. I am certainly +1 on the non-unit-test part of the patch as-is.

          Show
          cmccabe Colin P. McCabe added a comment - Chris wrote: My best friend, the git blame log, tells me that this traces back to commit 9e22afd8704f62e312e749ebdb882455569de560, which was HADOOP-3604 from 2008. We should be able to use the information in that issue to trace back to specific JVM versions that exhibited this bug. Assuming those are all very old JVM versions, then I am +1 for the approach of removing this locking completely. Thanks for taking this on Hui Zheng. Thanks, Chris. Very good research. Indeed, the "fix version" on all those java bugs is somewhere between 1.3 and 1.5, inclusive. Since our min JDK version is 1.7 now (or possibly 1.6 if we backport this to 2.6.x), it should be an issue. I agree that the new testZlibCompressDecompressInMultiThreads function needs some work. We definitely should be joining those threads before the end of the test. Does it make sense to commit what we have now (minus the new unit test), and create a new JIRA for adding proper bzip2 test coverage? That might keep the scope more manageable. I am certainly +1 on the non-unit-test part of the patch as-is.
          Hide
          huizane Hui Zheng added a comment -

          I also suggest to create a new JIRA for bzip2 test coverage and in this JIRA we should only delete the unnecessary lock/unlock code and add the testcase of multiple-threads.

          Show
          huizane Hui Zheng added a comment - I also suggest to create a new JIRA for bzip2 test coverage and in this JIRA we should only delete the unnecessary lock/unlock code and add the testcase of multiple-threads.
          Hide
          cnauroth Chris Nauroth added a comment -

          I agree with the plan of fixing the bug across all codecs here, and then deferring bzip2 test coverage to a separate jira. Thanks!

          Show
          cnauroth Chris Nauroth added a comment - I agree with the plan of fixing the bug across all codecs here, and then deferring bzip2 test coverage to a separate jira. Thanks!
          Hide
          huizane Hui Zheng added a comment -

          I updated the patch for zip,bzip2,lzo and snappy.

          Show
          huizane Hui Zheng added a comment - I updated the patch for zip,bzip2,lzo and snappy.
          Hide
          hadoopqa Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12702397/HADOOP-10027.3.patch
          against trunk revision 29bb689.

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 4 new or modified test files.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 core tests. The following test timeouts occurred in hadoop-common-project/hadoop-common:

          org.apache.hadoop.io.compress.TestCodec

          Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/5840//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/5840//console

          This message is automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12702397/HADOOP-10027.3.patch against trunk revision 29bb689. +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 4 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 core tests . The following test timeouts occurred in hadoop-common-project/hadoop-common: org.apache.hadoop.io.compress.TestCodec Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/5840//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/5840//console This message is automatically generated.
          Hide
          cmccabe Colin P. McCabe added a comment -

          Not sure what the issue was here. It looks kind of like a jenkins problem? Not sure.

          [INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ hadoop-auth ---
          FATAL: hudson.remoting.RequestAbortedException: hudson.remoting.Channel$OrderlyShutdown
          hudson.remoting.RequestAbortedException: hudson.remoting.RequestAbortedException: hudson.remoting.Channel$OrderlyShutdown
          	at hudson.remoting.RequestAbortedException.wrapForRethrow(RequestAbortedException.java:41)
          	at hudson.remoting.RequestAbortedException.wrapForRethrow(RequestAbortedException.java:34)
          	at hudson.remoting.Request.call(Request.java:174)
          	at hudson.remoting.Channel.call(Channel.java:742)
          	at hudson.remoting.RemoteInvocationHandler.invoke(RemoteInvocationHandler.java:168)
          	at com.sun.proxy.$Proxy57.join(Unknown Source)
          	at hudson.Launcher$RemoteLauncher$ProcImpl.join(Launcher.java:956)
          

          I will retrigger.

          Show
          cmccabe Colin P. McCabe added a comment - Not sure what the issue was here. It looks kind of like a jenkins problem? Not sure. [INFO] --- maven-resources-plugin:2.6:resources ( default -resources) @ hadoop-auth --- FATAL: hudson.remoting.RequestAbortedException: hudson.remoting.Channel$OrderlyShutdown hudson.remoting.RequestAbortedException: hudson.remoting.RequestAbortedException: hudson.remoting.Channel$OrderlyShutdown at hudson.remoting.RequestAbortedException.wrapForRethrow(RequestAbortedException.java:41) at hudson.remoting.RequestAbortedException.wrapForRethrow(RequestAbortedException.java:34) at hudson.remoting.Request.call(Request.java:174) at hudson.remoting.Channel.call(Channel.java:742) at hudson.remoting.RemoteInvocationHandler.invoke(RemoteInvocationHandler.java:168) at com.sun.proxy.$Proxy57.join(Unknown Source) at hudson.Launcher$RemoteLauncher$ProcImpl.join(Launcher.java:956) I will retrigger.
          Hide
          hadoopqa Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12702397/HADOOP-10027.3.patch
          against trunk revision ded0200.

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 4 new or modified test files.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 core tests. The following test timeouts occurred in hadoop-common-project/hadoop-common:

          org.apache.hadoop.io.compress.TestCodec

          Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/5851//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/5851//console

          This message is automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12702397/HADOOP-10027.3.patch against trunk revision ded0200. +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 4 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 core tests . The following test timeouts occurred in hadoop-common-project/hadoop-common: org.apache.hadoop.io.compress.TestCodec Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/5851//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/5851//console This message is automatically generated.
          Hide
          cmccabe Colin P. McCabe added a comment -

          It looks like the TestCodec failure is reproducible. Hui Zheng, can you look into this?

          Show
          cmccabe Colin P. McCabe added a comment - It looks like the TestCodec failure is reproducible. Hui Zheng , can you look into this?
          Hide
          huizane Hui Zheng added a comment -
          Show
          huizane Hui Zheng added a comment - Thank you to https://issues.apache.org/jira/secure/ViewProfile.jspa?name=cmccabe . I updated the patch.
          Hide
          hadoopqa Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12702977/HADOOP-10027.4.patch
          against trunk revision 952640f.

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 4 new or modified test files.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          -1 eclipse:eclipse. The patch failed to build with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          +1 core tests. The patch passed unit tests in .

          Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/5864//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/5864//console

          This message is automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12702977/HADOOP-10027.4.patch against trunk revision 952640f. +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 4 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. -1 eclipse:eclipse . The patch failed to build with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in . Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/5864//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/5864//console This message is automatically generated.
          Hide
          huizane Hui Zheng added a comment -

          I have no idea what is the problem " The patch failed to build with eclipse:eclipse."?

          Show
          huizane Hui Zheng added a comment - I have no idea what is the problem " The patch failed to build with eclipse:eclipse."?
          Hide
          cnauroth Chris Nauroth added a comment -

          Hui Zheng, this is looking good, and thank you for adding the new bzip2 test suite. I have just a few more minor comments:

          1. In each of the *.c files, please remove the *_clazz static variables. These are unused after your changes.
          2. TestBzip2CompressorDecompressor has a few small typos. Change "ather" to "after". Change "no less then original" to "no less than original".
          3. In all tests, I recommend not catching the Exception thrown from waitFor and just letting the exception propagate out of the test method. If there is ever a failure, we'll get more information that way, because the whole stack trace of the failure site will show up in the JUnit report.

          Thanks again. This is almost ready.

          Show
          cnauroth Chris Nauroth added a comment - Hui Zheng , this is looking good, and thank you for adding the new bzip2 test suite. I have just a few more minor comments: In each of the *.c files, please remove the *_clazz static variables. These are unused after your changes. TestBzip2CompressorDecompressor has a few small typos. Change "ather" to "after". Change "no less then original" to "no less than original". In all tests, I recommend not catching the Exception thrown from waitFor and just letting the exception propagate out of the test method. If there is ever a failure, we'll get more information that way, because the whole stack trace of the failure site will show up in the JUnit report. Thanks again. This is almost ready.
          Hide
          huizane Hui Zheng added a comment -

          Thanks ~Chris Nauroth.
          I updated the patch as your comment.

          Show
          huizane Hui Zheng added a comment - Thanks ~Chris Nauroth. I updated the patch as your comment.
          Hide
          cnauroth Chris Nauroth added a comment -

          +1 for the patch, pending Jenkins.

          Show
          cnauroth Chris Nauroth added a comment - +1 for the patch, pending Jenkins.
          Hide
          hadoopqa Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12704086/HADOOP-10027.5.patch
          against trunk revision 85f6d67.

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 4 new or modified test files.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 core tests. The patch failed these unit tests in hadoop-common-project/hadoop-common:

          org.apache.hadoop.ipc.TestRPCWaitForProxy

          Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/5922//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/5922//console

          This message is automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12704086/HADOOP-10027.5.patch against trunk revision 85f6d67. +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 4 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 core tests . The patch failed these unit tests in hadoop-common-project/hadoop-common: org.apache.hadoop.ipc.TestRPCWaitForProxy Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/5922//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/5922//console This message is automatically generated.
          Hide
          cnauroth Chris Nauroth added a comment -

          I'm retargeting this to 2.8.0, since we're now being more conservative about commits to branch-2.7 in preparation for the release.

          Show
          cnauroth Chris Nauroth added a comment - I'm retargeting this to 2.8.0, since we're now being more conservative about commits to branch-2.7 in preparation for the release.
          Hide
          cnauroth Chris Nauroth added a comment -

          I committed this to trunk and branch-2. Hui, thank you for the patch. Colin, thank you for helping with the code review.

          Show
          cnauroth Chris Nauroth added a comment - I committed this to trunk and branch-2. Hui, thank you for the patch. Colin, thank you for helping with the code review.
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-trunk-Commit #7308 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7308/)
          HADOOP-10027. *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd)

          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c
          • hadoop-common-project/hadoop-common/CHANGES.txt
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-trunk-Commit #7308 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7308/ ) HADOOP-10027 . *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd) hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c hadoop-common-project/hadoop-common/CHANGES.txt hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Yarn-trunk #864 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/864/)
          HADOOP-10027. *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd)

          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c
          • hadoop-common-project/hadoop-common/CHANGES.txt
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk #864 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/864/ ) HADOOP-10027 . *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd) hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c hadoop-common-project/hadoop-common/CHANGES.txt hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #130 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/130/)
          HADOOP-10027. *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd)

          • hadoop-common-project/hadoop-common/CHANGES.txt
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #130 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/130/ ) HADOOP-10027 . *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd) hadoop-common-project/hadoop-common/CHANGES.txt hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Hdfs-trunk #2062 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2062/)
          HADOOP-10027. *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd)

          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c
          • hadoop-common-project/hadoop-common/CHANGES.txt
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk #2062 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2062/ ) HADOOP-10027 . *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd) hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c hadoop-common-project/hadoop-common/CHANGES.txt hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #121 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/121/)
          HADOOP-10027. *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd)

          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java
          • hadoop-common-project/hadoop-common/CHANGES.txt
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #121 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/121/ ) HADOOP-10027 . *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd) hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java hadoop-common-project/hadoop-common/CHANGES.txt hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #130 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/130/)
          HADOOP-10027. *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd)

          • hadoop-common-project/hadoop-common/CHANGES.txt
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #130 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/130/ ) HADOOP-10027 . *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd) hadoop-common-project/hadoop-common/CHANGES.txt hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk #2080 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2080/)
          HADOOP-10027. *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd)

          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c
          • hadoop-common-project/hadoop-common/CHANGES.txt
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java
          • hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c
          • hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk #2080 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2080/ ) HADOOP-10027 . *Compressor_deflateBytesDirect passes instance instead of jclass to GetStaticObjectField. Contributed by Hui Zheng. (cnauroth: rev ff83ae72318fe6c0f266a7bf08138bd2fdb51cbd) hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/snappy/TestSnappyCompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Compressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.java hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/zlib/TestZlibCompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibDecompressor.c hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/bzip2/TestBzip2CompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.c hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/zlib/ZlibCompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/lz4/Lz4Compressor.c hadoop-common-project/hadoop-common/CHANGES.txt hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyDecompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/bzip2/Bzip2Decompressor.c hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/snappy/SnappyCompressor.java hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/compress/snappy/SnappyCompressor.c hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/lz4/TestLz4CompressorDecompressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/bzip2/Bzip2Compressor.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/ZlibCompressor.java

            People

            • Assignee:
              huizane Hui Zheng
              Reporter:
              spackler Eric Abbott
            • Votes:
              0 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development