From 6d371a5668a6beb37ee261f65f808974403791c1 Mon Sep 17 00:00:00 2001 From: jsdelfino Date: Mon, 14 Jul 2008 17:20:51 +0000 Subject: Starting to fix algorithm driving the resolution of the composite associated with a node. Before it was trying to resolve it with all contributions, causing exceptions when multiple contributions were used and the composite could not be fully resolved in one of them. git-svn-id: http://svn.us.apache.org/repos/asf/tuscany@676652 13f79535-47bb-0310-9956-ffa450edef68 --- .../org/apache/tuscany/sca/node/impl/NodeImpl.java | 136 +++++++++++---------- .../org/apache/tuscany/sca/node/impl/NodeUtil.java | 29 +++++ 2 files changed, 98 insertions(+), 67 deletions(-) create mode 100644 java/sca/modules/node2-impl/src/main/java/org/apache/tuscany/sca/node/impl/NodeUtil.java (limited to 'java/sca') diff --git a/java/sca/modules/node2-impl/src/main/java/org/apache/tuscany/sca/node/impl/NodeImpl.java b/java/sca/modules/node2-impl/src/main/java/org/apache/tuscany/sca/node/impl/NodeImpl.java index eff0ca30ea..af4d531e01 100644 --- a/java/sca/modules/node2-impl/src/main/java/org/apache/tuscany/sca/node/impl/NodeImpl.java +++ b/java/sca/modules/node2-impl/src/main/java/org/apache/tuscany/sca/node/impl/NodeImpl.java @@ -102,39 +102,6 @@ public class NodeImpl implements SCANode2, SCAClient { // The composite loaded into this node private Composite composite; - /** - * Constructs a new SCA node. - * - * @param configuration the the node configuration information. - */ - NodeImpl(ConfiguredNodeImplementation configuration) { - configurationName = configuration.getURI(); - logger.log(Level.INFO, "Creating node: " + configuration.getURI()); - - try { - // Initialize the runtime - initRuntime(); - - URL configurationURL = new URL(configuration.getURI()); - - // Resolve contribution URLs - for (Contribution contribution : configuration.getContributions()) { - URL contributionURL = new URL(configurationURL, contribution.getLocation()); - contribution.setLocation(contributionURL.toString()); - } - - // Resolve composite URL - URL compositeURL = new URL(configurationURL, configuration.getComposite().getURI()); - configuration.getComposite().setURI(compositeURL.toString()); - - // Configure the node - configureNode(configuration); - - } catch (Exception e) { - throw new ServiceRuntimeException(e); - } - } - /** * Constructs a new SCA node. * @@ -391,8 +358,7 @@ public class NodeImpl implements SCANode2, SCAClient { configuration = findNodeConfiguration(compositeURI, null); } else { // Create a node configuration - NodeImplementationFactory nodeImplementationFactory = - modelFactories.getFactory(NodeImplementationFactory.class); + NodeImplementationFactory nodeImplementationFactory = modelFactories.getFactory(NodeImplementationFactory.class); configuration = nodeImplementationFactory.createConfiguredNodeImplementation(); // Read the composite model @@ -488,10 +454,11 @@ public class NodeImpl implements SCANode2, SCAClient { // Load the contribution logger.log(Level.INFO, "Loading contribution: " + contributionURL); contributions.add(contributionService.contribute(contribution.getURI(), contributionURL, false)); - analyseProblems(); + analyzeProblems(); } // Resolve the metadata within the context of the contribution + //FIXME This doesn't seem to make sense here if (metadata != null) { StAXArtifactProcessor processor = artifactProcessors.getProcessor(ContributionMetadata.class); @@ -505,40 +472,51 @@ public class NodeImpl implements SCANode2, SCAClient { configuration.setComposite(composites.get(0)); } - // Load the specified composite - StAXArtifactProcessor compositeProcessor = artifactProcessors.getProcessor(Composite.class); - if (configuration.getComposite().getName() == null) { - URI uri = URI.create(configuration.getComposite().getURI()); - if (uri.getScheme() == null) { - // For relative URI, try to resolve it within the contributions - Artifact file = resolveCompositeArtifact(contributions, uri.toString()); - if (file == null) { - throw new IllegalArgumentException("Composite is not found in contributions: " + uri); - } - uri = URI.create(file.getLocation()); + Contribution contribution; + URL compositeURL; + + URI uri = URI.create(configuration.getComposite().getURI()); + if (uri.getScheme() == null) { + + // If the composite URI is a relative URI, try to resolve it within the contributions + contribution = contribution(contributions, uri.toString()); + if (contribution == null) { + throw new IllegalArgumentException("Composite is not found in contributions: " + uri); } - URL compositeURL = uri.toURL(); + compositeURL = new URL(location(contribution, uri.toString())); + + } else { + + // If the composite URI is an absolute URI, use it as is + compositeURL = uri.toURL(); + + // And resolve the composite within the scope of the last contribution + if (contributions.size() != 0) { + contribution = contributions.get(contributions.size() -1); + } else { + contribution = null; + } + } + + // Read the composite + StAXArtifactProcessor compositeProcessor = artifactProcessors.getProcessor(Composite.class); + composite = configuration.getComposite(); + if (composite.getName() == null) { logger.log(Level.INFO, "Loading composite: " + compositeURL); InputStream is = compositeURL.openStream(); XMLStreamReader reader = inputFactory.createXMLStreamReader(is); composite = compositeProcessor.read(reader); reader.close(); - } else { - composite = configuration.getComposite(); - } - - analyseProblems(); - // Resolve it within the context of the contribution - for (Contribution c : contributions) { - compositeProcessor.resolve(composite, c.getModelResolver()); - if (!composite.isUnresolved()) { - break; - } + analyzeProblems(); } - analyseProblems(); + // Resolve the given composite within the scope of the selected contribution + if (contribution != null) { + compositeProcessor.resolve(composite, contribution.getModelResolver()); + analyzeProblems(); + } // Create a top level composite to host our composite // This is temporary to make the activator happy @@ -558,21 +536,45 @@ public class NodeImpl implements SCANode2, SCAClient { // Build the composite runtime.buildComposite(composite); - analyseProblems(); + analyzeProblems(); } /** - * Search all contributions to resolve the relative composite URI - * @param contributions + * Returns the artifact representing the given composite. + * + * @param contribution * @param compositeURI * @return */ - private Artifact resolveCompositeArtifact(List contributions, String compositeURI) { + private String location(Contribution contribution, String uri) { + if (uri != null && uri.startsWith("/")) { + uri = uri.substring(1); + } ContributionFactory contributionFactory = modelFactories.getFactory(ContributionFactory.class); - // Remove the leading / + Artifact compositeFile = contributionFactory.createArtifact(); + compositeFile.setUnresolved(true); + compositeFile.setURI(uri); + ModelResolver resolver = contribution.getModelResolver(); + Artifact resolved = resolver.resolveModel(Artifact.class, compositeFile); + if (resolved != null && !resolved.isUnresolved()) { + return resolved.getLocation(); + } else { + return null; + } + } + + /** + * Returns the contribution containing the given composite. + * + * @param contributions + * @param compositeURI + * @return + */ + private Contribution contribution(List contributions, String compositeURI) { if (compositeURI != null && compositeURI.startsWith("/")) { compositeURI = compositeURI.substring(1); } + ContributionFactory contributionFactory = modelFactories.getFactory(ContributionFactory.class); Artifact compositeFile = contributionFactory.createArtifact(); compositeFile.setUnresolved(true); compositeFile.setURI(compositeURI); @@ -580,13 +582,13 @@ public class NodeImpl implements SCANode2, SCAClient { ModelResolver resolver = c.getModelResolver(); Artifact resolved = resolver.resolveModel(Artifact.class, compositeFile); if (resolved != null && !resolved.isUnresolved()) { - return resolved; + return c; } } return null; } - private void analyseProblems() throws Exception { + private void analyzeProblems() throws Exception { for (Problem problem : monitor.getProblems()) { if ((problem.getSeverity() == Severity.ERROR) && (!problem.getMessageId().equals("SchemaError"))) { diff --git a/java/sca/modules/node2-impl/src/main/java/org/apache/tuscany/sca/node/impl/NodeUtil.java b/java/sca/modules/node2-impl/src/main/java/org/apache/tuscany/sca/node/impl/NodeUtil.java new file mode 100644 index 0000000000..af6df7c6e9 --- /dev/null +++ b/java/sca/modules/node2-impl/src/main/java/org/apache/tuscany/sca/node/impl/NodeUtil.java @@ -0,0 +1,29 @@ +/* + * 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. + */ + +package org.apache.tuscany.sca.node.impl; + +/** + * NodeUtil + * + * @version $Rev: $ $Date: $ + */ +class NodeUtil { + +} -- cgit v1.2.3