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 BC6E911D83 for ; Thu, 15 May 2014 04:52:13 +0000 (UTC) Received: (qmail 437 invoked by uid 500); 10 May 2014 23:19:18 -0000 Delivered-To: apmail-cxf-dev-archive@cxf.apache.org Received: (qmail 81055 invoked by uid 500); 10 May 2014 23:09:53 -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 4770 invoked by uid 99); 10 May 2014 22:56:40 -0000 Received: from Unknown (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 10 May 2014 22:56:40 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of elakito@gmail.com designates 209.85.217.182 as permitted sender) Received: from [209.85.217.182] (HELO mail-lb0-f182.google.com) (209.85.217.182) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 07 May 2014 10:48:12 +0000 Received: by mail-lb0-f182.google.com with SMTP id q8so1068225lbi.13 for ; Wed, 07 May 2014 03:47:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=fZ2GCZZOcHoFIWIl3R9EPk7td036iRAZJUjQD2DnHgY=; b=Kyuccebmk20pDQY7VFrislhPVwpQ1RFlP9nPMAfJFHdUp7rV+DQfWlp2MGQL2L9m1W JmAVOeOqrcNuM3nXETJ/G8ixQerw7NcRyWWRaL+hCLUN3Utv19RvNLu18UVjABIIXAIz uJKn8xczy4V6yIfBBEsZjSbktJl8cZUM42MDERXzsDk5lIVpcdHrc2WdPxVSgUrxdTDt Kw7WP/X9gjN6ZCXGyAK+gEwQatknLFNMWx2uQalPBwJM4vFQKYFNPXF/F4L8Rl24tH/T u9ov1oQVm+6DM5Gt82X8JpaeJD9D5aetyDuv70HbQVIlgtY00A+sS9o5PQ0+sgfvFXqm z8fg== MIME-Version: 1.0 X-Received: by 10.112.186.98 with SMTP id fj2mr1829607lbc.54.1399459670749; Wed, 07 May 2014 03:47:50 -0700 (PDT) Received: by 10.114.5.225 with HTTP; Wed, 7 May 2014 03:47:50 -0700 (PDT) In-Reply-To: References: <8c5e7d45377a4d5d8941401ed9dd7bd6@git.apache.org> Date: Wed, 7 May 2014 12:47:50 +0200 Message-ID: Subject: Re: git commit: Reorder some of hte calls to get the createSequence stuff occuring immediately to fix the tests From: Aki Yoshida To: dev@cxf.apache.org Cc: Dennis Sosnoski Content-Type: text/plain; charset=UTF-8 X-Virus-Checked: Checked by ClamAV on apache.org 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); >>