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

ARTEMIS-2403 Consumer command, clientID is not saved during JMS exception #259

Closed
wants to merge 2 commits into from
Closed

ARTEMIS-2403 Consumer command, clientID is not saved during JMS exception #259

wants to merge 2 commits into from

Conversation

franz1981
Copy link
Contributor

@franz1981 franz1981 commented Aug 1, 2019 •
edited by michalxo

(cherry picked from commit 741fd43)
(cherry picked from commit 705f1db)
downstream: ENTMQBR-2612
test: org.apache.activemq.cli.test.RetryCLIClientIDTest#testWrongUserAndPass
component: Artemis
subcomponent: message_delivery
level: component
importance: medium
type: functional
subtype: compliance
verifies: AMQ-90

shailendra14k and others added 2 commits August 1, 2019 18:17
@shailendra14k @franz1981
ARTEMIS-2403 Consumer command, clientID is ignored on CLI during retr…
c9cabc1 
…y connection with stdin input

(cherry picked from commit 741fd43)

downstream: ENTMQBR-2612
@clebertsuconic @franz1981
ARTEMIS-2403 Adding test with clientID on retry after exceptions
484d1a5 
(cherry picked from commit 705f1db)

downstream: ENTMQBR-2612
@franz1981 franz1981 added QE Review Needed Engineering Review Needed TEST-ADDED There is a test as part of the PR. (This could be used to help QA to make their evaluation) labels Aug 1, 2019
clebertsuconic added a commit that referenced this pull request Aug 2, 2019
@clebertsuconic
This is PR #259
e071c8e
@michalxo michalxo added Customer Issue Whether this issue originates from customer (higher importance) Metadata-added Test case metadata in description of issue is provided labels Nov 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@clebertsuconic clebertsuconic Awaiting requested review from clebertsuconic

Assignees
No one assigned
Labels
Customer Issue Whether this issue originates from customer (higher importance) Engineering Ack Metadata-added Test case metadata in description of issue is provided 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
4 participants
@franz1981 @clebertsuconic @michalxo @shailendra14k