Details
-
Sub-task
-
Status: Resolved
-
Major
-
Resolution: Fixed
-
None
-
None
-
None
Description
This is from andrew.wang commented in HADOOP-13200:
Other questions/comments from looking at this code:
We also should rename RSRawEncoderLegacy to RSLegacyRawEncoder and RSRawErasureCoderFactoryLegacy to RSLegacyErasureCoderFactory, to match the naming of other subclasses.
TestRawEncoderBase, should this use a configured factory to get the raw coders, rather than referencing the raw coders directly? I didn't check other usages, but it seems like we should be creating via the appropriate factory whenever possible.