Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.0.0-alpha1
    • 3.0.0-alpha1
    • None
    • None

    Description

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

      • As discussed in HADOOP-11540 and suggested by cmccabe, better not to rely class inheritance to reuse the codes, instead they can be moved to some utility.
      • Suggested by jingzhao 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

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

              Dates

                Created:
                Updated:
                Resolved: