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

ISPN-10059 Off-heap clustered cache cannot start with segmentation di… #6779

Merged
merged 2 commits into from
Mar 21, 2019

Conversation

danberindei
Copy link
Member

…sabled

https://issues.jboss.org/browse/ISPN-10059

  • Implement addSegments/removeSegments in OffHeapDataContainer
  • Use the InternalDataContainerAdapter implementation in all
    non-segmented InternalDataContainer implementations
  • Only retry the join request on failure

…sabled

* Implement addSegments/removeSegments in OffHeapDataContainer
  and BoundedOffHeapDataContainer
* Use the InternalDataContainerAdapter implementation in all
  non-segmented InternalDataContainer implementations
* Only retry the join request on failure
* Skip tests that require segmentation when the feature is disabled
@danberindei danberindei force-pushed the ISPN-10059_offheapdatacontainer branch from d74de76 to 9db8bac Compare March 20, 2019 10:20
@danberindei
Copy link
Member Author

danberindei commented Mar 20, 2019

Updated to fix the retry, override InternalDataContainer methods in BoundedOffHeapDataContainer, and skip tests that require segmentation when the feature is disabled.

There's still a failure in ExpirationSingleFileStoreDistListenerFunctionalTest that might be related. Please run test suite with -Dorg.infinispan.feature.data-segmentation=false to verify.

@@ -57,4 +75,88 @@ public boolean containsKey(Object k) {
public void evict(WrappedBytes key) {
super.evict(0, key);
}

@Override
public void put(WrappedBytes key, WrappedBytes value, Metadata metadata) {
Copy link
Member

Choose a reason for hiding this comment

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

I wish this wasn't required, that we could reuse AbstractInternalDataContainer instead. But unfortunately these classes were written assuming they wouldn't be called in the wrong context, so I am fine with it.

@wburns
Copy link
Member

wburns commented Mar 20, 2019

Updated to fix the retry, override InternalDataContainer methods in BoundedOffHeapDataContainer, and skip tests that require segmentation when the feature is disabled.

There's still a failure in ExpirationSingleFileStoreDistListenerFunctionalTest that might be related. Please run test suite with -Dorg.infinispan.feature.data-segmentation=false to verify.

I can confirm it consistently fails for me as well.

@wburns wburns merged commit f5cf579 into infinispan:master Mar 21, 2019
@wburns
Copy link
Member

wburns commented Mar 21, 2019

Integrated into master, thanks @danberindei !

@wburns
Copy link
Member

wburns commented Mar 21, 2019

We still need a JIRA for max idle expiration being available in iteration for non segmented containers in a cluster.

@danberindei danberindei deleted the ISPN-10059_offheapdatacontainer branch March 21, 2019 18:09
@danberindei
Copy link
Member Author

I created https://issues.jboss.org/browse/ISPN-10074, although when I created it I thought the problem was with the segmented containers returning expired entries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants