Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.0-alpha1
    • Fix Version/s: 3.0.0-alpha1
    • Component/s: None
    • Labels:
      None

      Description

      This will refactor raw erasure coders according to some comments received so far.

      • As discussed in HADOOP-11540 and suggested by Colin McCabe, better not to rely class inheritance to reuse the codes, instead they can be moved to some utility.
      • Suggested by Jing Zhao somewhere quite some time ago, better to have a state holder to keep some checking results for later reuse during an encode/decode call.

      This would not get rid of some inheritance levels as doing so isn't clear yet for the moment and also incurs big impact. I do wish the end result by this refactoring will make all the levels more clear and easier to follow.

        Attachments

        1. HADOOP-13010-v1.patch
          57 kB
          Kai Zheng
        2. HADOOP-13010-v2.patch
          57 kB
          Kai Zheng
        3. HADOOP-13010-v3.patch
          103 kB
          Kai Zheng
        4. HADOOP-13010-v4.patch
          115 kB
          Rui Li
        5. HADOOP-13010-v5.patch
          120 kB
          Kai Zheng
        6. HADOOP-13010-v6.patch
          150 kB
          Kai Zheng
        7. HADOOP-13010-v7.patch
          150 kB
          Kai Zheng

          Issue Links

            Activity

              People

              • Assignee:
                drankye Kai Zheng
                Reporter:
                drankye Kai Zheng
              • Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: