Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ENTMQBR-2706 : Fix xa-recovery repeated logging issue #321

Closed
wants to merge 2 commits into from
Closed

ENTMQBR-2706 : Fix xa-recovery repeated logging issue #321

wants to merge 2 commits into from

Conversation

gaohoward
Copy link
Contributor

Repeating WARN log message "Notified of connection failure" after every xa recovery when read-timeout is configure with a smaller value than default client-failure-check-period (30 seconds)

spyrkob and others added 2 commits March 3, 2020 20:09
@spyrkob @howardgao
[ARTEMIS-2176] RA connection properties are not propagated to XARecov…
3712987 
…eryConfig

(cherry picked from commit eb41be78f36c6df42376c4b0b79e174703084fa8)

downstream: ENTMQBR-2706
@howardgao
ARTEMIS-2176 Refactoring
9ee0d65 
(cherry picked from commit cb8da541107355f16eb26b21b6563a06876b741f)

downstream: ENTMQBR-2706
@brusdev brusdev added the LTS Long term support label Mar 3, 2020
@michalxo
Copy link
Contributor

michalxo commented Mar 5, 2020 •
edited

Ran locally and passes. Waiting on CI job to finish.
Tests run: 24, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 9.322 sec - in org.apache.activemq.artemis.tests.integration.ra.ResourceAdapterTest
https://amq-broker-jenkins-csb-messaging-qe.cloud.paas.psi.redhat.com/job/amq-broker-7/job/midstream/job/upstream-test-suite/16/console

@michalxo michalxo added QE Ack TEST-ADDED There is a test as part of the PR. (This could be used to help QA to make their evaluation) and removed QE Review Needed labels Mar 5, 2020
brusdev added a commit that referenced this pull request Mar 5, 2020
@brusdev
This is PR #321
c11cb44
@brusdev
Copy link
Contributor

brusdev commented Mar 5, 2020

Merged it

@brusdev brusdev closed this Mar 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Engineering Ack LTS Long term support QE Ack TEST-ADDED There is a test as part of the PR. (This could be used to help QA to make their evaluation)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@gaohoward @michalxo @brusdev @franz1981 @spyrkob @howardgao