Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kryo gets ClassCastException when deserializing TreeSet with Comparator #139

Closed
ghost opened this issue Nov 11, 2013 · 4 comments
Closed

Comments

@ghost
Copy link

ghost commented Nov 11, 2013

From [email protected] on October 08, 2013 13:13:07

What steps will reproduce the problem?
serialize this after uncommenting the 2 lines mentioning Kryo.

  1. Kryo refuses to serialize inner class if not static
  2. Kryo gets ClassCast Exception when deserializing TreeSet

    http://code.google.com/p/fast-serialization/source/browse/trunk/src/test/java/de/ruedigermoeller/serialization/testclasses/basicstuff/CommonCollections.java

    What version of the Kryo are you using?
    2.2.2

Original issue: http://code.google.com/p/kryo/issues/detail?id=139

@ghost
Copy link
Author

ghost commented Nov 11, 2013

From martin.grotzke on October 09, 2013 02:41:27

  1. Kryo refuses to serialize inner class if not static

Use "kryo.setInstantiatorStrategy(new StdInstantiatorStrategy());" as a fix.

  1. Kryo gets ClassCast Exception when deserializing TreeSet

AFAICS we need a TreeSetSerializer for this.

Summary: Kryo gets ClassCastException when deserializing TreeSet with Comparator (was: 2 bugs with this testcase)
Status: Accepted
Owner: martin.grotzke

@ghost
Copy link
Author

ghost commented Nov 11, 2013

From martin.grotzke on October 11, 2013 02:56:33

Fixed in trunk.

@moru0011: Do you need this to be released soon (to maven central)?

Status: Fixed
Labels: Milestone-2.23

@ghost
Copy link
Author

ghost commented Nov 11, 2013

From [email protected] on October 11, 2013 03:09:53

No, as said I use kryo for benchmarking only, i just wanted to help improving.

@ghost
Copy link
Author

ghost commented Nov 11, 2013

From martin.grotzke on October 11, 2013 03:16:06

Alright, thanx for reporting!

@ghost ghost closed this as completed Nov 11, 2013
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

0 participants