[
https://issues.apache.org/jira/browse/MRUNIT-70?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jim Donofrio resolved MRUNIT-70.
--------------------------------
Resolution: Fixed
Fix Version/s: 1.0.0
Assignee: Jim Donofrio
added javadoc on Serialization class methods
change copy(orig, copy, conf) to allow null for copy which means to create a
new object
change copy(orig, conf) to not use ReflectionUtils but to pass in null to
copy(orig, copy, conf)
acknowledge in javadoc that depending on the serialization class, copy may or
may not copy orig into copy
return the result of deserialize since depending on serialization class, copy
may or may not copy orig into copy
committed in 1299150
> 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()
> ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: MRUNIT-70
> URL: https://issues.apache.org/jira/browse/MRUNIT-70
> Project: MRUnit
> Issue Type: Bug
> Affects Versions: 0.8.0
> Reporter: Meloss Xeloss
> Assignee: Jim Donofrio
> Priority: Minor
> Fix For: 1.0.0
>
> Attachments: MRUNIT-70-1.patch
>
>
> 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
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
