You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by bb...@apache.org on 2022/02/11 19:21:20 UTC
[geode-native] branch develop updated: GEODE-100043: Solve TransactionCleaningTest errors (#923)
This is an automated email from the ASF dual-hosted git repository.
bbender pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode-native.git
The following commit(s) were added to refs/heads/develop by this push:
new cd38c21 GEODE-100043: Solve TransactionCleaningTest errors (#923)
cd38c21 is described below
commit cd38c21282c7a11a6329428d0da41020bd9f6294
Author: Mario Salazar de Torres <ma...@est.tech>
AuthorDate: Fri Feb 11 20:21:09 2022 +0100
GEODE-100043: Solve TransactionCleaningTest errors (#923)
- From time to time this test was failing with the error "Concurrent
modification in the cache". Thing is this test was not intended to
check concurrent modifications.
- In GEODE-9322 this test was modified to avoid a race-condition
whenever restarting the server. To do so, it was needed to enable
notification subscription and change the region shortcut to
CACHE_PROXY. In doing so, concurrent checks are also enabled, and due
to some strange race-condition this is occasionally leading to the exception.
- Concurrency checks were disabled for the region to avoid the
race-condition to happen at all
---
cppcache/integration/test/TransactionCleaningTest.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/cppcache/integration/test/TransactionCleaningTest.cpp b/cppcache/integration/test/TransactionCleaningTest.cpp
index 84ab120..0f078e5 100644
--- a/cppcache/integration/test/TransactionCleaningTest.cpp
+++ b/cppcache/integration/test/TransactionCleaningTest.cpp
@@ -65,6 +65,7 @@ std::shared_ptr<apache::geode::client::Region> setupRegion(
apache::geode::client::RegionShortcut::CACHING_PROXY)
.setPoolName("default")
.setCacheListener(listener)
+ .setConcurrencyChecksEnabled(false)
.create("region");
return region;
}