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.

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

Paging Fixes #267

Merged

Conversation

clebertsuconic
Copy link
Contributor

@clebertsuconic clebertsuconic commented Aug 5, 2019

downstream: ENTMQBR-2730
test: org.apache.activemq.artemis.tests.extras.byteman.RaceOnCursorIteratorTest#testSkipLivePageCache,org.apache.activemq.artemis.tests.extras.byteman.RaceOnCursorIteratorTest#testSkipNullPageCache,org.apache.activemq.artemis.tests.integration.client.LargeMessageTest#testLargeMessageBodySize,org.apache.activemq.artemis.tests.integration.largemessage.ServerLargeMessageTest#testSendServerMessage,org.apache.activemq.artemis.tests.integration.paging.PagingTest#testPageEmptyFile,org.apache.activemq.artemis.tests.integration.paging.PagingTest#testLivePageCacheEvicted,org.apache.activemq.artemis.tests.integration.persistence.XmlImportExportTest#testLargeMessage,org.apache.activemq.artemis.tests.integration.server.ScaleDown3NodeTest#testBasicScaleDownInternal,org.apache.activemq.artemis.tests.unit.core.paging.impl.PageTest#testAdd,org.apache.activemq.artemis.tests.unit.core.paging.impl.PagingManagerImplTest#testPagingManager,org.apache.activemq.artemis.tests.unit.core.paging.impl.PagingStoreImplTest#testDepageMultiplePages,org.apache.activemq.artemis.tests.unit.core.paging.impl.PagingStoreImplTest#testWriteIncompletePage
component: Artemis
subcomponent: message_delivery
level: component
importance: medium
type: functional
subtype: compliance
verifies: AMQ-90

(cherry picked from commit fa46647)

downstream: ENTMQBR-2730
(cherry picked from commit 99158cc)

downstream: ENTMQBR-2730
(cherry picked from commit b92cde1)

downstream: ENTMQBR-2730
(cherry picked from commit dd62be1)

downstream: ENTMQBR-2730
(cherry picked from commit edace88)

downstream: ENTMQBR-2730
@clebertsuconic clebertsuconic added Engineering Review Needed 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 Aug 5, 2019
@clebertsuconic
Copy link
Contributor Author

It's impossible to bring these fixes in separate PRs.

@franz1981 if you could also please review I'm not missing any other commit.

Copy link
Contributor

@franz1981 franz1981 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

(cherry picked from commit aa5d76e)

downstream: ENTMQBR-2730
@clebertsuconic clebertsuconic merged commit 8ff4fa8 into rh-messaging:2.9.0.jbossorg-x Aug 6, 2019
clebertsuconic added a commit that referenced this pull request Aug 6, 2019
@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
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
Development

Successfully merging this pull request may close these issues.

None yet

5 participants