You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2021/04/12 13:09:23 UTC

[GitHub] [geode-native] gaussianrecurrence commented on pull request #757: GEODE-8991: Implement cached region clean up

gaussianrecurrence commented on pull request #757:
URL: https://github.com/apache/geode-native/pull/757#issuecomment-817797516


   > Seeing this failure in tests:
   > 
   > ```
   > The following tests FAILED:
   > 	 50 - PdxTypeRegistryTest.cleanupOnClusterRestart (SEGFAULT)
   > Errors while running CTest
   > ```
   > 
   > Is this perhaps GEODE-8968? Maybe rebasing onto develop again will fix it.
   
   Sorry for the delay, been kind of busy lately. The problem was that this test was triggering the cleanup but the registered region did not have caching enabled. When caching is not enabled m_entries is nullptr and that was causing a coredump. I added a check to verify that if caching is not enabled for a region, don't execute the entries cleanup on it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org