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... #2247
Conversation
import java.io.ObjectInput; | ||
import java.io.ObjectOutput; | ||
import java.io.OutputStream; | ||
import java.io.Serializable; | ||
|
||
/** | ||
* A dummy marshaller impl that uses object streams converted via XStream as current JBoss Marshalling implementation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't use XStream any longer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed the reference to xstream
I've only spotted two unused imports. Everything else looks fine to me. |
@@ -150,7 +152,10 @@ public void testRestrictionOnAddingToAsyncQueue() throws Exception { | |||
} | |||
|
|||
private TestObjectStreamMarshaller marshaller() { | |||
return testObjectStreamMarshaller; | |||
if (marshaller == null) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason this class is doing this through lazy initialization instead of a BeforeMethod/BeforeTest like the other classes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no reason but the fact that there was no @BeforeMethod annotated method present already. I've created one as obviously this approach raises questions for the readers of the class.
Pulling. |
…s not have a no-args constructor * an JBossMarshaller is built under the hood for each TestObjectStreamMarshaller instance * users need to call TestObjectStreamMarshaller.stop explicitly
Integrated, Thanks @mmarkus ! |
Sorry closed too early. |
Integrated this time :) |
https://issues.jboss.org/browse/ISPN-3498