Details

      Description

      After merging the protobuf-based-fsimage code from trunk, we need to fix some compilation errors.

        Activity

        Hide
        Jing Zhao added a comment -

        The patch can fix the compilation error. We still need to fix TestOfflineEditsViewer. But I guess we can also do that in the end in case that we make further changes to editlog.

        Show
        Jing Zhao added a comment - The patch can fix the compilation error. We still need to fix TestOfflineEditsViewer. But I guess we can also do that in the end in case that we make further changes to editlog.
        Hide
        Haohui Mai added a comment -

        The v0 patch merges the protobuf code into branch-2. It also includes fixes for TestOfflineImageViewer.

        Show
        Haohui Mai added a comment - The v0 patch merges the protobuf code into branch-2. It also includes fixes for TestOfflineImageViewer .
        Hide
        Jing Zhao added a comment -

        I guess the above comment should be posted in HDFS-5698, Haohui Mai ?

        Show
        Jing Zhao added a comment - I guess the above comment should be posted in HDFS-5698 , Haohui Mai ?
        Hide
        Haohui Mai added a comment -

        Yes. Sorry about the confusion.

        Show
        Haohui Mai added a comment - Yes. Sorry about the confusion.
        Hide
        Arpit Agarwal added a comment -

        +1 for the patch. I committed it to branch HDFS-5535. Thanks for taking care of this Jing!

        Show
        Arpit Agarwal added a comment - +1 for the patch. I committed it to branch HDFS-5535 . Thanks for taking care of this Jing!

          People

          • Assignee:
            Jing Zhao
            Reporter:
            Jing Zhao
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development