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

Christophe DENEUX made changes - Tue, 24 Jan 2012 - 12:00:18 +0100
Field Original Value New Value
Project Petals-ESB-Container [ 10029 ] Petals-CDK [ 10032 ]
Key PETALSESBCONT-171 PETALSCDK-72
Affects Version/s 5.1.2 [ 10180 ]
Affects Version/s 3.1.3 [ 10252 ]
Christophe DENEUX made changes - Tue, 24 Jan 2012 - 12:06:54 +0100
Summary processInterceptors methode no longer exists in the ServiceUnitManager class Unable to compute dynamically the provider service name
Description The new method getServiceName was added to AbstractServiceUnitManager class to allaw the change of the service name. 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.
Priority Blocker [ 1 ]
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.
Christophe DENEUX made changes - Tue, 24 Jan 2012 - 12:10:19 +0100
Status New [ 10000 ] Open [ 10002 ]
Component/s runtime [ 10075 ]
Christophe DENEUX made changes - Tue, 24 Jan 2012 - 12:10:56 +0100
Description 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.
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.
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)
Issam Bousserhane made changes - Tue, 21 Feb 2012 - 10:33:28 +0100
Attachment patch.txt [ 10709 ]
Issam Bousserhane made changes - Tue, 21 Feb 2012 - 10:49:42 +0100
Attachment patch.txt [ 10709 ]
Issam Bousserhane made changes - Tue, 21 Feb 2012 - 10:49:52 +0100
Attachment patch.patch [ 10710 ]
Christophe DENEUX made changes - Tue, 21 Feb 2012 - 12:25:39 +0100
Status Open [ 10002 ] In Progress [ 10003 ]
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
Christophe DENEUX made changes - Tue, 21 Feb 2012 - 12:26:28 +0100
Status In Progress [ 10003 ] Resolved [ 10004 ]
Fix Version/s 5.1.3 [ 10284 ]
Resolution Fixed [ 1 ]
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
Christophe DENEUX made changes - Tue, 21 Feb 2012 - 12:26:52 +0100
Status Resolved [ 10004 ] Open [ 10002 ]
Resolution Fixed [ 1 ]
Christophe DENEUX made changes - Wed, 21 Mar 2012 - 15:48:26 +0100
Fix Version/s 5.3.0 [ 10338 ]
Christophe DENEUX made changes - Mon, 26 Mar 2012 - 14:42:18 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
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
Christophe DENEUX made changes - Mon, 26 Mar 2012 - 14:43:57 +0200
Status In Progress [ 10003 ] Resolved [ 10004 ]
Resolution Fixed [ 1 ]
Transition Status Change Time Execution Times Last Executer Last Execution Date
New New Open Open
23m 28s
1
Christophe DENEUX
Tue, 24 Jan 2012 - 12:10:19 +0100
Open Open In Progress In Progress
28d 15m
1
Christophe DENEUX
Tue, 21 Feb 2012 - 12:25:39 +0100
In Progress In Progress Resolved Resolved
49s
1
Christophe DENEUX
Tue, 21 Feb 2012 - 12:26:28 +0100
Resolved Resolved Open Open
24s
1
Christophe DENEUX
Tue, 21 Feb 2012 - 12:26:52 +0100
Open Open In Progress In Progress
34d 1h 15m
1
Christophe DENEUX
Mon, 26 Mar 2012 - 14:42:18 +0200
In Progress In Progress Resolved Resolved
1m 39s
1
Christophe DENEUX
Mon, 26 Mar 2012 - 14:43:57 +0200



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