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-4306 - large message fix on DLQ and Expiry #427

Closed
wants to merge 3 commits into from
Closed

ENTMQBR-4306 - large message fix on DLQ and Expiry #427

wants to merge 3 commits into from

Conversation

clebertsuconic
Copy link
Contributor

No description provided.

@clebertsuconic
ARTEMIS-3019 Before applying the fix into two methods, I needed to mo…
4d42472 
…ve the method into the right place

There is no real change in this commit other than moving the methods to the right place

(cherry picked from commit ccefbfc)

downstream: ENTMQBR-4306
@clebertsuconic
ARTEMIS-3019 expiry changes in AMQP Large Message
d616869 
(cherry picked from commit e6a6e81)

downstream: ENTMQBR-4306
@clebertsuconic clebertsuconic added QE 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 Dec 7, 2020
@clebertsuconic
ARTEMIS-3023 Avoid opening AMQP Large Message for final parsing
2069049 
(cherry picked from commit 6a6a3b3)

downstream: ENTMQBR-4306
@brusdev brusdev added the LTS-7.8 label Mar 4, 2021
@feuillemorte feuillemorte self-requested a review March 4, 2021 15:30
@feuillemorte
Copy link
Contributor

There are 94 failures and 117 skipped in unittests results. Please, review if needed: https://amq-broker-jenkins-csb-messaging-qe.cloud.paas.psi.redhat.com/job/amq-broker-7/job/midstream/job/upstream-test-suite/job/upstream-test-suite/46/testReport/

brusdev added a commit that referenced this pull request Mar 22, 2021
@brusdev
This is PR #427
edd970a
@brusdev
Copy link
Contributor

brusdev commented Mar 22, 2021

merged

@brusdev brusdev closed this Mar 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@feuillemorte feuillemorte feuillemorte approved these changes

Assignees
No one assigned
Labels
LTS-7.8 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
3 participants
@clebertsuconic @feuillemorte @brusdev