Le 3/16/12 3:38 AM, Selcuk AYA a écrit :
> On Thu, Mar 15, 2012 at 7:11 AM, Emmanuel Lécharny<elecharny@gmail.com>  wrote:
>> Hi,
>> I continued to review the serialization of the DataChangeContainer
>> structure, and found that we call writeObject() a couple of times :
>> - in the ParentIdAndRdn class to serialize the UUID. It's easy to replace
>> this call
>> - in the IndexChange, for keys which are not String, byte[], Long. I added
>> two cases : UUID and ParentIdAndRdn so that we can now use the
>> read/writeExternal methods instead
>>
>> I also have a modifciation related to the Enums : I find the way Selcuk is
>> dealing with them way better than mine (ie, use the ordinal() method instead
>> of inject values the way I did). So I reverted my changes to stick with what
>> Selcuk did
>>
>> Those changes are purely internal and have no impact on the way the server
>> works. Tests are passing.
>>
>> Here is the proposed patch : http://pastebin.com/UDRF3qPa
> looks good, Would be good to add some test for deserializing these
> objects as we dont have code to excersize them.

The patch has been applied, a test has been added (which demonstrated 
some issues in the deserialization of DataChangeContainer, which needed 
some fixes).

Thanks !

-- 
Regards,
Cordialement,
Emmanuel Lécharny
www.iktek.com