[MRUNIT-70] copy(orig, conf) in Serialization shouldn't require objects to have a no-args constructor, and copy(orig, copy, conf) seems to violate contract for deserializer.deserialize()

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.8.0
• Fix Version/s: 0.9.0
• Assignee: Jim Donofrio
• Reporter: Meloss Xeloss
• Created: 28/Feb/12 15:53
• Updated: 07/Aug/12 06:55
• Resolved: 10/Mar/12 05:22

Description

The copy(orig, conf) method requires objects to have a no-args constructor, which is non-ideal, since it passes on to Deserializer.deserialize, which can create a new object if passed in null. Additionally, the contract for deserialize only can fill in the object passed in, so the copy method will not perform as expected if it does not.

http://hadoop.apache.org/common/docs/current/api/org/apache/hadoop/io/serializer/Deserializer.html#deserialize%28T%29