Hadoop Map/Reduce
  1. Hadoop Map/Reduce
  2. MAPREDUCE-1125

SerialUtils.cc: deserializeFloat is out of sync with SerialUtils.hh

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.21.0
    • Fix Version/s: 3.0.0-alpha1
    • Component/s: pipes
    • Labels:
    • Tags:
      utils

      Description

      *** SerialUtils.hh ***
        float deserializeFloat(InStream& stream);
      
      *** SerialUtils.cc ***
        void deserializeFloat(float& t, InStream& stream)
        {
          char buf[sizeof(float)];
          stream.read(buf, sizeof(float));
          XDR xdrs;
          xdrmem_create(&xdrs, buf, sizeof(float), XDR_DECODE);
          xdr_float(&xdrs, &t);
        }
      
      1. MAPREDUCE-1125-3.patch
        4 kB
        Simone Leo
      2. MAPREDUCE-1125-2.patch
        4 kB
        Simone Leo
      3. MAPREDUCE-1125.004.patch
        1 kB
        Masatake Iwasaki

        Activity

        Hide
        Simone Leo added a comment -

        Attaching proposed patch

        Show
        Simone Leo added a comment - Attaching proposed patch
        Hide
        Greg Roelofs added a comment -

        Is this still an issue? I'm not familiar with the C++ code in question--where does it live?

        The patch looks fine as far as it goes (other than the lack of path info in the filenames), but I'm kind of amazed it ever compiled in the first place. Is SerialUtils.hh not actually included in SerialUtils.cc? Does Hudson even look at this stuff, wherever it lives?

        Show
        Greg Roelofs added a comment - Is this still an issue? I'm not familiar with the C++ code in question--where does it live? The patch looks fine as far as it goes (other than the lack of path info in the filenames), but I'm kind of amazed it ever compiled in the first place. Is SerialUtils.hh not actually included in SerialUtils.cc? Does Hudson even look at this stuff, wherever it lives?
        Hide
        Simone Leo added a comment -

        Patch for current trunk@1004113

        Show
        Simone Leo added a comment - Patch for current trunk@1004113
        Hide
        Simone Leo added a comment -

        Attaching patch for current trunk

        Show
        Simone Leo added a comment - Attaching patch for current trunk
        Hide
        Simone Leo added a comment -

        As suggested on the mapreduce-dev mailing list, I'm attaching results from my local test-patch run:

        [exec] +1 overall.
        [exec]
        [exec] +1 @author. The patch does not contain any @author tags.
        [exec]
        [exec] +1 tests included. The patch appears to include 7 new or modified tests.
        [exec]
        [exec] +1 javadoc. The javadoc tool did not generate any warning messages.
        [exec]
        [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings.
        [exec]
        [exec] +1 findbugs. The patch does not introduce any new Findbugs warnings.
        [exec]
        [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings.
        [exec]
        [exec] +1 system tests framework. The patch passed system tests framework compile.

        Show
        Simone Leo added a comment - As suggested on the mapreduce-dev mailing list, I'm attaching results from my local test-patch run: [exec] +1 overall. [exec] [exec] +1 @author. The patch does not contain any @author tags. [exec] [exec] +1 tests included. The patch appears to include 7 new or modified tests. [exec] [exec] +1 javadoc. The javadoc tool did not generate any warning messages. [exec] [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings. [exec] [exec] +1 findbugs. The patch does not introduce any new Findbugs warnings. [exec] [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings. [exec] [exec] +1 system tests framework. The patch passed system tests framework compile.
        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12456393/MAPREDUCE-1125.patch
        against trunk revision 1075216.

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

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

        -1 patch. The patch command could not apply the patch.

        Console output: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/92//console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12456393/MAPREDUCE-1125.patch against trunk revision 1075216. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 7 new or modified tests. -1 patch. The patch command could not apply the patch. Console output: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/92//console This message is automatically generated.
        Hide
        Tom White added a comment -

        This has fallen out of date. Can you sync with trunk then hopefully a committer familiar with pipes will review and commit. Thanks.

        Show
        Tom White added a comment - This has fallen out of date. Can you sync with trunk then hopefully a committer familiar with pipes will review and commit. Thanks.
        Hide
        Simone Leo added a comment -

        Updated for current trunk

        Show
        Simone Leo added a comment - Updated for current trunk
        Hide
        Simone Leo added a comment -

        Updated for current trunk

        Show
        Simone Leo added a comment - Updated for current trunk
        Hide
        Simone Leo added a comment -

        Attached version 2 of the patch – put in sync with current trunk

        Show
        Simone Leo added a comment - Attached version 2 of the patch – put in sync with current trunk
        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12472943/MAPREDUCE-1125-2.patch
        against trunk revision 1079072.

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

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

        +1 javadoc. The javadoc tool did not generate any warning messages.

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

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

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

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

        -1 contrib tests. The patch failed contrib unit tests.

        +1 system test framework. The patch passed system test framework compile.

        Test results: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/130//testReport/
        Findbugs warnings: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/130//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
        Console output: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/130//console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12472943/MAPREDUCE-1125-2.patch against trunk revision 1079072. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 5 new or modified tests. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. +1 core tests. The patch passed core unit tests. -1 contrib tests. The patch failed contrib unit tests. +1 system test framework. The patch passed system test framework compile. Test results: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/130//testReport/ Findbugs warnings: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/130//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/130//console This message is automatically generated.
        Hide
        Simone Leo added a comment -

        Looks like some tests failed due to timeout. Those tests have nothing to do with this patch, anyway.

        Show
        Simone Leo added a comment - Looks like some tests failed due to timeout. Those tests have nothing to do with this patch, anyway.
        Hide
        Todd Lipcon added a comment -

        This is a sort of incompatible change - some user might be explicitly declaring the extern void deserializeFloat(float &t, InStream &stream), right?

        Do we need to leave the old version around for a version? Perhaps someone who makes use of SerialUtils can comment - I never have worked on significant amounts of C++ code with Hadoop

        Show
        Todd Lipcon added a comment - This is a sort of incompatible change - some user might be explicitly declaring the extern void deserializeFloat(float &t, InStream &stream), right? Do we need to leave the old version around for a version? Perhaps someone who makes use of SerialUtils can comment - I never have worked on significant amounts of C++ code with Hadoop
        Hide
        Simone Leo added a comment -

        In Pydoop we are currently patching SerialUtils.cc at compile time, before building the Python extension with Boost.

        Show
        Simone Leo added a comment - In Pydoop we are currently patching SerialUtils.cc at compile time, before building the Python extension with Boost.
        Hide
        Arun C Murthy added a comment -

        Sorry to come in late, the patch has gone stale. Can you please rebase? Thanks.

        Show
        Arun C Murthy added a comment - Sorry to come in late, the patch has gone stale. Can you please rebase? Thanks.
        Hide
        Simone Leo added a comment -

        rebased for current trunk

        Show
        Simone Leo added a comment - rebased for current trunk
        Hide
        Hadoop QA added a comment -

        +1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12494006/MAPREDUCE-1125-3.patch
        against trunk revision .

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

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

        +1 javadoc. The javadoc tool did not generate any warning messages.

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

        +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) 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 .

        +1 contrib tests. The patch passed contrib unit tests.

        Test results: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//testReport/
        Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-mapreduce-client-hs.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-mapreduce-client-shuffle.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-mapreduce-client-common.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-mapreduce-client-app.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-mapreduce-client-jobclient.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-mapreduce-client-core.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-yarn-server-common.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-yarn-server-resourcemanager.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-yarn-server-nodemanager.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-yarn-common.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-yarn-api.html
        Console output: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - +1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12494006/MAPREDUCE-1125-3.patch against trunk revision . +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 5 new or modified tests. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) 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 . +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-mapreduce-client-hs.html Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-mapreduce-client-shuffle.html Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-mapreduce-client-common.html Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-mapreduce-client-app.html Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-mapreduce-client-jobclient.html Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-mapreduce-client-core.html Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-yarn-server-common.html Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-yarn-server-resourcemanager.html Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-yarn-server-nodemanager.html Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-yarn-common.html Findbugs warnings: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-yarn-api.html Console output: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/702//console This message is automatically generated.
        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12494006/MAPREDUCE-1125-3.patch
        against trunk revision d6e05c5.

        -1 patch. The patch command could not apply the patch.

        Console output: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5249//console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12494006/MAPREDUCE-1125-3.patch against trunk revision d6e05c5. -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5249//console This message is automatically generated.
        Hide
        Brahma Reddy Battula added a comment -

        Simone Leo can you please rebase the patch with latest trunk..?

        Show
        Brahma Reddy Battula added a comment - Simone Leo can you please rebase the patch with latest trunk..?
        Hide
        Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        -1 patch 0m 0s The patch command could not apply the patch during dryrun.



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12494006/MAPREDUCE-1125-3.patch
        Optional Tests javac unit
        git revision trunk / 6471d18
        Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5692/console

        This message was automatically generated.

        Show
        Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment -1 patch 0m 0s The patch command could not apply the patch during dryrun. Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12494006/MAPREDUCE-1125-3.patch Optional Tests javac unit git revision trunk / 6471d18 Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5692/console This message was automatically generated.
        Hide
        Masatake Iwasaki added a comment -

        I attached rebased patch. I left the version of old signature for backward compatibility. There is no test code for .cc in the current tree of hadoop-pipes

        Show
        Masatake Iwasaki added a comment - I attached rebased patch. I left the version of old signature for backward compatibility. There is no test code for .cc in the current tree of hadoop-pipes
        Hide
        Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        0 pre-patch 5m 31s Pre-patch trunk compilation is healthy.
        +1 @author 0m 0s The patch does not contain any @author tags.
        -1 tests included 0m 0s 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 7m 47s There were no new javac warning messages.
        +1 release audit 0m 20s The applied patch does not increase the total number of release audit warnings.
        +1 whitespace 0m 0s The patch has no lines that end in whitespace.
        +1 install 1m 36s mvn install still works.
        +1 eclipse:eclipse 0m 31s The patch built with eclipse:eclipse.
            15m 48s  



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12741423/MAPREDUCE-1125.004.patch
        Optional Tests javac unit
        git revision trunk / 49dfad9
        Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5833/testReport/
        Java 1.7.0_55
        uname Linux asf903.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
        Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5833/console

        This message was automatically generated.

        Show
        Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 5m 31s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s 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 7m 47s There were no new javac warning messages. +1 release audit 0m 20s The applied patch does not increase the total number of release audit warnings. +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 install 1m 36s mvn install still works. +1 eclipse:eclipse 0m 31s The patch built with eclipse:eclipse.     15m 48s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12741423/MAPREDUCE-1125.004.patch Optional Tests javac unit git revision trunk / 49dfad9 Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5833/testReport/ Java 1.7.0_55 uname Linux asf903.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5833/console This message was automatically generated.
        Hide
        Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 5s docker + precommit patch detected.
        +1 @author 0m 0s The patch does not contain any @author tags.
        -1 test4tests 0m 0s 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 mvninstall 2m 50s trunk passed
        +1 compile 0m 16s trunk passed with JDK v1.8.0_60
        +1 compile 0m 18s trunk passed with JDK v1.7.0_79
        +1 mvneclipse 0m 9s trunk passed
        +1 mvninstall 0m 9s the patch passed
        +1 compile 0m 17s the patch passed with JDK v1.8.0_60
        +1 cc 0m 17s the patch passed
        +1 javac 0m 17s the patch passed
        +1 compile 0m 18s the patch passed with JDK v1.7.0_79
        +1 cc 0m 18s the patch passed
        +1 javac 0m 18s the patch passed
        +1 mvneclipse 0m 10s the patch passed
        +1 whitespace 0m 0s Patch has no whitespace issues.
        +1 unit 0m 17s hadoop-pipes in the patch passed with JDK v1.8.0_60.
        +1 unit 0m 17s hadoop-pipes in the patch passed with JDK v1.7.0_79.
        +1 asflicense 0m 21s Patch does not generate ASF License warnings.
        5m 31s



        Subsystem Report/Notes
        Docker Client=1.7.1 Server=1.7.1 Image:test-patch-base-hadoop-date2015-11-06
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12741423/MAPREDUCE-1125.004.patch
        JIRA Issue MAPREDUCE-1125
        Optional Tests asflicense cc unit javac compile
        uname Linux 4773eb2d375b 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /home/jenkins/jenkins-slave/workspace/PreCommit-MAPREDUCE-Build/patchprocess/apache-yetus-ee5baeb/precommit/personality/hadoop.sh
        git revision trunk / 286cc64
        Default Java 1.7.0_79
        Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_60 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_79
        JDK v1.7.0_79 Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/6121/testReport/
        modules C: hadoop-tools/hadoop-pipes U: hadoop-tools/hadoop-pipes
        Max memory used 228MB
        Powered by Apache Yetus http://yetus.apache.org
        Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/6121/console

        This message was automatically generated.

        Show
        Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 5s docker + precommit patch detected. +1 @author 0m 0s The patch does not contain any @author tags. -1 test4tests 0m 0s 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 mvninstall 2m 50s trunk passed +1 compile 0m 16s trunk passed with JDK v1.8.0_60 +1 compile 0m 18s trunk passed with JDK v1.7.0_79 +1 mvneclipse 0m 9s trunk passed +1 mvninstall 0m 9s the patch passed +1 compile 0m 17s the patch passed with JDK v1.8.0_60 +1 cc 0m 17s the patch passed +1 javac 0m 17s the patch passed +1 compile 0m 18s the patch passed with JDK v1.7.0_79 +1 cc 0m 18s the patch passed +1 javac 0m 18s the patch passed +1 mvneclipse 0m 10s the patch passed +1 whitespace 0m 0s Patch has no whitespace issues. +1 unit 0m 17s hadoop-pipes in the patch passed with JDK v1.8.0_60. +1 unit 0m 17s hadoop-pipes in the patch passed with JDK v1.7.0_79. +1 asflicense 0m 21s Patch does not generate ASF License warnings. 5m 31s Subsystem Report/Notes Docker Client=1.7.1 Server=1.7.1 Image:test-patch-base-hadoop-date2015-11-06 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12741423/MAPREDUCE-1125.004.patch JIRA Issue MAPREDUCE-1125 Optional Tests asflicense cc unit javac compile uname Linux 4773eb2d375b 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /home/jenkins/jenkins-slave/workspace/PreCommit-MAPREDUCE-Build/patchprocess/apache-yetus-ee5baeb/precommit/personality/hadoop.sh git revision trunk / 286cc64 Default Java 1.7.0_79 Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_60 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_79 JDK v1.7.0_79 Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/6121/testReport/ modules C: hadoop-tools/hadoop-pipes U: hadoop-tools/hadoop-pipes Max memory used 228MB Powered by Apache Yetus http://yetus.apache.org Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/6121/console This message was automatically generated.
        Hide
        Allen Wittenauer added a comment -

        +1 committed

        Thanks!

        Show
        Allen Wittenauer added a comment - +1 committed Thanks!
        Hide
        Hudson added a comment -

        FAILURE: Integrated in Hadoop-trunk-Commit #8764 (See https://builds.apache.org/job/Hadoop-trunk-Commit/8764/)
        MAPREDUCE-1125. SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2)

        • hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc
        • hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh
        • hadoop-mapreduce-project/CHANGES.txt
        Show
        Hudson added a comment - FAILURE: Integrated in Hadoop-trunk-Commit #8764 (See https://builds.apache.org/job/Hadoop-trunk-Commit/8764/ ) MAPREDUCE-1125 . SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2) hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh hadoop-mapreduce-project/CHANGES.txt
        Hide
        Hudson added a comment -

        FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #634 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/634/)
        MAPREDUCE-1125. SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2)

        • hadoop-mapreduce-project/CHANGES.txt
        • hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh
        • hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc
        Show
        Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #634 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/634/ ) MAPREDUCE-1125 . SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2) hadoop-mapreduce-project/CHANGES.txt hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc
        Hide
        Hudson added a comment -

        SUCCESS: Integrated in Hadoop-Yarn-trunk-Java8 #644 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/644/)
        MAPREDUCE-1125. SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2)

        • hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh
        • hadoop-mapreduce-project/CHANGES.txt
        • hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc
        Show
        Hudson added a comment - SUCCESS: Integrated in Hadoop-Yarn-trunk-Java8 #644 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/644/ ) MAPREDUCE-1125 . SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2) hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh hadoop-mapreduce-project/CHANGES.txt hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc
        Hide
        Hudson added a comment -

        SUCCESS: Integrated in Hadoop-Yarn-trunk #1368 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/1368/)
        MAPREDUCE-1125. SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2)

        • hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc
        • hadoop-mapreduce-project/CHANGES.txt
        • hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh
        Show
        Hudson added a comment - SUCCESS: Integrated in Hadoop-Yarn-trunk #1368 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/1368/ ) MAPREDUCE-1125 . SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2) hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc hadoop-mapreduce-project/CHANGES.txt hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh
        Hide
        Hudson added a comment -

        FAILURE: Integrated in Hadoop-Mapreduce-trunk #2574 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2574/)
        MAPREDUCE-1125. SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2)

        • hadoop-mapreduce-project/CHANGES.txt
        • hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh
        • hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc
        Show
        Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk #2574 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2574/ ) MAPREDUCE-1125 . SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2) hadoop-mapreduce-project/CHANGES.txt hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc
        Hide
        Hudson added a comment -

        FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #576 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/576/)
        MAPREDUCE-1125. SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2)

        • hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh
        • hadoop-mapreduce-project/CHANGES.txt
        • hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc
        Show
        Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #576 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/576/ ) MAPREDUCE-1125 . SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2) hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh hadoop-mapreduce-project/CHANGES.txt hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc
        Hide
        Hudson added a comment -

        FAILURE: Integrated in Hadoop-Hdfs-trunk #2515 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2515/)
        MAPREDUCE-1125. SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2)

        • hadoop-mapreduce-project/CHANGES.txt
        • hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc
        • hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh
        Show
        Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk #2515 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2515/ ) MAPREDUCE-1125 . SerialUtils.cc: deserializeFloat is out of sync with (aw: rev 19a0c2660cce0dccf968376bed249de94b58b8a2) hadoop-mapreduce-project/CHANGES.txt hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh

          People

          • Assignee:
            Simone Leo
            Reporter:
            Simone Leo
          • Votes:
            0 Vote for this issue
            Watchers:
            10 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development