Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-3634

Add self-contained, mavenized fuse_dfs test

    Details

    • Type: Test Test
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.0-alpha
    • Fix Version/s: 2.0.2-alpha
    • Component/s: fuse-dfs
    • Labels:
      None
    • Target Version/s:
    • Hadoop Flags:
      Reviewed

      Description

      We should have a self-contained, mavenized FUSE unit test which runs as part of the normal build and can detect problems. Of course, because FUSE is an optional build component, the unit test won't run unless the user has FUSE installed. However, it would be very useful in improving the quality of fuse_dfs and detecting regressions.

      1. HDFS-3634.006.patch
        32 kB
        Colin Patrick McCabe
      2. HDFS-3634.005.patch
        32 kB
        Colin Patrick McCabe
      3. HDFS-3634.004.patch
        32 kB
        Colin Patrick McCabe
      4. HDFS-3634.003.patch
        34 kB
        Colin Patrick McCabe
      5. HDFS-3634.002.patch
        30 kB
        Colin Patrick McCabe

        Issue Links

          Activity

          Colin Patrick McCabe created issue -
          Colin Patrick McCabe made changes -
          Field Original Value New Value
          Link This issue is blocked by HDFS-3606 [ HDFS-3606 ]
          Colin Patrick McCabe made changes -
          Link This issue relates to HDFS-3227 [ HDFS-3227 ]
          Colin Patrick McCabe made changes -
          Attachment HDFS-3634.002.patch [ 12538100 ]
          Colin Patrick McCabe made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12538100/HDFS-3634.002.patch
          against trunk revision .

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

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

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

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

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

          +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 failed these unit tests in hadoop-hdfs-project/hadoop-hdfs:

          org.apache.hadoop.hdfs.server.datanode.TestDirectoryScanner

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

          Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/2917//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/2917//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/12538100/HDFS-3634.002.patch against trunk revision . +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 3 new or modified test files. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 javadoc. The javadoc tool did not generate any warning messages. +1 eclipse:eclipse. The patch built with eclipse:eclipse. +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 failed these unit tests in hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.server.datanode.TestDirectoryScanner +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/2917//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/2917//console This message is automatically generated.
          Hide
          Colin Patrick McCabe added a comment -
          • be more careful to close all file descriptors on error – to avoid hangs
          • wait until fuse_dfs finishes mounting the filesystem before starting the test. (Add waitForMount).
          • put sleepNoSig into a common utility function. Since we use alarm(), we can't use sleep().
          Show
          Colin Patrick McCabe added a comment - be more careful to close all file descriptors on error – to avoid hangs work around HDFS-2551 and add comment wait until fuse_dfs finishes mounting the filesystem before starting the test. (Add waitForMount). put sleepNoSig into a common utility function. Since we use alarm(), we can't use sleep().
          Colin Patrick McCabe made changes -
          Attachment HDFS-3634.003.patch [ 12538469 ]
          Colin Patrick McCabe made changes -
          Link This issue is related to HDFS-2551 [ HDFS-2551 ]
          Hide
          Colin Patrick McCabe added a comment -

          exposed by this test

          Show
          Colin Patrick McCabe added a comment - exposed by this test
          Hide
          Andy Isaacson added a comment -
          +static int expectDirs(const struct dirent *de, void *v)
          +{
          +  const char **names = (const char **)v;
          

          no need to cast a void* in C, just assign.

          + * @return                 0 on success; error code otherwise
          

          The function returns a negative errno, may as well document that.

          +    fprintf(stderr, "FUSE_TEST: failed to fork: error %d\n", ret);
          

          please print strerror(errno) as well. (Several instances of this pattern, IMO we should never print errno without also printing strerror.)

          +      c = ((src[i + 1] - '0') << 16) |
          +           ((src[i + 2] - '0') << 8) |
          +            (src[i + 3] - '0');
          

          That's not the right way to decode a 3-digit octal string, it yields 0x10503 given "0153". I think you would want <<2 and <<5 but given that we're discussing it, clearly this needs a standalone function and a unit test; I'd copy to a temporary array and use strtol(buf,8,&p) then check p.

          +  f = fopen("/proc/mounts", "r");
          ...
          +    line = fgets(buf, sizeof(buf), f);
          

          Please use getmntent(3) rather than rolling our own.

          +  snprintf(scratch, sizeof(scratch), "%s", argv0);
          

          strncpy is more idiomatic.

          +  char mntTmp[PATH_MAX] = { 0 };
          

          Initialize strings with strings, so this should be

          char mntTmp[PATH_MAX] = "";

          .

          +int recursiveDeleteContents(const char *path)
          

          Do we really need this (potentially dangerous) code in the testcase? I'd hate to see a bug result in an accidental rm -rf $HOME. (I've looked at the obvious cases and don't see any bugs, but that's small comfort.) The target hdfs will be deleted afterwards so no need to delete there; the local target is pretty small so leaving it around is no big deal. Some of the code seems to indicate that something will error out if you attempt to mount over a directory with contents, but that seems like just a bug?

          +    if ((de->d_name[0] == '.') && (de->d_name[1] == '\0'))
          +      continue;
          +    if ((de->d_name[0] == '.') && (de->d_name[1] == '.') &&
          +        (de->d_name[2] == '\0'))
          

          These would be more idiomatic as if (!strcmp(de->d_name, ".")), IMHO. But, your call.

          +    // canonicalize non-abosolute TMPDIR
          

          abosolute -> absolute

          Show
          Andy Isaacson added a comment - + static int expectDirs( const struct dirent *de, void *v) +{ + const char **names = ( const char **)v; no need to cast a void* in C, just assign. + * @ return 0 on success; error code otherwise The function returns a negative errno, may as well document that. + fprintf(stderr, "FUSE_TEST: failed to fork: error %d\n" , ret); please print strerror(errno) as well. (Several instances of this pattern, IMO we should never print errno without also printing strerror.) + c = ((src[i + 1] - '0') << 16) | + ((src[i + 2] - '0') << 8) | + (src[i + 3] - '0'); That's not the right way to decode a 3-digit octal string, it yields 0x10503 given "0153". I think you would want <<2 and <<5 but given that we're discussing it, clearly this needs a standalone function and a unit test; I'd copy to a temporary array and use strtol(buf,8,&p) then check p. + f = fopen( "/proc/mounts" , "r" ); ... + line = fgets(buf, sizeof(buf), f); Please use getmntent(3) rather than rolling our own. + snprintf(scratch, sizeof(scratch), "%s" , argv0); strncpy is more idiomatic. + char mntTmp[PATH_MAX] = { 0 }; Initialize strings with strings, so this should be char mntTmp[PATH_MAX] = ""; . + int recursiveDeleteContents( const char *path) Do we really need this (potentially dangerous) code in the testcase? I'd hate to see a bug result in an accidental rm -rf $HOME . (I've looked at the obvious cases and don't see any bugs, but that's small comfort.) The target hdfs will be deleted afterwards so no need to delete there; the local target is pretty small so leaving it around is no big deal. Some of the code seems to indicate that something will error out if you attempt to mount over a directory with contents, but that seems like just a bug? + if ((de->d_name[0] == '.') && (de->d_name[1] == '\0')) + continue ; + if ((de->d_name[0] == '.') && (de->d_name[1] == '.') && + (de->d_name[2] == '\0')) These would be more idiomatic as if (!strcmp(de->d_name, ".")) , IMHO. But, your call. + // canonicalize non-abosolute TMPDIR abosolute -> absolute
          Hide
          Andy Isaacson added a comment -
          +void sleepNoSig(int sec)
          +{
          +  struct timespec req, rem;
          +
          +  req.tv_sec = sec;
          +  req.tv_nsec = 0;
          +  memset(&rem, 0, sizeof(rem));
          +  nanosleep(&req, &rem);
          +}
          

          Is this supposed to resume the sleep if interrupted? If so we need a loop. If not, we can drop rem and just nanosleep(&req, 0);.

          Show
          Andy Isaacson added a comment - +void sleepNoSig( int sec) +{ + struct timespec req, rem; + + req.tv_sec = sec; + req.tv_nsec = 0; + memset(&rem, 0, sizeof(rem)); + nanosleep(&req, &rem); +} Is this supposed to resume the sleep if interrupted? If so we need a loop. If not, we can drop rem and just nanosleep(&req, 0); .
          Hide
          Colin Patrick McCabe added a comment -
          • fix a few style issues
          • use getmntent rather than reading from /proc/mounts directly. This also means we don't need the code to parse octal escapes.
          • don't call recursiveDeleteContents on mount point before mounting: instead, give the -ononempty option to FUSE.
          • sleepNoSig: sleep the full period even in the presence of signals
          Show
          Colin Patrick McCabe added a comment - fix a few style issues use getmntent rather than reading from /proc/mounts directly. This also means we don't need the code to parse octal escapes. don't call recursiveDeleteContents on mount point before mounting: instead, give the -ononempty option to FUSE. sleepNoSig: sleep the full period even in the presence of signals
          Colin Patrick McCabe made changes -
          Attachment HDFS-3634.004.patch [ 12539392 ]
          Hide
          Andy Isaacson added a comment -
          +    req.tv_sec = rem.tv_sec;
          +    req.tv_nsec = rem.tv_nsec;
          

          This can simply be written req = rem;.

          +  } while (rem.tv_sec || rem.tv_nsec);
          

          I don't see anywhere in the docs that say rem is zeroed on successful sleep, nor that it isn't modified on successful sleep. The docs say the return value will be 0 on successful sleep. So we should do something like

             do {
                req = rem;
                ret = nanosleep(&req, &rem);
             } while (ret == -1 && errno == EINTR);
             if (ret == -1) {
                 fprintf(stderr, "sleepNoSig: nanosleep: %s\n", strerror(errno));
             }
          
          Show
          Andy Isaacson added a comment - + req.tv_sec = rem.tv_sec; + req.tv_nsec = rem.tv_nsec; This can simply be written req = rem; . + } while (rem.tv_sec || rem.tv_nsec); I don't see anywhere in the docs that say rem is zeroed on successful sleep, nor that it isn't modified on successful sleep. The docs say the return value will be 0 on successful sleep. So we should do something like do { req = rem; ret = nanosleep(&req, &rem); } while (ret == -1 && errno == EINTR); if (ret == -1) { fprintf(stderr, "sleepNoSig: nanosleep: %s\n" , strerror(errno)); }
          Hide
          Hadoop QA added a comment -

          +1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12539392/HDFS-3634.004.patch
          against trunk revision .

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

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

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

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

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

          +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 hadoop-hdfs-project/hadoop-hdfs.

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

          Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/2960//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/2960//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/12539392/HDFS-3634.004.patch against trunk revision . +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 3 new or modified test files. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 javadoc. The javadoc tool did not generate any warning messages. +1 eclipse:eclipse. The patch built with eclipse:eclipse. +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 hadoop-hdfs-project/hadoop-hdfs. +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/2960//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/2960//console This message is automatically generated.
          Colin Patrick McCabe made changes -
          Link This issue is duplicated by HDFS-3250 [ HDFS-3250 ]
          Hide
          Colin Patrick McCabe added a comment -
          • improve sleepNoSig
          Show
          Colin Patrick McCabe added a comment - improve sleepNoSig
          Colin Patrick McCabe made changes -
          Attachment HDFS-3634.005.patch [ 12539704 ]
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12539704/HDFS-3634.005.patch
          against trunk revision .

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

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

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

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

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

          +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 failed these unit tests in hadoop-hdfs-project/hadoop-hdfs:

          org.apache.hadoop.hdfs.TestDFSClientRetries

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

          Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/2965//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/2965//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/12539704/HDFS-3634.005.patch against trunk revision . +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 3 new or modified test files. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 javadoc. The javadoc tool did not generate any warning messages. +1 eclipse:eclipse. The patch built with eclipse:eclipse. +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 failed these unit tests in hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.TestDFSClientRetries +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/2965//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/2965//console This message is automatically generated.
          Hide
          Aaron T. Myers added a comment -

          Looks like the patch has a few tab characters in it. Otherwise it looks good to me.

          Andy, does Colin's latest patch look good to you? If so, I'll go ahead and commit this patch.

          Show
          Aaron T. Myers added a comment - Looks like the patch has a few tab characters in it. Otherwise it looks good to me. Andy, does Colin's latest patch look good to you? If so, I'll go ahead and commit this patch.
          Hide
          Colin Patrick McCabe added a comment -
          • remove stray tabs
          Show
          Colin Patrick McCabe added a comment - remove stray tabs
          Colin Patrick McCabe made changes -
          Attachment HDFS-3634.006.patch [ 12539771 ]
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12539771/HDFS-3634.006.patch
          against trunk revision .

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

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

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

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

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

          +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 failed these unit tests in hadoop-hdfs-project/hadoop-hdfs:

          org.apache.hadoop.hdfs.TestFileConcurrentReader

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

          Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/2969//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/2969//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/12539771/HDFS-3634.006.patch against trunk revision . +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 3 new or modified test files. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 javadoc. The javadoc tool did not generate any warning messages. +1 eclipse:eclipse. The patch built with eclipse:eclipse. +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 failed these unit tests in hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.TestFileConcurrentReader +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/2969//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/2969//console This message is automatically generated.
          Hide
          Andy Isaacson added a comment -

          Thanks for addressing the feedback, Colin. The .006 patch looks good to me. I'm a bit concerned that we don't have any automated way to validate the interface, for example the JNI interface description strings, and I didn't have time to manually check them... but I suppose it'll blow up visibly if it's wrong, and this shouldn't break any existing code, so go ahead and check it in. We can address any issues in a followup jira.

          Show
          Andy Isaacson added a comment - Thanks for addressing the feedback, Colin. The .006 patch looks good to me. I'm a bit concerned that we don't have any automated way to validate the interface, for example the JNI interface description strings, and I didn't have time to manually check them... but I suppose it'll blow up visibly if it's wrong, and this shouldn't break any existing code, so go ahead and check it in. We can address any issues in a followup jira.
          Hide
          Aaron T. Myers added a comment -

          +1, the latest patch looks good to me as well. I'm going to commit this momentarily.

          Show
          Aaron T. Myers added a comment - +1, the latest patch looks good to me as well. I'm going to commit this momentarily.
          Hide
          Aaron T. Myers added a comment -

          I've just committed this to trunk and branch-2.

          Thanks a lot for the contribution, Colin, and thanks a lot to Andy for the very thorough reviews.

          Show
          Aaron T. Myers added a comment - I've just committed this to trunk and branch-2. Thanks a lot for the contribution, Colin, and thanks a lot to Andy for the very thorough reviews.
          Aaron T. Myers made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Hadoop Flags Reviewed [ 10343 ]
          Fix Version/s 2.2.0-alpha [ 12322472 ]
          Resolution Fixed [ 1 ]
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk-Commit #2631 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/2631/)
          HDFS-3634. Add self-contained, mavenized fuse_dfs test. Contributed by Colin Patrick McCabe. (Revision 1371231)

          Result = SUCCESS
          atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1371231
          Files :

          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/CMakeLists.txt
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/CMakeLists.txt
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/fuse_workload.c
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/fuse_workload.h
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/test_fuse_dfs.c
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/expect.h
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/native_mini_dfs.c
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/native_mini_dfs.h
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/util/posix_util.c
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/util/posix_util.h
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #2631 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/2631/ ) HDFS-3634 . Add self-contained, mavenized fuse_dfs test. Contributed by Colin Patrick McCabe. (Revision 1371231) Result = SUCCESS atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1371231 Files : /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/CMakeLists.txt /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/CMakeLists.txt /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/fuse_workload.c /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/fuse_workload.h /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/test_fuse_dfs.c /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/expect.h /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/native_mini_dfs.c /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/native_mini_dfs.h /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/util/posix_util.c /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/util/posix_util.h
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #2566 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/2566/)
          HDFS-3634. Add self-contained, mavenized fuse_dfs test. Contributed by Colin Patrick McCabe. (Revision 1371231)

          Result = SUCCESS
          atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1371231
          Files :

          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/CMakeLists.txt
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/CMakeLists.txt
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/fuse_workload.c
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/fuse_workload.h
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/test_fuse_dfs.c
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/expect.h
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/native_mini_dfs.c
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/native_mini_dfs.h
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/util/posix_util.c
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/util/posix_util.h
          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #2566 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/2566/ ) HDFS-3634 . Add self-contained, mavenized fuse_dfs test. Contributed by Colin Patrick McCabe. (Revision 1371231) Result = SUCCESS atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1371231 Files : /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/CMakeLists.txt /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/CMakeLists.txt /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/fuse_workload.c /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/fuse_workload.h /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/test_fuse_dfs.c /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/expect.h /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/native_mini_dfs.c /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/native_mini_dfs.h /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/util/posix_util.c /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/util/posix_util.h
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk-Commit #2586 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/2586/)
          HDFS-3634. Add self-contained, mavenized fuse_dfs test. Contributed by Colin Patrick McCabe. (Revision 1371231)

          Result = FAILURE
          atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1371231
          Files :

          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/CMakeLists.txt
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/CMakeLists.txt
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/fuse_workload.c
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/fuse_workload.h
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/test_fuse_dfs.c
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/expect.h
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/native_mini_dfs.c
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/native_mini_dfs.h
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/util/posix_util.c
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/util/posix_util.h
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk-Commit #2586 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/2586/ ) HDFS-3634 . Add self-contained, mavenized fuse_dfs test. Contributed by Colin Patrick McCabe. (Revision 1371231) Result = FAILURE atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1371231 Files : /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/CMakeLists.txt /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/CMakeLists.txt /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/fuse_workload.c /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/fuse_workload.h /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/fuse-dfs/test/test_fuse_dfs.c /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/expect.h /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/native_mini_dfs.c /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/libhdfs/native_mini_dfs.h /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/util/posix_util.c /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/native/util/posix_util.h
          Arun C Murthy made changes -
          Affects Version/s 2.0.0-alpha [ 12320353 ]
          Affects Version/s 2.1.0-alpha [ 12321440 ]
          Arun C Murthy made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          15d 23h 19m 1 Colin Patrick McCabe 27/Jul/12 01:32
          Patch Available Patch Available Resolved Resolved
          13d 14h 13m 1 Aaron T. Myers 09/Aug/12 15:45
          Resolved Resolved Closed Closed
          63d 3h 1 Arun C Murthy 11/Oct/12 18:46

            People

            • Assignee:
              Colin Patrick McCabe
              Reporter:
              Colin Patrick McCabe
            • Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development