[jira] [Commented] (CAMEL-8000) Add global notion of CamelContextRegistry

2014-11-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CAMEL-8000?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14225942#comment-14225942
 ] 

ASF GitHub Bot commented on CAMEL-8000:
---

Github user tdiesler closed the pull request at:

https://github.com/apache/camel/pull/322


 Add global notion of CamelContextRegistry
 -

 Key: CAMEL-8000
 URL: https://issues.apache.org/jira/browse/CAMEL-8000
 Project: Camel
  Issue Type: Improvement
Affects Versions: 2.14.0
Reporter: Thomas Diesler
Assignee: Claus Ibsen

 There are a number of issues with the Container API that make it unusable in 
 WildFly
 # Concept of unsynchronised singleton
 # Call to 3rd party code with partially constructed objects
 # Unsynchronised access to a shared resource
 Currently there are at least two components that compete for the Container 
 singleton. The Camel Subsystem and Insight Camel.
 I suspect that the Container API cannot be fixed in a compatible way. 
 Instead the notion of a CamelContextRegistry that fixes the issues with 
 Container may need to get added.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CAMEL-8000) Add global notion of CamelContextRegistry

2014-11-17 Thread Thomas Diesler (JIRA)

[ 
https://issues.apache.org/jira/browse/CAMEL-8000?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14214676#comment-14214676
 ] 

Thomas Diesler commented on CAMEL-8000:
---

The clear() shadows an issue with insufficiently defined context lifecycle. 

AFAICS, a CamelContext is owned by the client that creates it. The client 
therefore also controls the (softly defined) identity and the sharing. The 
context ends life when the client does no longer hold a reference to it. A call 
to CamelContext.stop() is optional.

We are now trying to introduce an entity that has a lifecycle that outlives 
individual context and (although it does not own them) holds references to 
them. To model this properly, we would probably want to have something like a 
CamelRuntime - similar to Framework and Bundle in OSGi. The runtime is the 
owner of all contexts, it provides a view of all registered contexts, it can be 
queried and possibly own other constructs that are applicable to multiple 
contexts (i.e. defaults for newly created contexts). This would be a larger 
change to camel and probably requires a lot more thought.

CamelContextRegistry.clear() is still wrong, but it is also wrong to assume 
that a context gets an non-optional call to CamelContext.stop() which ends its 
life. 

The registry probably needs to hold weak references, to avoid hogging of stale 
contexts.

 Add global notion of CamelContextRegistry
 -

 Key: CAMEL-8000
 URL: https://issues.apache.org/jira/browse/CAMEL-8000
 Project: Camel
  Issue Type: Improvement
Affects Versions: 2.14.0
Reporter: Thomas Diesler
Assignee: Claus Ibsen
 Fix For: 2.15.0


 There are a number of issues with the Container API that make it unusable in 
 WildFly
 # Concept of unsynchronised singleton
 # Call to 3rd party code with partially constructed objects
 # Unsynchronised access to a shared resource
 Currently there are at least two components that compete for the Container 
 singleton. The Camel Subsystem and Insight Camel.
 I suspect that the Container API cannot be fixed in a compatible way. 
 Instead the notion of a CamelContextRegistry that fixes the issues with 
 Container may need to get added.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CAMEL-8000) Add global notion of CamelContextRegistry

2014-11-10 Thread Thomas Diesler (JIRA)

[ 
https://issues.apache.org/jira/browse/CAMEL-8000?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14204805#comment-14204805
 ] 

Thomas Diesler commented on CAMEL-8000:
---

Pull request sent: https://github.com/apache/camel/pull/322

 Add global notion of CamelContextRegistry
 -

 Key: CAMEL-8000
 URL: https://issues.apache.org/jira/browse/CAMEL-8000
 Project: Camel
  Issue Type: Improvement
Affects Versions: 2.14.0
Reporter: Thomas Diesler
Assignee: Claus Ibsen
 Fix For: 2.15.0


 There are a number of issues with the Container API that make it unusable in 
 WildFly
 # Concept of unsynchronised singleton
 # Call to 3rd party code with partially constructed objects
 # Unsynchronised access to a shared resource
 Currently there are at least two components that compete for the Container 
 singleton. The Camel Subsystem and Insight Camel.
 I suspect that the Container API cannot be fixed in a compatible way. 
 Instead the notion of a CamelContextRegistry that fixes the issues with 
 Container may need to get added.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CAMEL-8000) Add global notion of CamelContextRegistry

2014-11-07 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CAMEL-8000?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14201850#comment-14201850
 ] 

ASF GitHub Bot commented on CAMEL-8000:
---

GitHub user tdiesler opened a pull request:

https://github.com/apache/camel/pull/322

[CAMEL-8000] Add global notion of CamelContextRegistry



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/tdiesler/camel bug8000

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/camel/pull/322.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #322


commit 6f428a8390910786de64abd4771604a213171842
Author: Thomas Diesler thomas.dies...@jboss.com
Date:   2014-11-07T09:48:15Z

[CAMEL-8000] Add global notion of CamelContextRegistry




 Add global notion of CamelContextRegistry
 -

 Key: CAMEL-8000
 URL: https://issues.apache.org/jira/browse/CAMEL-8000
 Project: Camel
  Issue Type: Bug
Affects Versions: 2.14.0
Reporter: Thomas Diesler
 Fix For: 2.15.0


 There are a number of issues with the Container API that make it unusable in 
 WildFly
 # Concept of unsynchronised singleton
 # Call to 3rd party code with partially constructed objects
 # Unsynchronised access to a shared resource
 Currently there are at least two components that compete for the Container 
 singleton. The Camel Subsystem and Insight Camel.
 I suspect that the Container API cannot be fixed in a compatible way. 
 Instead the notion of a CamelContextRegistry that fixes the issues with 
 Container may need to get added.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CAMEL-8000) Add global notion of CamelContextRegistry

2014-11-07 Thread Thomas Diesler (JIRA)

[ 
https://issues.apache.org/jira/browse/CAMEL-8000?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14201854#comment-14201854
 ] 

Thomas Diesler commented on CAMEL-8000:
---

Pull request sent: https://github.com/apache/camel/pull/322

 Add global notion of CamelContextRegistry
 -

 Key: CAMEL-8000
 URL: https://issues.apache.org/jira/browse/CAMEL-8000
 Project: Camel
  Issue Type: Bug
Affects Versions: 2.14.0
Reporter: Thomas Diesler
 Fix For: 2.15.0


 There are a number of issues with the Container API that make it unusable in 
 WildFly
 # Concept of unsynchronised singleton
 # Call to 3rd party code with partially constructed objects
 # Unsynchronised access to a shared resource
 Currently there are at least two components that compete for the Container 
 singleton. The Camel Subsystem and Insight Camel.
 I suspect that the Container API cannot be fixed in a compatible way. 
 Instead the notion of a CamelContextRegistry that fixes the issues with 
 Container may need to get added.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CAMEL-8000) Add global notion of CamelContextRegistry

2014-11-07 Thread Thomas Diesler (JIRA)

[ 
https://issues.apache.org/jira/browse/CAMEL-8000?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14202538#comment-14202538
 ] 

Thomas Diesler commented on CAMEL-8000:
---

merci

 Add global notion of CamelContextRegistry
 -

 Key: CAMEL-8000
 URL: https://issues.apache.org/jira/browse/CAMEL-8000
 Project: Camel
  Issue Type: Improvement
Affects Versions: 2.14.0
Reporter: Thomas Diesler
Assignee: Claus Ibsen
 Fix For: 2.15.0


 There are a number of issues with the Container API that make it unusable in 
 WildFly
 # Concept of unsynchronised singleton
 # Call to 3rd party code with partially constructed objects
 # Unsynchronised access to a shared resource
 Currently there are at least two components that compete for the Container 
 singleton. The Camel Subsystem and Insight Camel.
 I suspect that the Container API cannot be fixed in a compatible way. 
 Instead the notion of a CamelContextRegistry that fixes the issues with 
 Container may need to get added.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)