You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ff...@apache.org on 2021/03/02 14:19:23 UTC
[camel] branch camel-3.8.x updated: [CAMEL-16063]don't start camel
context if event is from the self management ApplicationContext
This is an automated email from the ASF dual-hosted git repository.
ffang pushed a commit to branch camel-3.8.x
in repository https://gitbox.apache.org/repos/asf/camel.git
The following commit(s) were added to refs/heads/camel-3.8.x by this push:
new 7ef2913 [CAMEL-16063]don't start camel context if event is from the self management ApplicationContext
7ef2913 is described below
commit 7ef2913089f6f5307ad2d75092f0e54421892928
Author: Freeman Fang <fr...@gmail.com>
AuthorDate: Tue Mar 2 09:18:52 2021 -0500
[CAMEL-16063]don't start camel context if event is from the self management ApplicationContext
---
.../java/org/apache/camel/spring/CamelContextFactoryBean.java | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java b/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java
index e91072e..43f4529 100644
--- a/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java
+++ b/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java
@@ -462,6 +462,14 @@ public class CamelContextFactoryBean extends AbstractCamelContextFactoryBean<Spr
// being started/refreshed, there could be a race condition with
// other ApplicationListeners that react to
// ContextRefreshedEvent but this is the best that we can do
+ if (event.getSource() instanceof ApplicationContext) {
+ ApplicationContext appCtx = (ApplicationContext) event.getSource();
+ if (appCtx.getId().equals("application:management")) {
+ //don't start camel context if
+ //event is from the self management ApplicationContext
+ return;
+ }
+ }
start();
}
Re: [camel] branch camel-3.8.x updated: [CAMEL-16063]don't start
camel context if event is from the self management ApplicationContext
Posted by Freeman Fang <fr...@gmail.com>.
Got it, thanks for the clarification!
Freeman
On Tue, Mar 2, 2021 at 9:25 AM Andrea Cosentino <an...@gmail.com> wrote:
> This is not needed, 3.8.x won't be released anymore, it's not an LTS. No
> need to revert, but just to avoid confusion.
>
> Il giorno mar 2 mar 2021 alle ore 15:19 <ff...@apache.org> ha scritto:
>
> > This is an automated email from the ASF dual-hosted git repository.
> >
> > ffang pushed a commit to branch camel-3.8.x
> > in repository https://gitbox.apache.org/repos/asf/camel.git
> >
> >
> > The following commit(s) were added to refs/heads/camel-3.8.x by this
> push:
> > new 7ef2913 [CAMEL-16063]don't start camel context if event is from
> > the self management ApplicationContext
> > 7ef2913 is described below
> >
> > commit 7ef2913089f6f5307ad2d75092f0e54421892928
> > Author: Freeman Fang <fr...@gmail.com>
> > AuthorDate: Tue Mar 2 09:18:52 2021 -0500
> >
> > [CAMEL-16063]don't start camel context if event is from the self
> > management ApplicationContext
> > ---
> > .../java/org/apache/camel/spring/CamelContextFactoryBean.java | 8
> > ++++++++
> > 1 file changed, 8 insertions(+)
> >
> > diff --git
> >
> a/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java
> >
> b/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java
> > index e91072e..43f4529 100644
> > ---
> >
> a/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java
> > +++
> >
> b/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java
> > @@ -462,6 +462,14 @@ public class CamelContextFactoryBean extends
> > AbstractCamelContextFactoryBean<Spr
> > // being started/refreshed, there could be a race condition with
> > // other ApplicationListeners that react to
> > // ContextRefreshedEvent but this is the best that we can do
> > + if (event.getSource() instanceof ApplicationContext) {
> > + ApplicationContext appCtx = (ApplicationContext)
> > event.getSource();
> > + if (appCtx.getId().equals("application:management")) {
> > + //don't start camel context if
> > + //event is from the self management ApplicationContext
> > + return;
> > + }
> > + }
> > start();
> > }
> >
> >
> >
>
Re: [camel] branch camel-3.8.x updated: [CAMEL-16063]don't start
camel context if event is from the self management ApplicationContext
Posted by Andrea Cosentino <an...@gmail.com>.
This is not needed, 3.8.x won't be released anymore, it's not an LTS. No
need to revert, but just to avoid confusion.
Il giorno mar 2 mar 2021 alle ore 15:19 <ff...@apache.org> ha scritto:
> This is an automated email from the ASF dual-hosted git repository.
>
> ffang pushed a commit to branch camel-3.8.x
> in repository https://gitbox.apache.org/repos/asf/camel.git
>
>
> The following commit(s) were added to refs/heads/camel-3.8.x by this push:
> new 7ef2913 [CAMEL-16063]don't start camel context if event is from
> the self management ApplicationContext
> 7ef2913 is described below
>
> commit 7ef2913089f6f5307ad2d75092f0e54421892928
> Author: Freeman Fang <fr...@gmail.com>
> AuthorDate: Tue Mar 2 09:18:52 2021 -0500
>
> [CAMEL-16063]don't start camel context if event is from the self
> management ApplicationContext
> ---
> .../java/org/apache/camel/spring/CamelContextFactoryBean.java | 8
> ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git
> a/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java
> b/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java
> index e91072e..43f4529 100644
> ---
> a/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java
> +++
> b/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java
> @@ -462,6 +462,14 @@ public class CamelContextFactoryBean extends
> AbstractCamelContextFactoryBean<Spr
> // being started/refreshed, there could be a race condition with
> // other ApplicationListeners that react to
> // ContextRefreshedEvent but this is the best that we can do
> + if (event.getSource() instanceof ApplicationContext) {
> + ApplicationContext appCtx = (ApplicationContext)
> event.getSource();
> + if (appCtx.getId().equals("application:management")) {
> + //don't start camel context if
> + //event is from the self management ApplicationContext
> + return;
> + }
> + }
> start();
> }
>
>
>