Petals CDK

Unable to compute dynamically the provider service name

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 5.1.2
  • Fix Version/s: 5.1.3, 5.3.0
  • Component/s: runtime
  • Security Level: Public
  • Description:
    Hide

    We have a specific binding component previously developped with CDK 4. Our BC needs to define dynamically the provider service name according to its associated resource. Several instances of the BC, connected to different instances of the same applicative resource, are running on Petals network. The service name is defined from the service name provided by the SU and a property value defined at the BC instance.

    So, with the CDK 4, we used the method ServcieUnitManager.processProvidesNode(...) and ServcieUnitManager.processProvidesNodes(...) to compute the right service name.
    These methods are no more visible inthe CDK 5. They are defined as 'private' methods of AbstractServiceUnitManager.

    Please, add an extension point in CDK-5 to be able to redefine the provider service name.

    Show
    We have a specific binding component previously developped with CDK 4. Our BC needs to define dynamically the provider service name according to its associated resource. Several instances of the BC, connected to different instances of the same applicative resource, are running on Petals network. The service name is defined from the service name provided by the SU and a property value defined at the BC instance. So, with the CDK 4, we used the method ServcieUnitManager.processProvidesNode(...) and ServcieUnitManager.processProvidesNodes(...) to compute the right service name. These methods are no more visible inthe CDK 5. They are defined as 'private' methods of AbstractServiceUnitManager. Please, add an extension point in CDK-5 to be able to redefine the provider service name.
  • Environment:
    Windows XP
  1. patch.patch
    (5 kB)
    Issam Bousserhane
    Tue, 21 Feb 2012 - 10:49:52 +0100

Activity

Hide
Christophe DENEUX added a comment - Tue, 24 Jan 2012 - 12:10:19 +0100

I propose you to add the 'protected' method AbstractServiceUnitManager.getServiceName(final originalServiceName) that returns the new service name or null if the service name has not to be changed.

So you need only to override it to add your specific service name computation.

Show
Christophe DENEUX added a comment - Tue, 24 Jan 2012 - 12:10:19 +0100 I propose you to add the 'protected' method AbstractServiceUnitManager.getServiceName(final originalServiceName) that returns the new service name or null if the service name has not to be changed. So you need only to override it to add your specific service name computation.
Hide
Issam Bousserhane added a comment - Tue, 21 Feb 2012 - 10:33:28 +0100 - edited

The patch providing the extension point: getServiceName(final QName originalServiceName)

Show
Issam Bousserhane added a comment - Tue, 21 Feb 2012 - 10:33:28 +0100 - edited The patch providing the extension point: getServiceName(final QName originalServiceName)
Hide
Christophe DENEUX added a comment - Tue, 21 Feb 2012 - 12:26:28 +0100

Patch applied in branch petals-enterprise-3.1.x

Show
Christophe DENEUX added a comment - Tue, 21 Feb 2012 - 12:26:28 +0100 Patch applied in branch petals-enterprise-3.1.x
Hide
Christophe DENEUX added a comment - Tue, 21 Feb 2012 - 12:26:52 +0100

To merge in trunk

Show
Christophe DENEUX added a comment - Tue, 21 Feb 2012 - 12:26:52 +0100 To merge in trunk
Hide
Christophe DENEUX added a comment - Mon, 26 Mar 2012 - 14:43:57 +0200

Merged in trunk

Show
Christophe DENEUX added a comment - Mon, 26 Mar 2012 - 14:43:57 +0200 Merged in trunk

People

Dates

  • Created:
    Tue, 24 Jan 2012 - 11:46:51 +0100
    Updated:
    Mon, 26 Mar 2012 - 14:43:57 +0200
    Resolved:
    Mon, 26 Mar 2012 - 14:43:55 +0200