[jira] [Commented] (CAMEL-12588) AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call

2018-06-26 Thread Sudharsan Govindarajan (JIRA)


[ 
https://issues.apache.org/jira/browse/CAMEL-12588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16523704#comment-16523704
 ] 

Sudharsan Govindarajan commented on CAMEL-12588:


OK, will change the code to use the boolean iso null check and run the
tests again..

On Tue, Jun 26, 2018 at 3:13 PM ASF GitHub Bot (JIRA) 



> AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call
> -
>
> Key: CAMEL-12588
> URL: https://issues.apache.org/jira/browse/CAMEL-12588
> Project: Camel
>  Issue Type: Improvement
>  Components: camel-core
>Affects Versions: 2.21.1
>Reporter: Sudharsan Govindarajan
>Assignee: Andrea Cosentino
>Priority: Major
>
> When a Aggregator route is stopped, only the AggregateRecoveryChecker threads 
> are stopped and not the AggregateTimeoutChecker threads. They keep lingering 
> around.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Reopened] (CAMEL-12588) AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call

2018-06-20 Thread Sudharsan Govindarajan (JIRA)


 [ 
https://issues.apache.org/jira/browse/CAMEL-12588?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sudharsan Govindarajan reopened CAMEL-12588:


The problem still exists eventhough the previous fix PR had caused regression

> AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call
> -
>
> Key: CAMEL-12588
> URL: https://issues.apache.org/jira/browse/CAMEL-12588
> Project: Camel
>  Issue Type: Improvement
>  Components: camel-core
>Affects Versions: 2.21.1
>Reporter: Sudharsan Govindarajan
>Assignee: Andrea Cosentino
>Priority: Major
>
> When a Aggregator route is stopped, only the AggregateRecoveryChecker threads 
> are stopped and not the AggregateTimeoutChecker threads. They keep lingering 
> around.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12588) AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call

2018-06-20 Thread Sudharsan Govindarajan (JIRA)


[ 
https://issues.apache.org/jira/browse/CAMEL-12588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16518261#comment-16518261
 ] 

Sudharsan Govindarajan commented on CAMEL-12588:


doStart()  starts back timeoutCheckerExecutorService  only if it is null (it 
does not consider the
state of the pool)

So, I have set timeoutCheckerExecutorService to null in doStop() (like in 
doShutDown()) and made a new PR [https://github.com/apache/camel/pull/2387]

I have tested this a bit more thorough than the previous times. Both the 
recovery checker and timeout checker threads now stop and start as per the 
method calls. Sorry about the previous oversight

> AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call
> -
>
> Key: CAMEL-12588
> URL: https://issues.apache.org/jira/browse/CAMEL-12588
> Project: Camel
>  Issue Type: Improvement
>  Components: camel-core
>Affects Versions: 2.21.1
>Reporter: Sudharsan Govindarajan
>Assignee: Andrea Cosentino
>Priority: Major
>
> When a Aggregator route is stopped, only the AggregateRecoveryChecker threads 
> are stopped and not the AggregateTimeoutChecker threads. They keep lingering 
> around.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12588) AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call

2018-06-20 Thread Sudharsan Govindarajan (JIRA)


[ 
https://issues.apache.org/jira/browse/CAMEL-12588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16518148#comment-16518148
 ] 

Sudharsan Govindarajan commented on CAMEL-12588:


Thanks! 

> AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call
> -
>
> Key: CAMEL-12588
> URL: https://issues.apache.org/jira/browse/CAMEL-12588
> Project: Camel
>  Issue Type: Improvement
>  Components: camel-core
>Affects Versions: 2.21.1
>Reporter: Sudharsan Govindarajan
>Assignee: Andrea Cosentino
>Priority: Major
>
> When a Aggregator route is stopped, only the AggregateRecoveryChecker threads 
> are stopped and not the AggregateTimeoutChecker threads. They keep lingering 
> around.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12588) AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call

2018-06-20 Thread Sudharsan Govindarajan (JIRA)


[ 
https://issues.apache.org/jira/browse/CAMEL-12588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16518132#comment-16518132
 ] 

Sudharsan Govindarajan commented on CAMEL-12588:


Hmm, that is weird. AggregateRecoveryChecker threads are stopped already in 
doStop(). I just made the working of AggregateTimeoutChecker threads consistent 
to how AggregateRecoveryChecker threads work.

> AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call
> -
>
> Key: CAMEL-12588
> URL: https://issues.apache.org/jira/browse/CAMEL-12588
> Project: Camel
>  Issue Type: Improvement
>  Components: camel-core
>Affects Versions: 2.21.1
>Reporter: Sudharsan Govindarajan
>Assignee: Andrea Cosentino
>Priority: Major
>
> When a Aggregator route is stopped, only the AggregateRecoveryChecker threads 
> are stopped and not the AggregateTimeoutChecker threads. They keep lingering 
> around.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12588) AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call

2018-06-20 Thread Sudharsan Govindarajan (JIRA)


[ 
https://issues.apache.org/jira/browse/CAMEL-12588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16518114#comment-16518114
 ] 

Sudharsan Govindarajan commented on CAMEL-12588:


But this option shutdownTimeoutCheckerExecutorService is not checked for / used 
in doStop() method. 

> AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call
> -
>
> Key: CAMEL-12588
> URL: https://issues.apache.org/jira/browse/CAMEL-12588
> Project: Camel
>  Issue Type: Improvement
>  Components: camel-core
>Affects Versions: 2.21.1
>Reporter: Sudharsan Govindarajan
>Assignee: Andrea Cosentino
>Priority: Major
>
> When a Aggregator route is stopped, only the AggregateRecoveryChecker threads 
> are stopped and not the AggregateTimeoutChecker threads. They keep lingering 
> around.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Resolved] (CAMEL-12588) AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call

2018-06-19 Thread Sudharsan Govindarajan (JIRA)


 [ 
https://issues.apache.org/jira/browse/CAMEL-12588?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sudharsan Govindarajan resolved CAMEL-12588.

   Resolution: Fixed
Fix Version/s: Future

> AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call
> -
>
> Key: CAMEL-12588
> URL: https://issues.apache.org/jira/browse/CAMEL-12588
> Project: Camel
>  Issue Type: Improvement
>  Components: camel-core
>Affects Versions: 2.21.1
>Reporter: Sudharsan Govindarajan
>Priority: Major
> Fix For: Future
>
>
> When a Aggregator route is stopped, only the AggregateRecoveryChecker threads 
> are stopped and not the AggregateTimeoutChecker threads. They keep lingering 
> around.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12588) AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call

2018-06-19 Thread Sudharsan Govindarajan (JIRA)


[ 
https://issues.apache.org/jira/browse/CAMEL-12588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16516982#comment-16516982
 ] 

Sudharsan Govindarajan commented on CAMEL-12588:


PR here [https://github.com/apache/camel/pull/2384]
Thanks!

> AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call
> -
>
> Key: CAMEL-12588
> URL: https://issues.apache.org/jira/browse/CAMEL-12588
> Project: Camel
>  Issue Type: Improvement
>  Components: camel-core
>Affects Versions: 2.21.1
>Reporter: Sudharsan Govindarajan
>Priority: Major
>
> When a Aggregator route is stopped, only the AggregateRecoveryChecker threads 
> are stopped and not the AggregateTimeoutChecker threads. They keep lingering 
> around.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (CAMEL-12588) AggregateProcessor does not stop AggregateTimeoutChecker threads on stop call

2018-06-19 Thread Sudharsan Govindarajan (JIRA)
Sudharsan Govindarajan created CAMEL-12588:
--

 Summary: AggregateProcessor does not stop AggregateTimeoutChecker 
threads on stop call
 Key: CAMEL-12588
 URL: https://issues.apache.org/jira/browse/CAMEL-12588
 Project: Camel
  Issue Type: Improvement
  Components: camel-core
Affects Versions: 2.21.1
Reporter: Sudharsan Govindarajan


When a Aggregator route is stopped, only the AggregateRecoveryChecker threads 
are stopped and not the AggregateTimeoutChecker threads. They keep lingering 
around.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)