TUSCANY-4028 - don't duplicate intents on the Java implementation model
git-svn-id: http://svn.us.apache.org/repos/asf/tuscany@1301369 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
e96dd3904b
commit
64fa14787b
7 changed files with 82 additions and 6 deletions
|
@ -1106,7 +1106,7 @@ public class CompositeProcessor extends BaseAssemblyProcessor implements StAXArt
|
|||
//for this that is resolved, the specified intents and policysets are safe in the
|
||||
//component and not lost
|
||||
|
||||
List<PolicySet> policySets = new ArrayList<PolicySet>(implementation.getPolicySets());
|
||||
List<PolicySet> policySets = new ArrayList<PolicySet>(implementation.getPolicySets());
|
||||
List<Intent> intents = new ArrayList<Intent>(implementation.getRequiredIntents());
|
||||
implementation = resolveImplementation(implementation, resolver, context);
|
||||
|
||||
|
@ -1117,7 +1117,15 @@ public class CompositeProcessor extends BaseAssemblyProcessor implements StAXArt
|
|||
implementation.getPolicySets().addAll(policySets);
|
||||
}
|
||||
|
||||
implementation.getRequiredIntents().addAll(intents);
|
||||
//implementation.getRequiredIntents().addAll(intents);
|
||||
|
||||
// Make sure we don't repeat any intents that are already on the
|
||||
// resolved implementation
|
||||
for (Intent intent : intents){
|
||||
if (!implementation.getRequiredIntents().contains(intent)){
|
||||
implementation.getRequiredIntents().add(intent);
|
||||
}
|
||||
}
|
||||
|
||||
// resolve any policy on implementation operations
|
||||
for (Operation op : implementation.getOperations()){
|
||||
|
|
|
@ -20,11 +20,15 @@ package org.apache.tuscany.sca.test.performance.impl;
|
|||
|
||||
import org.apache.tuscany.sca.test.performance.BeanA;
|
||||
import org.apache.tuscany.sca.test.performance.Helloworld;
|
||||
import org.oasisopen.sca.annotation.Requires;
|
||||
|
||||
@Requires("{http://tuscany.apache.org/xmlns/sca/1.1}logging")
|
||||
public class HelloworldServiceImpl implements Helloworld {
|
||||
|
||||
public String sayHello(String name, BeanA beanA) {
|
||||
return "Hello " + name + " " + beanA.getField1();
|
||||
String returnStrng = "Hello " + name + " " + beanA.getField1();
|
||||
System.out.println("At HelloworldServiceImpl - " + returnStrng);
|
||||
return returnStrng;
|
||||
}
|
||||
|
||||
}
|
||||
|
|
|
@ -0,0 +1,16 @@
|
|||
# Licensed to the Apache Software Foundation (ASF) under one
|
||||
# or more contributor license agreements. See the NOTICE file
|
||||
# distributed with this work for additional information
|
||||
# regarding copyright ownership. The ASF licenses this file
|
||||
# to you under the Apache License, Version 2.0 (the
|
||||
# "License"); you may not use this file except in compliance
|
||||
# with the License. You may obtain a copy of the License at
|
||||
#
|
||||
# http://www.apache.org/licenses/LICENSE-2.0
|
||||
#
|
||||
# Unless required by applicable law or agreed to in writing,
|
||||
# software distributed under the License is distributed on an
|
||||
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
|
||||
# KIND, either express or implied. See the License for the
|
||||
# specific language governing permissions and limitations
|
||||
# under the License.
|
|
@ -0,0 +1,41 @@
|
|||
<?xml version="1.0" encoding="ASCII"?>
|
||||
<!--
|
||||
* Licensed to the Apache Software Foundation (ASF) under one
|
||||
* or more contributor license agreements. See the NOTICE file
|
||||
* distributed with this work for additional information
|
||||
* regarding copyright ownership. The ASF licenses this file
|
||||
* to you under the Apache License, Version 2.0 (the
|
||||
* "License"); you may not use this file except in compliance
|
||||
* with the License. You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing,
|
||||
* software distributed under the License is distributed on an
|
||||
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
|
||||
* KIND, either express or implied. See the License for the
|
||||
* specific language governing permissions and limitations
|
||||
* under the License.
|
||||
-->
|
||||
<definitions xmlns="http://docs.oasis-open.org/ns/opencsa/sca/200912"
|
||||
xmlns:sca="http://docs.oasis-open.org/ns/opencsa/sca/200912"
|
||||
targetNamespace="http://tuscany.apache.org/xmlns/sca/1.1"
|
||||
xmlns:tuscany="http://tuscany.apache.org/xmlns/sca/1.1">
|
||||
|
||||
<intent name="mediation" constrains="sca:binding"/>
|
||||
|
||||
<policySet name="Mediation1"
|
||||
provides="tuscany:mediation"
|
||||
appliesTo="//sca:service"
|
||||
attachTo="//sca:component[@name='BusinessLogicComponent']">
|
||||
<tuscany:mediation name="mediation1.med"/>
|
||||
</policySet>
|
||||
|
||||
<policySet name="Mediation2"
|
||||
provides="tuscany:mediation"
|
||||
appliesTo="//sca:reference"
|
||||
attachTo="//sca:component[@name='BusinessLogicComponent']">
|
||||
<tuscany:mediation name="mediation2.med"/>
|
||||
</policySet>
|
||||
|
||||
</definitions>
|
|
@ -20,6 +20,7 @@
|
|||
<composite xmlns="http://docs.oasis-open.org/ns/opencsa/sca/200912"
|
||||
xmlns:foo="http://foo"
|
||||
targetNamespace="http://foo"
|
||||
xmlns:tuscany="http://tuscany.apache.org/xmlns/sca/1.1"
|
||||
name="PerformanceTest">
|
||||
|
||||
<component name="HelloWorldClientComponent">
|
||||
|
@ -28,7 +29,7 @@
|
|||
</component>
|
||||
|
||||
<component name="HelloWorldServiceComponent">
|
||||
<implementation.java class="org.apache.tuscany.sca.test.performance.impl.HelloworldServiceImpl" />
|
||||
<implementation.java class="org.apache.tuscany.sca.test.performance.impl.HelloworldServiceImpl"/>
|
||||
<service name="Helloworld">
|
||||
<binding.ws/>
|
||||
</service>
|
||||
|
|
|
@ -46,6 +46,12 @@
|
|||
<artifactId>tuscany-host-jetty</artifactId>
|
||||
<version>2.0-SNAPSHOT</version>
|
||||
</dependency>
|
||||
|
||||
<dependency>
|
||||
<groupId>org.apache.tuscany.sca</groupId>
|
||||
<artifactId>tuscany-policy-logging</artifactId>
|
||||
<version>2.0-SNAPSHOT</version>
|
||||
</dependency>
|
||||
|
||||
</dependencies>
|
||||
</project>
|
||||
|
|
|
@ -224,11 +224,11 @@ public class PerformanceTestCase {
|
|||
System.out.println("You can watch a JVM run using \n" +
|
||||
" jconsole \n" +
|
||||
"You can manually dump the heap using \n" +
|
||||
" jmap -dump:file=heapdump.bin 345" +
|
||||
" jmap -dump:file=heap_stop_" + name + ".bin 345" +
|
||||
"Where 345 is the process id from jconsole \n" +
|
||||
"The program dumps the heap at the start and end. You can look at them using \n" +
|
||||
" jhat -J-Xmx512m heap_start.bin\n" +
|
||||
" jhat -J-Xmx512m heap_stop.bin\n" +
|
||||
" jhat -J-Xmx512m heap_stop_" + name + ".bin\n" +
|
||||
"Then point your browser at\n" +
|
||||
" http://localhost:7000/");
|
||||
} else {
|
||||
|
|
Loading…
Add table
Reference in a new issue