javaee / metro-wsit

https://javaee.github.io/metro-wsit/
Other
9 stars 24 forks source link

StreamMessage has been already consumed #938

Closed glassfishrobot closed 14 years ago

glassfishrobot commented 15 years ago

With all RM request-response and from-java-built one-way cases as well following error appears with build from http://kohsuke.sfbay/hudson/job/wsit-javanet/3220/artifact/dist/metro.zip (June 18, after fix to #919 one-way errors) java.lang.AssertionError: StreamMessage has been already consumed. See the nested exception for where it's consumed at com.sun.xml.ws.message.stream.StreamMessage.unconsumed(StreamMessage.java:477) at com.sun.xml.ws.message.stream.StreamMessage.consume(StreamMessage.java:270) at com.sun.xml.ws.rm.runtime.PacketAdapter.consume(PacketAdapter.java:166) at com.sun.xml.ws.rm.runtime.Rm10ClientSession.terminateOutboundSequence(Rm10ClientSession.java:187) at com.sun.xml.ws.rm.runtime.ClientSession.close(ClientSession.java:209) at com.sun.xml.ws.rm.runtime.ClientRmTube.preDestroy(ClientRmTube.java:180) at com.sun.xml.ws.api.pipe.helper.AbstractFilterTubeImpl.preDestroy(AbstractFilterTubeImpl.java:93) at com.sun.xml.ws.api.pipe.helper.AbstractFilterTubeImpl.preDestroy(AbstractFilterTubeImpl.java:93) at com.sun.xml.ws.api.pipe.helper.AbstractFilterTubeImpl.preDestroy(AbstractFilterTubeImpl.java:93) at com.sun.xml.ws.api.pipe.helper.AbstractFilterTubeImpl.preDestroy(AbstractFilterTubeImpl.java:93) at com.sun.xml.ws.api.pipe.helper.AbstractFilterTubeImpl.preDestroy(AbstractFilterTubeImpl.java:93) at com.sun.xml.ws.client.Stub.close(Stub.java:312) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) at java.lang.reflect.Method.invoke(Method.java:585) at com.sun.xml.ws.client.sei.SEIStub.invoke(SEIStub.java:122) at $Proxy37.close(Unknown Source) at tango2tango.add605soap2rsd2krequestresponse.client.ReliableRoundtripTest.sendEchoCore(ReliableRoundtripTest.java:184) at tango2tango.add605soap2rsd2krequestresponse.client.ReliableRoundtripTest.sendEcho(ReliableRoundtripTest.java:229) at tango2tango.add605soap2rsd2krequestresponse.client.ReliableRoundtripTest.testSendEcho1(ReliableRoundtripTest.java:97) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) at java.lang.reflect.Method.invoke(Method.java:585) at junit.framework.TestCase.runTest(TestCase.java:154) at org.netbeans.junit.NbTestCase.runBare(NbTestCase.java:136) at junit.framework.TestResult$1.protect(TestResult.java:106) at junit.framework.TestResult.runProtected(TestResult.java:124) at junit.framework.TestResult.run(TestResult.java:109) at junit.framework.TestCase.run(TestCase.java:118) at org.netbeans.junit.NbTestCase.run(NbTestCase.java:122) at junit.framework.TestSuite.runTest(TestSuite.java:208) at junit.framework.TestSuite.run(TestSuite.java:203) at org.netbeans.xtest.testrunner.JUnitTestRunner.runTests(JUnitTestRunner.java:167) at org.netbeans.xtest.testrunner.JUnitTestRunner.runTests(JUnitTestRunner.java:118) at org.netbeans.xtest.plugin.jvm.JUnitTestRunnerLauncher.main(JUnitTestRunnerLauncher.java:41) Caused by: java.lang.Exception at com.sun.xml.ws.message.stream.StreamMessage.unconsumed(StreamMessage.java:481) at com.sun.xml.ws.message.stream.StreamMessage.readPayloadAsJAXB(StreamMessage.java:242) at com.sun.xml.ws.rm.runtime.PacketAdapter.unmarshallMessage(PacketAdapter.java:430) at com.sun.xml.ws.rm.runtime.Rm10ClientSession.terminateOutboundSequence(Rm10ClientSession.java:174) ... 33 more

Environment

Operating System: All Platform: All URL: http://beetle.czech.sun.com/~dr199838/results/938-test/

Affected Versions

[1.3]

glassfishrobot commented 15 years ago

Reported by daveronge@java.net

glassfishrobot commented 15 years ago

daveronge@java.net said: Added URL to test results including logs.

glassfishrobot commented 15 years ago

m_potociar@java.net said: Reassigning to the jaxws runtime:

The exception is raised in a StreamMessage as a result of (probably multiple) calls to Message.consume() method. However, in the method definition and javadoc there is no warning that it is illegal to call the method multiple times:

/**

More, I didn't find any method via which I would be able to tell, whether a message was already consumed or not (something like Message.isConsumed()). Jitu, could you clarify this?

glassfishrobot commented 15 years ago

jitu@java.net said: I will clarify the semantics of consume(). But what's the need for RM to call consume() method ?

glassfishrobot commented 15 years ago

m_potociar@java.net said: Created an attachment (id=616) Patch that should fix the issue (+ few others)

glassfishrobot commented 15 years ago

m_potociar@java.net said: Hello Jitu, I have created a patch for you. Please review it and check it in.

Thanks, Marek

glassfishrobot commented 15 years ago

jitu@java.net said: I think we shouldn't touch readXXX methods now. But just fix that consume() can be called multiple times. Does that work for you ?

glassfishrobot commented 15 years ago

jitu@java.net said: Pretty risky for 1.3. Targeting for 1.4

glassfishrobot commented 15 years ago

jitu@java.net said: will target for 1.5

glassfishrobot commented 14 years ago

jitu@java.net said: Reducing the priority to P3 as the intention was to not to call consume() multiple times.

glassfishrobot commented 14 years ago

jitu@java.net said: Closing this one as there is a JAX-WS issue: https://jax-ws.dev.java.net/issues/show_bug.cgi?id=590

glassfishrobot commented 15 years ago

File: StreamMessage.patch Attached By: m_potociar@java.net

glassfishrobot commented 15 years ago

Was assigned to jitu@java.net

glassfishrobot commented 7 years ago

This issue was imported from java.net JIRA WSIT-938

glassfishrobot commented 14 years ago

Marked as won't fix on Wednesday, October 7th 2009, 9:55:14 am