You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@synapse.apache.org by Shafreen <an...@gmail.com> on 2020/05/10 03:44:01 UTC
Re: Adding another interval for retry in Scheduled Message Forwarding
Processor (SMFP)
Hi All,
On Fri, Apr 24, 2020 at 6:29 PM shafreen anfar <an...@gmail.com>
wrote:
> Hi All,
>
> Please find the below documentation PR.
>
> https://github.com/apache/synapse/pull/43
>
Build has passed for the above PR. In case someone get some free time,
please kindly have a look :)
>
>
> On Sun, Apr 12, 2020 at 11:01 PM shafreen anfar <an...@gmail.com>
> wrote:
>
>> Hi All,
>>
>> I will send another PR for documentation changes once the feature PR is
>> merged.
>>
>> On Sun, Apr 12, 2020 at 9:50 PM shafreen anfar <an...@gmail.com>
>> wrote:
>>
>>> Hi All,
>>>
>>> Please find the implementation of this PR in [1].
>>>
>>> [1] https://github.com/apache/synapse/pull/40
>>>
>>> On Tue, Mar 31, 2020 at 7:55 AM shafreen anfar <an...@gmail.com>
>>> wrote:
>>>
>>>> On Mon, Mar 30, 2020 at 10:07 PM Isuru Udana <is...@gmail.com> wrote:
>>>>
>>>>> Hi Shafreen,
>>>>>
>>>>> +1, as you said fetching messages and retrying are two different
>>>>> concepts and we need to have two configurable parameters.
>>>>>
>>>>
>>>> Thanks for the response, Isuru. Will start working on it.
>>>>
>>>>
>>>>>
>>>>> Thanks.
>>>>>
>>>>> On Sun, Mar 29, 2020 at 10:32 AM shafreen anfar <
>>>>> anfar.shafreen@gmail.com> wrote:
>>>>>
>>>>>> Hi All,
>>>>>>
>>>>>> At the moment we only support one interval for both fetching messages
>>>>>> from the store and retrying when there is a failure. But, IMO, these are
>>>>>> two different concepts and deserve two different configurations. Therefore,
>>>>>> I am proposing a new configuration parameter for latter. Something as below,
>>>>>>
>>>>>> <parameter name="retry.interval">10000</parameter>
>>>>>>
>>>>>> This will allow the user to control the speed of retry when there is
>>>>>> a failure.
>>>>>>
>>>>>> What do you think?
>>>>>>
>>>>>> Thanks,
>>>>>> Shafreen
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> *Isuru Udana*
>>>>> Associate Director/Architect
>>>>>
>>>>>
>>>>>
>>>>> *; WSO2 Inc.; http://wso2.com <http://wso2.com>email:
>>>>> isudana@gmail.com <is...@gmail.com> *
>>>>>
>>>>>
>>>>>
Re: Adding another interval for retry in Scheduled Message Forwarding
Processor (SMFP)
Posted by Vanjikumaran Sivajothy <va...@apache.org>.
Great! Thanks.
On Mon, May 11, 2020 at 6:42 AM Shafreen <an...@gmail.com> wrote:
> Hi Vanji,
>
> Thanks for reviewing the PR. I have updated the PR [1] accordingly.
>
> [1] https://github.com/apache/synapse/pull/43
>
> On Sun, May 10, 2020 at 9:14 AM Shafreen <an...@gmail.com> wrote:
>
>> Hi All,
>>
>> On Fri, Apr 24, 2020 at 6:29 PM shafreen anfar <an...@gmail.com>
>> wrote:
>>
>>> Hi All,
>>>
>>> Please find the below documentation PR.
>>>
>>> https://github.com/apache/synapse/pull/43
>>>
>>
>> Build has passed for the above PR. In case someone get some free time,
>> please kindly have a look :)
>>
>>
>>>
>>>
>>> On Sun, Apr 12, 2020 at 11:01 PM shafreen anfar <
>>> anfar.shafreen@gmail.com> wrote:
>>>
>>>> Hi All,
>>>>
>>>> I will send another PR for documentation changes once the feature PR is
>>>> merged.
>>>>
>>>> On Sun, Apr 12, 2020 at 9:50 PM shafreen anfar <
>>>> anfar.shafreen@gmail.com> wrote:
>>>>
>>>>> Hi All,
>>>>>
>>>>> Please find the implementation of this PR in [1].
>>>>>
>>>>> [1] https://github.com/apache/synapse/pull/40
>>>>>
>>>>> On Tue, Mar 31, 2020 at 7:55 AM shafreen anfar <
>>>>> anfar.shafreen@gmail.com> wrote:
>>>>>
>>>>>> On Mon, Mar 30, 2020 at 10:07 PM Isuru Udana <is...@gmail.com>
>>>>>> wrote:
>>>>>>
>>>>>>> Hi Shafreen,
>>>>>>>
>>>>>>> +1, as you said fetching messages and retrying are two different
>>>>>>> concepts and we need to have two configurable parameters.
>>>>>>>
>>>>>>
>>>>>> Thanks for the response, Isuru. Will start working on it.
>>>>>>
>>>>>>
>>>>>>>
>>>>>>> Thanks.
>>>>>>>
>>>>>>> On Sun, Mar 29, 2020 at 10:32 AM shafreen anfar <
>>>>>>> anfar.shafreen@gmail.com> wrote:
>>>>>>>
>>>>>>>> Hi All,
>>>>>>>>
>>>>>>>> At the moment we only support one interval for both fetching
>>>>>>>> messages from the store and retrying when there is a failure. But, IMO,
>>>>>>>> these are two different concepts and deserve two different configurations.
>>>>>>>> Therefore, I am proposing a new configuration parameter for latter.
>>>>>>>> Something as below,
>>>>>>>>
>>>>>>>> <parameter name="retry.interval">10000</parameter>
>>>>>>>>
>>>>>>>> This will allow the user to control the speed of retry when there
>>>>>>>> is a failure.
>>>>>>>>
>>>>>>>> What do you think?
>>>>>>>>
>>>>>>>> Thanks,
>>>>>>>> Shafreen
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> *Isuru Udana*
>>>>>>> Associate Director/Architect
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> *; WSO2 Inc.; http://wso2.com <http://wso2.com>email:
>>>>>>> isudana@gmail.com <is...@gmail.com> *
>>>>>>>
>>>>>>>
>>>>>>>
Re: Adding another interval for retry in Scheduled Message Forwarding
Processor (SMFP)
Posted by Shafreen <an...@gmail.com>.
Hi Vanji,
Thanks for reviewing the PR. I have updated the PR [1] accordingly.
[1] https://github.com/apache/synapse/pull/43
On Sun, May 10, 2020 at 9:14 AM Shafreen <an...@gmail.com> wrote:
> Hi All,
>
> On Fri, Apr 24, 2020 at 6:29 PM shafreen anfar <an...@gmail.com>
> wrote:
>
>> Hi All,
>>
>> Please find the below documentation PR.
>>
>> https://github.com/apache/synapse/pull/43
>>
>
> Build has passed for the above PR. In case someone get some free time,
> please kindly have a look :)
>
>
>>
>>
>> On Sun, Apr 12, 2020 at 11:01 PM shafreen anfar <an...@gmail.com>
>> wrote:
>>
>>> Hi All,
>>>
>>> I will send another PR for documentation changes once the feature PR is
>>> merged.
>>>
>>> On Sun, Apr 12, 2020 at 9:50 PM shafreen anfar <an...@gmail.com>
>>> wrote:
>>>
>>>> Hi All,
>>>>
>>>> Please find the implementation of this PR in [1].
>>>>
>>>> [1] https://github.com/apache/synapse/pull/40
>>>>
>>>> On Tue, Mar 31, 2020 at 7:55 AM shafreen anfar <
>>>> anfar.shafreen@gmail.com> wrote:
>>>>
>>>>> On Mon, Mar 30, 2020 at 10:07 PM Isuru Udana <is...@gmail.com>
>>>>> wrote:
>>>>>
>>>>>> Hi Shafreen,
>>>>>>
>>>>>> +1, as you said fetching messages and retrying are two different
>>>>>> concepts and we need to have two configurable parameters.
>>>>>>
>>>>>
>>>>> Thanks for the response, Isuru. Will start working on it.
>>>>>
>>>>>
>>>>>>
>>>>>> Thanks.
>>>>>>
>>>>>> On Sun, Mar 29, 2020 at 10:32 AM shafreen anfar <
>>>>>> anfar.shafreen@gmail.com> wrote:
>>>>>>
>>>>>>> Hi All,
>>>>>>>
>>>>>>> At the moment we only support one interval for both fetching
>>>>>>> messages from the store and retrying when there is a failure. But, IMO,
>>>>>>> these are two different concepts and deserve two different configurations.
>>>>>>> Therefore, I am proposing a new configuration parameter for latter.
>>>>>>> Something as below,
>>>>>>>
>>>>>>> <parameter name="retry.interval">10000</parameter>
>>>>>>>
>>>>>>> This will allow the user to control the speed of retry when there is
>>>>>>> a failure.
>>>>>>>
>>>>>>> What do you think?
>>>>>>>
>>>>>>> Thanks,
>>>>>>> Shafreen
>>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> *Isuru Udana*
>>>>>> Associate Director/Architect
>>>>>>
>>>>>>
>>>>>>
>>>>>> *; WSO2 Inc.; http://wso2.com <http://wso2.com>email:
>>>>>> isudana@gmail.com <is...@gmail.com> *
>>>>>>
>>>>>>
>>>>>>