Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.22.0
    • Fix Version/s: 0.21.0
    • Component/s: build
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      compile-fault-inject:
      [echo] Start weaving aspects in place
      [iajc] error at fc = FileContext.getFileContext(cluster.getFileSystem());
      [iajc] ^^^^^^^^^

      TestRename needs to be changed to work with changes from Hadoop-6223.

      1. hdfs-750.patch
        0.5 kB
        Suresh Srinivas

        Activity

        Hide
        Suresh Srinivas added a comment -

        New patch fixes the build and also verified the TestRename runs successfully.

        Show
        Suresh Srinivas added a comment - New patch fixes the build and also verified the TestRename runs successfully.
        Hide
        gary murry added a comment -

        I tested this patch local and it worked fine. It would be good to get this commited soon so that the build gets up broken.

        Show
        gary murry added a comment - I tested this patch local and it worked fine. It would be good to get this commited soon so that the build gets up broken.
        Hide
        Konstantin Shvachko added a comment -

        This fixes the complie error, but build is still failing with

        BUILD FAILED
        hdfs/build.xml:335: hdfs/build-fi/compile-fi.log doesn't exist
        
        Show
        Konstantin Shvachko added a comment - This fixes the complie error, but build is still failing with BUILD FAILED hdfs/build.xml:335: hdfs/build-fi/compile-fi.log doesn't exist
        Hide
        Suresh Srinivas added a comment -

        Can you try "ant injectfaults", looks like that is the target to trigger compilation of fault injection, which sets up all the build dependencies.

        Show
        Suresh Srinivas added a comment - Can you try "ant injectfaults", looks like that is the target to trigger compilation of fault injection, which sets up all the build dependencies.
        Hide
        Konstantin Shvachko added a comment -

        +1
        May be we should add a comment that "compile-fault-inject" target should not be called directly.

        Show
        Konstantin Shvachko added a comment - +1 May be we should add a comment that "compile-fault-inject" target should not be called directly.
        Hide
        Suresh Srinivas added a comment -

        Cannot run test-patch due to build failure. TestRename unit test passed.

        Show
        Suresh Srinivas added a comment - Cannot run test-patch due to build failure. TestRename unit test passed.
        Hide
        Suresh Srinivas added a comment -

        Committed the patch to trunk.

        Show
        Suresh Srinivas added a comment - Committed the patch to trunk.
        Hide
        Konstantin Boudnik added a comment -

        May be we should add a comment that "compile-fault-inject" target should not be called directly.

        "compile-fault-inject" shouldn't be called directly - it isn't a top-level target. Actually, to prevent people from running it directly we can rename it to "-compile-fault-inject". How about this?

        Show
        Konstantin Boudnik added a comment - May be we should add a comment that "compile-fault-inject" target should not be called directly. "compile-fault-inject" shouldn't be called directly - it isn't a top-level target. Actually, to prevent people from running it directly we can rename it to "-compile-fault-inject". How about this?
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Hdfs-trunk-Commit #96 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/96/)
        . Fix build failure due to TestRename. Contributed by Suresh Srinivas.

        Show
        Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #96 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/96/ ) . Fix build failure due to TestRename. Contributed by Suresh Srinivas.
        Hide
        Suresh Srinivas added a comment -

        Adding a comment to compile-fault-inject makes sense. Though I think this is a low priority issue. I will following up on this.

        Show
        Suresh Srinivas added a comment - Adding a comment to compile-fault-inject makes sense. Though I think this is a low priority issue. I will following up on this.
        Hide
        Hudson added a comment -

        Integrated in Hdfs-Patch-h5.grid.sp2.yahoo.net #100 (See http://hudson.zones.apache.org/hudson/job/Hdfs-Patch-h5.grid.sp2.yahoo.net/100/)

        Show
        Hudson added a comment - Integrated in Hdfs-Patch-h5.grid.sp2.yahoo.net #100 (See http://hudson.zones.apache.org/hudson/job/Hdfs-Patch-h5.grid.sp2.yahoo.net/100/ )
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Hdfs-trunk #136 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk/136/)

        Show
        Hudson added a comment - Integrated in Hadoop-Hdfs-trunk #136 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk/136/ )
        Hide
        Hudson added a comment -

        Integrated in Hdfs-Patch-h2.grid.sp2.yahoo.net #72 (See http://hudson.zones.apache.org/hudson/job/Hdfs-Patch-h2.grid.sp2.yahoo.net/72/)

        Show
        Hudson added a comment - Integrated in Hdfs-Patch-h2.grid.sp2.yahoo.net #72 (See http://hudson.zones.apache.org/hudson/job/Hdfs-Patch-h2.grid.sp2.yahoo.net/72/ )

          People

          • Assignee:
            Suresh Srinivas
            Reporter:
            Suresh Srinivas
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development