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

ISPN-3498 Cannot construct org.infinispan.util.KeyValuePair as it does n... #2088

Closed
wants to merge 1 commit into from

Conversation

mmarkus
Copy link
Contributor

@mmarkus mmarkus commented Sep 20, 2013

@pruivo
Copy link
Member

pruivo commented Sep 23, 2013

@mgencur @mmarkus is this ok to be integrated?

@mgencur
Copy link
Contributor

mgencur commented Sep 23, 2013

Not sure. IMO, this will solve the problem only partially. There will still be tests that will be failing, only a few though.

@mmarkus
Copy link
Contributor Author

mmarkus commented Sep 24, 2013

@mgencur xstream is used in tests as an "alternative" marshaller implementation, and not in production. Funny enough it doesn't have this no-arg requirement for other vms.
I've added an no-arg ctor to InternalMetadataImpl as well, if there's another class having this problem ping me.

@mgencur
Copy link
Contributor

mgencur commented Sep 25, 2013

OK. I think this PR can be integrated. Now I don't think there are any other classes that need this fix.

@pruivo
Copy link
Member

pruivo commented Sep 25, 2013

@mgencur @mmarkus pulling...

@pruivo
Copy link
Member

pruivo commented Sep 25, 2013

Integrated!
Thanks!

@pruivo pruivo closed this Sep 25, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants