Return-Path: X-Original-To: apmail-cxf-dev-archive@www.apache.org Delivered-To: apmail-cxf-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 825AE11F54 for ; Thu, 15 May 2014 00:56:53 +0000 (UTC) Received: (qmail 67594 invoked by uid 500); 10 May 2014 23:24:38 -0000 Delivered-To: apmail-cxf-dev-archive@cxf.apache.org Received: (qmail 25010 invoked by uid 500); 10 May 2014 23:05:26 -0000 Mailing-List: contact dev-help@cxf.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cxf.apache.org Delivered-To: mailing list dev@cxf.apache.org Received: (qmail 38167 invoked by uid 99); 10 May 2014 22:58:48 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 10 May 2014 22:58:48 +0000 X-ASF-Spam-Status: No, hits=-0.0 required=5.0 tests=SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: local policy) Received: from [72.249.186.197] (HELO sosnoski.com) (72.249.186.197) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 08 May 2014 22:12:45 +0000 Received: from linux-a5lu.site (ip-58-28-158-218.static-xdsl.xnet.co.nz [58.28.158.218]) by sosnoski.com (Postfix) with ESMTPSA id B68DB1008D; Thu, 8 May 2014 22:12:23 +0000 (UTC) Message-ID: <536BFF73.2020106@sosnoski.com> Date: Fri, 09 May 2014 10:04:35 +1200 From: Dennis Sosnoski User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 MIME-Version: 1.0 To: Aki Yoshida , dev@cxf.apache.org, asoldano@redhat.com Subject: Re: git commit: Reorder some of hte calls to get the createSequence stuff occuring immediately to fix the tests References: <8c5e7d45377a4d5d8941401ed9dd7bd6@git.apache.org> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org I reverted the change yesterday, so let me know if you're still seeing any problems. I don't know why I didn't see the issues with the integration tests, which led to Dan changing the order, before I checked in the original change. Normally I run the RM integration tests before every commit. Sorry for the slip-up. Please copy me directly on any issues and emails for now - I'm not receiving emails from the CXF lists at present. - Dennis On 05/07/2014 10:47 PM, Aki Yoshida wrote: > Hi, > Alessio identified the cause of this problem to be CXF-5726 and I just > commented on that ticket about this issue. > regards, aki > > 2014-05-06 20:01 GMT+02:00 Aki Yoshida : >> Hi Dennis, >> I was wondering about the code change that caused the test error that >> lead to this test code change. >> >> I am seeing the following exception in one of my local old scenarios >> that invokes oneway and request-responser operations in that order. >> This has been working for 2.7.x and also working with >> 3.0.0-milerstonres2. >> >> However, I noticed that it is failing with 3.0.0-SNAPSHOT with the >> following exception. And this seems to be the same error. >> >> org.apache.cxf.interceptor.Fault: It is not possible to send a create >> sequence request to the anonymous address >> http://www.w3.org/2005/08/addressing/anonymous >> at org.apache.cxf.ws.rm.AbstractRMInterceptor.handleMessage(AbstractRMInterceptor.java:104) >> at org.apache.cxf.phase.PhaseInterceptorChain.doIntercept(PhaseInterceptorChain.java:307) >> at org.apache.cxf.interceptor.OutgoingChainInterceptor.handleMessage(OutgoingChainInterceptor.java:81) >> at org.apache.cxf.phase.PhaseInterceptorChain.doIntercept(PhaseInterceptorChain.java:307) >> at org.apache.cxf.transport.ChainInitiationObserver.onMessage(ChainInitiationObserver.java:121) >> at org.apache.cxf.transport.http.AbstractHTTPDestination.invoke(AbstractHTTPDestination.java:243) >> at org.apache.cxf.transport.http_jetty.JettyHTTPDestination.doService(JettyHTTPDestination.java:259) >> at org.apache.cxf.transport.http_jetty.JettyHTTPHandler.handle(JettyHTTPHandler.java:65) >> at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1088) >> at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1024) >> at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:135) >> at org.eclipse.jetty.server.handler.ContextHandlerCollection.handle(ContextHandlerCollection.java:255) >> at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:116) >> at org.eclipse.jetty.server.Server.handle(Server.java:370) >> at org.eclipse.jetty.server.AbstractHttpConnection.handleRequest(AbstractHttpConnection.java:494) >> at org.eclipse.jetty.server.AbstractHttpConnection.content(AbstractHttpConnection.java:982) >> at org.eclipse.jetty.server.AbstractHttpConnection$RequestHandler.content(AbstractHttpConnection.java:1043) >> at org.eclipse.jetty.http.HttpParser.parseNext(HttpParser.java:865) >> at org.eclipse.jetty.http.HttpParser.parseAvailable(HttpParser.java:240) >> at org.eclipse.jetty.server.AsyncHttpConnection.handle(AsyncHttpConnection.java:82) >> at org.eclipse.jetty.io.nio.SelectChannelEndPoint.handle(SelectChannelEndPoint.java:696) >> at org.eclipse.jetty.io.nio.SelectChannelEndPoint$1.run(SelectChannelEndPoint.java:53) >> at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:608) >> at org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:543) >> at java.lang.Thread.run(Thread.java:695) >> Caused by: org.apache.cxf.ws.rm.RMException: It is not possible to >> send a create sequence request to the anonymous address >> http://www.w3.org/2005/08/addressing/anonymous >> at org.apache.cxf.ws.rm.RMManager.getSequence(RMManager.java:458) >> at org.apache.cxf.ws.rm.RMCaptureOutInterceptor.handle(RMCaptureOutInterceptor.java:156) >> at org.apache.cxf.ws.rm.AbstractRMInterceptor.handleMessage(AbstractRMInterceptor.java:83) >> ... 24 more >> >> I think there is some regression issue in the code. >> >> >> regards, aki >> >> 2014-05-02 20:29 GMT+02:00 : >>> Repository: cxf >>> Updated Branches: >>> refs/heads/master 126775733 -> a4c85510a >>> >>> >>> Reorder some of hte calls to get the createSequence stuff occuring immediately to fix the tests >>> >>> >>> Project: http://git-wip-us.apache.org/repos/asf/cxf/repo >>> Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/a4c85510 >>> Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/a4c85510 >>> Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/a4c85510 >>> >>> Branch: refs/heads/master >>> Commit: a4c85510a14854349d877510d3688c47457e6b3b >>> Parents: 1267757 >>> Author: Daniel Kulp >>> Authored: Fri May 2 14:14:55 2014 -0400 >>> Committer: Daniel Kulp >>> Committed: Fri May 2 14:14:55 2014 -0400 >>> >>> ---------------------------------------------------------------------- >>> .../cxf/systest/ws/policy/RM10PolicyWsdlTest.java | 11 ++++++----- >>> .../cxf/systest/ws/policy/RM12PolicyWsdlTest.java | 13 +++++++------ >>> .../org/apache/cxf/systest/ws/policy/RMPolicyTest.java | 13 +++++++------ >>> 3 files changed, 20 insertions(+), 17 deletions(-) >>> ---------------------------------------------------------------------- >>> >>> >>> http://git-wip-us.apache.org/repos/asf/cxf/blob/a4c85510/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM10PolicyWsdlTest.java >>> ---------------------------------------------------------------------- >>> diff --git a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM10PolicyWsdlTest.java b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM10PolicyWsdlTest.java >>> index 3bb79e8..b19536f 100644 >>> --- a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM10PolicyWsdlTest.java >>> +++ b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM10PolicyWsdlTest.java >>> @@ -82,13 +82,14 @@ public class RM10PolicyWsdlTest extends RMPolicyWsdlTestBase { >>> >>> ConnectionHelper.setKeepAliveConnection(greeter, true); >>> >>> - // oneway >>> - >>> - greeter.greetMeOneWay("CXF"); >>> >>> // two-way >>> >>> assertEquals("CXF", greeter.greetMe("cxf")); >>> + >>> + // oneway >>> + >>> + greeter.greetMeOneWay("CXF"); >>> >>> // exception >>> >>> @@ -118,14 +119,14 @@ public class RM10PolicyWsdlTest extends RMPolicyWsdlTestBase { >>> >>> mf.verifyMessages(5, true); >>> String[] expectedActions = new String[] {RM10Constants.INSTANCE.getCreateSequenceAction(), >>> - GREETMEONEWAY_ACTION, >>> GREETME_ACTION, >>> + GREETMEONEWAY_ACTION, >>> PINGME_ACTION, >>> PINGME_ACTION}; >>> mf.verifyActions(expectedActions, true); >>> mf.verifyMessageNumbers(new String[] {null, "1", "2", "3", "4"}, true); >>> mf.verifyLastMessage(new boolean[] {false, false, false, false, false}, true); >>> - mf.verifyAcknowledgements(new boolean[] {false, false, false, true, true}, true); >>> + mf.verifyAcknowledgements(new boolean[] {false, false, true, false, true}, true); >>> >>> mf.verifyMessages(4, false); >>> // mf.verifyMessages(9, false); >>> >>> http://git-wip-us.apache.org/repos/asf/cxf/blob/a4c85510/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM12PolicyWsdlTest.java >>> ---------------------------------------------------------------------- >>> diff --git a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM12PolicyWsdlTest.java b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM12PolicyWsdlTest.java >>> index e31d86f..e1b9191 100644 >>> --- a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM12PolicyWsdlTest.java >>> +++ b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM12PolicyWsdlTest.java >>> @@ -81,14 +81,15 @@ public class RM12PolicyWsdlTest extends RMPolicyWsdlTestBase { >>> ConnectionHelper.setKeepAliveConnection(greeter, true); >>> LOG.fine("Created greeter client."); >>> >>> - // oneway >>> - >>> - greeter.greetMeOneWay("CXF"); >>> >>> // two-way >>> >>> assertEquals("CXF", greeter.greetMe("cxf")); >>> - >>> + >>> + // oneway >>> + >>> + greeter.greetMeOneWay("CXF"); >>> + >>> // exception >>> >>> try { >>> @@ -117,14 +118,14 @@ public class RM12PolicyWsdlTest extends RMPolicyWsdlTestBase { >>> >>> mf.verifyMessages(5, true); >>> String[] expectedActions = new String[] {RM11Constants.INSTANCE.getCreateSequenceAction(), >>> - GREETMEONEWAY_ACTION, >>> GREETME_ACTION, >>> + GREETMEONEWAY_ACTION, >>> PINGME_ACTION, >>> PINGME_ACTION}; >>> mf.verifyActions(expectedActions, true); >>> mf.verifyMessageNumbers(new String[] {null, "1", "2", "3", "4"}, true); >>> mf.verifyLastMessage(new boolean[] {false, false, false, false, false}, true); >>> - mf.verifyAcknowledgements(new boolean[] {false, false, false, true, true}, true); >>> + mf.verifyAcknowledgements(new boolean[] {false, false, true, false, true}, true); >>> >>> mf.verifyMessages(4, false); >>> // mf.verifyMessages(9, false); >>> >>> http://git-wip-us.apache.org/repos/asf/cxf/blob/a4c85510/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RMPolicyTest.java >>> ---------------------------------------------------------------------- >>> diff --git a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RMPolicyTest.java b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RMPolicyTest.java >>> index 4507d96..0f18399 100644 >>> --- a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RMPolicyTest.java >>> +++ b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RMPolicyTest.java >>> @@ -145,14 +145,15 @@ public class RMPolicyTest extends AbstractBusClientServerTestBase { >>> LOG.fine("Created greeter client."); >>> >>> ConnectionHelper.setKeepAliveConnection(greeter, true); >>> - // oneway >>> - >>> - greeter.greetMeOneWay("CXF"); >>> >>> // two-way >>> >>> assertEquals("CXF", greeter.greetMe("cxf")); >>> - >>> + >>> + // oneway >>> + >>> + greeter.greetMeOneWay("CXF"); >>> + >>> // exception >>> >>> try { >>> @@ -178,14 +179,14 @@ public class RMPolicyTest extends AbstractBusClientServerTestBase { >>> >>> mf.verifyMessages(5, true); >>> String[] expectedActions = new String[] {RM10Constants.INSTANCE.getCreateSequenceAction(), >>> - GREETMEONEWAY_ACTION, >>> GREETME_ACTION, >>> + GREETMEONEWAY_ACTION, >>> PINGME_ACTION, >>> PINGME_ACTION}; >>> mf.verifyActions(expectedActions, true); >>> mf.verifyMessageNumbers(new String[] {null, "1", "2", "3", "4"}, true); >>> mf.verifyLastMessage(new boolean[] {false, false, false, false, false}, true); >>> - mf.verifyAcknowledgements(new boolean[] {false, false, false, true, true}, true); >>> + mf.verifyAcknowledgements(new boolean[] {false, false, true, false, true}, true); >>> >>> mf.verifyMessages(4, false); >>> // mf.verifyMessages(9, false); >>>