[
https://issues.apache.org/jira/browse/CAMEL-19497?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Otavio Rodolfo Piske updated CAMEL-19497:
-----------------------------------------
Description:
I am not sure if this is a bug or not, so this is more like an investigative
task.
The DefaultAsyncProcessorAwaitManager can [lock
indefinitely|https://github.com/apache/camel/blob/b2f6af4a695439d51ace77331f79157547f6cfd8/core/camel-base-engine/src/main/java/org/apache/camel/impl/engine/DefaultAsyncProcessorAwaitManager.java#L105]
under some circumstances.
One scenario where this could happen is by ignoring/not handling the exceptions
on [this block on the
AbstractBeanProcessor|https://github.com/apache/camel/blob/main/components/camel-bean/src/main/java/org/apache/camel/component/bean/AbstractBeanProcessor.java#L109].
This was originally found when fixing the code to not catch a Throwable and
then running the BeanThrowAssertionErrorTest. Because that test would throw a
descendant of an AssertionError, it would cause the code to not handle the
exception.
This [has been fixed already|https://github.com/apache/camel/pull/10489], but
there could be other circumstances that could cause this.
Maybe we should investigate if it's feasible to have a timeout or something
similar.
was:
I am not sure if this is a bug or not, so this is more like an investigative
task.
The DefaultAsyncProcessorAwaitManager can [lock
indefinitely|https://github.com/apache/camel/blob/b2f6af4a695439d51ace77331f79157547f6cfd8/core/camel-base-engine/src/main/java/org/apache/camel/impl/engine/DefaultAsyncProcessorAwaitManager.java#L105]
under some circumstances.
One scenario where this could happen is by ignoring/not handling the exceptions
on [this block on the
AbstractBeanProcessor|https://github.com/apache/camel/blob/main/components/camel-bean/src/main/java/org/apache/camel/component/bean/AbstractBeanProcessor.java#L109].
This was originally found when fixing the code to not catch a Throwable and
then running the BeanThrowAssertionErrorTest. Because that test would throw a
descendant of an AssertionError, it would cause the code to not handle the
exception.
This [has been fixed already|https://github.com/apache/camel/pull/10489], but
there could be other circumstances that could cause this.
Maybe we should investigate if it's feasible to have a timeout or something
similar.
> camel-core: DefaultAsyncProcessorAwaitManager can lock forever
> --------------------------------------------------------------
>
> Key: CAMEL-19497
> URL: https://issues.apache.org/jira/browse/CAMEL-19497
> Project: Camel
> Issue Type: Task
> Reporter: Otavio Rodolfo Piske
> Priority: Major
>
> I am not sure if this is a bug or not, so this is more like an investigative
> task.
>
> The DefaultAsyncProcessorAwaitManager can [lock
> indefinitely|https://github.com/apache/camel/blob/b2f6af4a695439d51ace77331f79157547f6cfd8/core/camel-base-engine/src/main/java/org/apache/camel/impl/engine/DefaultAsyncProcessorAwaitManager.java#L105]
> under some circumstances.
>
> One scenario where this could happen is by ignoring/not handling the
> exceptions on [this block on the
> AbstractBeanProcessor|https://github.com/apache/camel/blob/main/components/camel-bean/src/main/java/org/apache/camel/component/bean/AbstractBeanProcessor.java#L109].
> This was originally found when fixing the code to not catch a Throwable and
> then running the BeanThrowAssertionErrorTest. Because that test would throw a
> descendant of an AssertionError, it would cause the code to not handle the
> exception.
> This [has been fixed already|https://github.com/apache/camel/pull/10489], but
> there could be other circumstances that could cause this.
>
> Maybe we should investigate if it's feasible to have a timeout or something
> similar.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
