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 1321B11C7A for ; Wed, 14 May 2014 21:04:58 +0000 (UTC) Received: (qmail 48587 invoked by uid 500); 10 May 2014 23:15:05 -0000 Delivered-To: apmail-cxf-dev-archive@cxf.apache.org Received: (qmail 58333 invoked by uid 500); 10 May 2014 23:00:17 -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 82164 invoked by uid 99); 10 May 2014 22:55:13 -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:55:13 +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.173 as permitted sender) Received: from [209.85.217.173] (HELO mail-lb0-f173.google.com) (209.85.217.173) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 09 May 2014 09:19:30 +0000 Received: by mail-lb0-f173.google.com with SMTP id 10so5301702lbg.32 for ; Fri, 09 May 2014 02:19:09 -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=gFj4LZlz9aiQAjJwCBLx3XVPgzpPd5ttPmQ1JDeHEfY=; b=gj8LLqPTLMieIbeSqZDckkeCmvyHvtmlLpq65Eego0VYkFS1rUf2pIQaaTLX/4hCzM uxfpYmty7QgI4FkQ/S3TMwHs8Q5VqW+k6WERpqWTmIuBH6IH6KdYOFM4TBf+i5icsDhR AESEaohcYSeDkhpAZGV8XwfeIfC7zRQbCqSKwxnIRS0oefySqh7skpISA4gDjco4l69a 5GCkeuQtF1OaMTlWM+hajL4oh4AoSP1BpRD/Np2t8MlQ2NoQ5CLApb76yIbkmidFUZqz 6TBmUTuzqI/jUzHa1WbmGOQUPVh4rT1/aTlK16DQLlvYyXu5QEilAJjf68Zep0GTpp5q 3cWw== MIME-Version: 1.0 X-Received: by 10.152.87.71 with SMTP id v7mr1149277laz.10.1399627149229; Fri, 09 May 2014 02:19:09 -0700 (PDT) Received: by 10.114.5.225 with HTTP; Fri, 9 May 2014 02:19:09 -0700 (PDT) In-Reply-To: <536BFF73.2020106@sosnoski.com> References: <8c5e7d45377a4d5d8941401ed9dd7bd6@git.apache.org> <536BFF73.2020106@sosnoski.com> Date: Fri, 9 May 2014 11:19:09 +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: Dennis Sosnoski Cc: dev@cxf.apache.org, asoldano@redhat.com Content-Type: text/plain; charset=UTF-8 X-Virus-Checked: Checked by ClamAV on apache.org Hi Dennis, it's working fine now. Thanks. the infra guys have been working on the mail server problem. https://blogs.apache.org/infra/entry/mail_outage regards, aki 2014-05-09 0:04 GMT+02:00 Dennis Sosnoski : > 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); >>>> >