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

Justin Bertram updated ARTEMIS-1691:
------------------------------------
    Description: 
h4. Setup
JMSBridge on server a pulls messages from a remote server B.

failureRetryInterval = 1 sec

maxRetries = 1
h4. To Reproduce

Restart the remote server (need to take more than one second.)

JMS Bridge prints out:
{noformat}
JMS Bridge unable to set up connections, bridge will be stopped{noformat}
After the remote server is up again, start the bridge again.

JMS Bridge prints out:
{noformat}
Started JMS Bridge testbridge{noformat} 
h4. Result:

The bridge is started, but does still not consume messages. 
h4. Reason

In {{JMSBridgeImpl}} line 1624, the failed boolean parameter is checked. It is 
not set to true anywhere outside the failure handlers ({{FailureHandler}} and 
{{StartupFailureHandler}}) or on initial adding of the bridge.

But the {{FailureHandler}} has already exited. So the bridge will wait all time 
long for 500 ms in line 1626. 
h4. Solution

I would add a
{code:java}
failed = false;{code}
after line 403.

I don't know how the release process is handled here, if I can make a pull 
request or something like that, I will do, if it is needed.

  was:
h4. Setup:

JMSBridge on server a pulls messages from a remote server b.

failureRetryInterval = 1 sec

maxRetries = 1

 
h4. To reproduce:

Restart the remote server (need to take more than one second.)

JMS Bridge prints out:
{noformat}
JMS Bridge unable to set up connections, bridge will be stopped{noformat}
After the remote server is up again, start the bridge again.

JMS Bridge prints out:
{noformat}
Started JMS Bridge testbridge{noformat}
 
h4. Result:

The bridge ist started, but does still not consume messages

 
h4. Reason:

In the Class JMSBridgeImpl line 1624, the failed boolean parameter is checked. 
It is not set to true anywhere outside the failure handlers (FailureHandler and 
StartupFailureHandler) or on initial adding of the bridge.

But the FailureHandler has already exited. So the bridge will wait all time 
long for 500 ms in line 1626.

 
h4. Solution:

I would add a
{code:java}
failed = false;{code}
after line 403.

I don't know how the release process is handled here, if I can make a pull 
request or something like that, I will do, if it is needed.

 

Cheers,

Markus


> JMS Bridge: After the FailureHandler exits, a bridge cannot be started full 
> functional anymore
> ----------------------------------------------------------------------------------------------
>
>                 Key: ARTEMIS-1691
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-1691
>             Project: ActiveMQ Artemis
>          Issue Type: Bug
>    Affects Versions: 2.4.0
>         Environment: WildFly Servers.
>            Reporter: Markus F
>            Priority: Minor
>
> h4. Setup
> JMSBridge on server a pulls messages from a remote server B.
> failureRetryInterval = 1 sec
> maxRetries = 1
> h4. To Reproduce
> Restart the remote server (need to take more than one second.)
> JMS Bridge prints out:
> {noformat}
> JMS Bridge unable to set up connections, bridge will be stopped{noformat}
> After the remote server is up again, start the bridge again.
> JMS Bridge prints out:
> {noformat}
> Started JMS Bridge testbridge{noformat} 
> h4. Result:
> The bridge is started, but does still not consume messages. 
> h4. Reason
> In {{JMSBridgeImpl}} line 1624, the failed boolean parameter is checked. It 
> is not set to true anywhere outside the failure handlers ({{FailureHandler}} 
> and {{StartupFailureHandler}}) or on initial adding of the bridge.
> But the {{FailureHandler}} has already exited. So the bridge will wait all 
> time long for 500 ms in line 1626. 
> h4. Solution
> I would add a
> {code:java}
> failed = false;{code}
> after line 403.
> I don't know how the release process is handled here, if I can make a pull 
> request or something like that, I will do, if it is needed.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to