support serialization of scala sortedmap

[koertkuipers]

No description provided.

[koertkuipers]

i would also like to add the requirement in KryoSpec:
orig.getClass.asInstanceOf[Class[Any]] must be_==(serdeser.getClass.asInstanceOf[Class[Any]])

it is currently true for all but one:
HashMap("good" -> 0.5, "bad" -> -1.0) starts life as a scala.collection.immutable.HashMap$HashTrieMap but comes out of the serialization and deserialization as a scala.collection.immutable.Map$Map2

[johnynek]

I agree with the constraint that .getClass is an equivalence. I think the way to go here would be to make sure a TrieMap serializer is being used (and to create such a thing).

Also, we should really add more scala-check: randomly generate object graphs and make sure they (and their class identity) round-trips.

[koertkuipers]

we already register HashMap.

i believe this line is causing the problem:
.forConcreteTraversableClass(Map[Any, Any]('a -> 'a, 'b -> 'b, 'c -> 'c, 'd
-> 'd, 'e -> 'e))

i debugged and that is the serializer/deserializer that gets used... which
should not happen. is it because of this:

scala> Map[Any, Any]('a -> 'a, 'b -> 'b, 'c -> 'c, 'd -> 'd).getClass
res17: java.lang.Class[_ <: scala.collection.immutable.Map[Any,Any]] =
class scala.collection.immutable.Map$Map4

scala> Map[Any, Any]('a -> 'a, 'b -> 'b, 'c -> 'c, 'd -> 'd, 'e ->
'e).getClass
res18: java.lang.Class[_ <: scala.collection.immutable.Map[Any,Any]] =
class scala.collection.immutable.HashMap$HashTrieMap

On Sun, Nov 10, 2013 at 5:45 PM, P. Oscar Boykin
notifications@github.comwrote:

I agree with the constraint that .getClass is an equivalence. I think the
way to go here would be to make sure a TrieMap serializer is being used
(and to create such a thing).

Also, we should really add more scala-check: randomly generate object
graphs and make sure they (and their class identity) round-trips.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/162#issuecomment-28163221
.