Review Board 1.7.22


HADOOP-7328. Improve the SerializationFactory functions.

Review Request #884 - Created June 11, 2011 and submitted

Harsh J
trunk
HADOOP-7328
Reviewers
hadoop-common
tlipcon
hadoop-common-git
Since getSerialization() can possibly return a null, it is only right that getSerializer() and getDeserializer() usage functions do the same, instead of throwing up NPEs.

Related issue to which this improvement is required: https://issues.apache.org/jira/browse/MAPREDUCE-2584
Existing SequenceFile serialization factory tests pass. The change is merely to make the functions return null instead of throwing an NPE within.
Review request changed
Updated (June 16, 2011, 12:13 p.m.)
Throw exceptions (getting rid of nulls). Add appropriate javadocs and fix one checkstyle nit.