[ 
https://issues.apache.org/jira/browse/IGNITE-3172?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15438617#comment-15438617
 ] 

Igor Rudyak commented on IGNITE-3172:
-------------------------------------

Alexey,

In general you are right regarding tests for KryoSerializer - just replace 
JavaSerializer with KryoSerializer in the tests and that's it, but there is one 
issue. The problem with KryoSerializer is that *ignite-cassandra-serializers* 
project doesn't have dependency on *ignite-cassandra-store* project. Because of 
this I can't implement valuable tests for KryoSerializer inside 
*ignite-cassandra-serializers* project, can just implement rather generic test 
to serialize/deserialize some object to/from byte stream. If you have any ideas 
on this - just let me know.

As for *IllegalStateException* I just decided that it's better reflects the 
situation when it failed to serialize object - either something wrong with the 
object state or something wrong with the stream used for 
serialization/deserialization. If you see any problems with this - please 
explain your concerns.

Good point about *U.closeQuite* just completely forgot about it.



> Ignite-Cassandra serializers
> ----------------------------
>
>                 Key: IGNITE-3172
>                 URL: https://issues.apache.org/jira/browse/IGNITE-3172
>             Project: Ignite
>          Issue Type: Improvement
>          Components: cache
>            Reporter: Igor Rudyak
>            Assignee: Igor Rudyak
>
> Ignite-Cassandra module should contain only "standard" serializer based on 
> Java serialization mechanism. 
> It should be a separate module in Ignite project for all other alternative 
> implementations of serializers (based on Kryo, providing compression, 
> encryption and etc.)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to