You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ode.apache.org by "Alexis Midon (JIRA)" <ji...@apache.org> on 2009/03/04 20:19:56 UTC
[jira] Created: (ODE-539) ODEServer must not try to hold references
on ExternalServices
ODEServer must not try to hold references on ExternalServices
-------------------------------------------------------------
Key: ODE-539
URL: https://issues.apache.org/jira/browse/ODE-539
Project: ODE
Issue Type: Improvement
Affects Versions: 1.2
Reporter: Alexis Midon
Assignee: Alexis Midon
ODEServer has a multi-key map named _externalServices. This map is supposed to hold references to already-created external services sorted by service qname, port name.
This idea of this map is wrong because external service references should not be shared across processes. So the key should not be {service name, port name} but {process-conf, service name, port name} [1]. But for a given value of this 3-element key the reference to the external service will be requested only once. So the cash will be populated but never hit later. which makes the map useless.
[1] has no side effect in the current implementations, because the cache is not read properly, fortunately ;)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Closed: (ODE-539) ODEServer must not try to hold references
on ExternalServices
Posted by "Alexis Midon (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ODE-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Alexis Midon closed ODE-539.
----------------------------
> ODEServer must not try to hold references on ExternalServices
> -------------------------------------------------------------
>
> Key: ODE-539
> URL: https://issues.apache.org/jira/browse/ODE-539
> Project: ODE
> Issue Type: Improvement
> Affects Versions: 1.2
> Reporter: Alexis Midon
> Assignee: Alexis Midon
> Fix For: 1.3.1, 2.0
>
>
> ODEServer has a multi-key map named _externalServices. This map is supposed to hold references to already-created external services sorted by service qname, port name.
> This idea of this map is wrong because external service references should not be shared across processes. So the key should not be {service name, port name} but {process-conf, service name, port name} [1]. But for a given value of this 3-element key the reference to the external service will be requested only once. So the cash will be populated but never hit later. which makes the map useless.
> [1] has no side effect in the current implementations, because the cache is not read properly, fortunately ;)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Resolved: (ODE-539) ODEServer must not try to hold
references on ExternalServices
Posted by "Alexis Midon (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ODE-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Alexis Midon resolved ODE-539.
------------------------------
Resolution: Fixed
Fix Version/s: 2.0
1.3.1
committed in trunk and branch 1.X
> ODEServer must not try to hold references on ExternalServices
> -------------------------------------------------------------
>
> Key: ODE-539
> URL: https://issues.apache.org/jira/browse/ODE-539
> Project: ODE
> Issue Type: Improvement
> Affects Versions: 1.2
> Reporter: Alexis Midon
> Assignee: Alexis Midon
> Fix For: 1.3.1, 2.0
>
>
> ODEServer has a multi-key map named _externalServices. This map is supposed to hold references to already-created external services sorted by service qname, port name.
> This idea of this map is wrong because external service references should not be shared across processes. So the key should not be {service name, port name} but {process-conf, service name, port name} [1]. But for a given value of this 3-element key the reference to the external service will be requested only once. So the cash will be populated but never hit later. which makes the map useless.
> [1] has no side effect in the current implementations, because the cache is not read properly, fortunately ;)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.