seda default queue size

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

seda default queue size

dennisholu
Hi,

according to the seda documetation
https://github.com/apache/camel/blob/master/camel-core/src/main/docs/seda-component.adoc
the default size value is 2147483647. However I observe that it will
actually default to 1000, which is the default value of queueSize
option. Source code of SedaEndpoint:

Integer size = getSize() == Integer.MAX_VALUE ? null : getSize();

Why there are two size options with different default values?

Dan
Reply | Threaded
Open this post in threaded view
|

Re: seda default queue size

Claus Ibsen-2
Hi

In the beginning there was only one size you had to set per endpoint.
So later we added a default size on the component so you can set it
once.
Then alter that default was reduced from infinitie to 1000. So today
its 1000 and there is a glitch in the docs.
On Tue, Sep 11, 2018 at 11:30 AM Dennis Holunder <[hidden email]> wrote:

>
> Hi,
>
> according to the seda documetation
> https://github.com/apache/camel/blob/master/camel-core/src/main/docs/seda-component.adoc
> the default size value is 2147483647. However I observe that it will
> actually default to 1000, which is the default value of queueSize
> option. Source code of SedaEndpoint:
>
> Integer size = getSize() == Integer.MAX_VALUE ? null : getSize();
>
> Why there are two size options with different default values?
>
> Dan



--
Claus Ibsen
-----------------
http://davsclaus.com @davsclaus
Camel in Action 2: https://www.manning.com/ibsen2
Reply | Threaded
Open this post in threaded view
|

Re: seda default queue size

Claus Ibsen-2
Hi

Its this ticket about the 1000 as new default size
https://issues.apache.org/jira/browse/CAMEL-12542
On Wed, Sep 19, 2018 at 12:48 PM Claus Ibsen <[hidden email]> wrote:

>
> Hi
>
> In the beginning there was only one size you had to set per endpoint.
> So later we added a default size on the component so you can set it
> once.
> Then alter that default was reduced from infinitie to 1000. So today
> its 1000 and there is a glitch in the docs.
> On Tue, Sep 11, 2018 at 11:30 AM Dennis Holunder <[hidden email]> wrote:
> >
> > Hi,
> >
> > according to the seda documetation
> > https://github.com/apache/camel/blob/master/camel-core/src/main/docs/seda-component.adoc
> > the default size value is 2147483647. However I observe that it will
> > actually default to 1000, which is the default value of queueSize
> > option. Source code of SedaEndpoint:
> >
> > Integer size = getSize() == Integer.MAX_VALUE ? null : getSize();
> >
> > Why there are two size options with different default values?
> >
> > Dan
>
>
>
> --
> Claus Ibsen
> -----------------
> http://davsclaus.com @davsclaus
> Camel in Action 2: https://www.manning.com/ibsen2



--
Claus Ibsen
-----------------
http://davsclaus.com @davsclaus
Camel in Action 2: https://www.manning.com/ibsen2
Reply | Threaded
Open this post in threaded view
|

Re: seda default queue size

Claus Ibsen-2
Hi

I have fixed the docs
On Wed, Sep 19, 2018 at 12:50 PM Claus Ibsen <[hidden email]> wrote:

>
> Hi
>
> Its this ticket about the 1000 as new default size
> https://issues.apache.org/jira/browse/CAMEL-12542
> On Wed, Sep 19, 2018 at 12:48 PM Claus Ibsen <[hidden email]> wrote:
> >
> > Hi
> >
> > In the beginning there was only one size you had to set per endpoint.
> > So later we added a default size on the component so you can set it
> > once.
> > Then alter that default was reduced from infinitie to 1000. So today
> > its 1000 and there is a glitch in the docs.
> > On Tue, Sep 11, 2018 at 11:30 AM Dennis Holunder <[hidden email]> wrote:
> > >
> > > Hi,
> > >
> > > according to the seda documetation
> > > https://github.com/apache/camel/blob/master/camel-core/src/main/docs/seda-component.adoc
> > > the default size value is 2147483647. However I observe that it will
> > > actually default to 1000, which is the default value of queueSize
> > > option. Source code of SedaEndpoint:
> > >
> > > Integer size = getSize() == Integer.MAX_VALUE ? null : getSize();
> > >
> > > Why there are two size options with different default values?
> > >
> > > Dan
> >
> >
> >
> > --
> > Claus Ibsen
> > -----------------
> > http://davsclaus.com @davsclaus
> > Camel in Action 2: https://www.manning.com/ibsen2
>
>
>
> --
> Claus Ibsen
> -----------------
> http://davsclaus.com @davsclaus
> Camel in Action 2: https://www.manning.com/ibsen2



--
Claus Ibsen
-----------------
http://davsclaus.com @davsclaus
Camel in Action 2: https://www.manning.com/ibsen2
Reply | Threaded
Open this post in threaded view
|

Handled Exception in RecipientList

DariusX
In reply to this post by Claus Ibsen-2
I use RecipientList with a custom Aggregator. If there is an error in a
sub-route (one of the "recipients") that is "handled" there.
I want the main route to continue
In the Aggregator, I was setting:
    if (oldExchange == null) {
        oldExchange = newExchange;
   }

Because of this, the Exchange had properties like "CamelExceptionCaught"
and "CamelErrorHandlerHandled"
And, because these were present in the Aggregated Exchange the flow was
not continuing through the outer route

After removing these two, the flow continued through the main route

I have two questions about this:
1. Does this sound like the right way to handle things in the Aggregator?
2. Any other Properties that I need to watch out for an remove, if present?

---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus