From fa70213aa136ddb50e0076ac74c2dd229c0aaf18 Mon Sep 17 00:00:00 2001 From: Mike Martell Date: Thu, 14 Apr 2022 06:56:12 -0700 Subject: [PATCH] Remove no-ack from testThinClientTransactionsWithSticky --- .../ThinClientTransactionsWithSticky.hpp | 974 ++++++++++++++++++ .../testThinClientTransactionsWithSticky.cpp | 2 +- 2 files changed, 975 insertions(+), 1 deletion(-) create mode 100644 cppcache/integration-test/ThinClientTransactionsWithSticky.hpp diff --git a/cppcache/integration-test/ThinClientTransactionsWithSticky.hpp b/cppcache/integration-test/ThinClientTransactionsWithSticky.hpp new file mode 100644 index 0000000000..7bd9d8f2a7 --- /dev/null +++ b/cppcache/integration-test/ThinClientTransactionsWithSticky.hpp @@ -0,0 +1,974 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +#pragma once + +#ifndef GEODE_INTEGRATION_TEST_THINCLIENTTRANSACTIONS_H_ +#define GEODE_INTEGRATION_TEST_THINCLIENTTRANSACTIONS_H_ + +#include "fw_dunit.hpp" + +#include + +#include +#include + +#define ROOT_NAME "ThinClientTransactions" +#define ROOT_SCOPE DISTRIBUTED_ACK + +#include "CacheHelper.hpp" +#include "util/concurrent/binary_semaphore.hpp" + +namespace { // NOLINT(google-build-namespaces) + +using apache::geode::client::binary_semaphore; +using apache::geode::client::CacheableKey; +using apache::geode::client::CacheableString; +using apache::geode::client::CacheHelper; +using apache::geode::client::CacheServerException; +using apache::geode::client::EntryExistsException; +using apache::geode::client::EntryNotFoundException; +using apache::geode::client::IllegalStateException; +using apache::geode::client::Properties; +using apache::geode::client::TransactionException; +using apache::geode::client::TransactionId; + +#define CLIENT1 s1p1 +#define CLIENT2 s1p2 +#define SERVER1 s2p1 +#define SERVER2 s2p2 +#define CREATE_TWICE_KEY "__create_twice_key" +#define CREATE_TWICE_VALUE "__create_twice_value" +CacheHelper* cacheHelper = nullptr; +static bool isLocalServer = false; +static bool isLocator = false; +static int numberOfLocators = 0; + +const std::string locatorsG = + CacheHelper::getLocatorHostPort(isLocator, isLocalServer, numberOfLocators); + +void initClient(const bool isthinClient) { + if (cacheHelper == nullptr) { + auto config = Properties::create(); + config->insert("suspended-tx-timeout", std::chrono::minutes(1)); + cacheHelper = new CacheHelper(isthinClient, config); + } + ASSERT(cacheHelper, "Failed to create a CacheHelper client instance."); +} + +void cleanProc() { + if (cacheHelper != nullptr) { + delete cacheHelper; + cacheHelper = nullptr; + } +} + +CacheHelper* getHelper() { + ASSERT(cacheHelper != nullptr, "No cacheHelper initialized."); + return cacheHelper; +} + +void _verifyEntry(const char* name, const char* key, const char* val, + bool noKey) { + // Verify key and value exist in this region, in this process. + const char* value = val ? val : ""; + std::string msg; + if (noKey) { + msg = + std::string("Verify key ") + key + " does not exist in region " + name; + } else if (!val) { + msg = std::string("Verify value for key ") + key + + " does not exist in region " + name; + } else { + msg = std::string("Verify value for key ") + key + " is: " + value + + " in region " + name; + } + LOG(msg); + + auto regPtr = getHelper()->getRegion(name); + ASSERT(regPtr != nullptr, "Region not found."); + + auto keyPtr = CacheableKey::create(key); + + // if the region is no ack, then we may need to wait... + if (noKey == false) { // need to find the key! + ASSERT(regPtr->containsKey(keyPtr), "Key not found in region."); + } + if (val != nullptr) { // need to have a value! + ASSERT(regPtr->containsValueForKey(keyPtr), "Value not found in region."); + } + + // loop up to MAX times, testing condition + uint32_t MAX = 100; + uint32_t SLEEP = 10; // milliseconds + uint32_t containsKeyCnt = 0; + uint32_t containsValueCnt = 0; + uint32_t testValueCnt = 0; + + for (int i = MAX; i >= 0; i--) { + if (noKey) { + if (regPtr->containsKey(keyPtr)) { + containsKeyCnt++; + } else { + break; + } + ASSERT(containsKeyCnt < MAX, "Key found in region."); + } + if (val == nullptr) { + if (regPtr->containsValueForKey(keyPtr)) { + containsValueCnt++; + } else { + break; + } + ASSERT(containsValueCnt < MAX, "Value found in region."); + } + + if (val != nullptr) { + auto checkPtr = + std::dynamic_pointer_cast(regPtr->get(keyPtr)); + + ASSERT(checkPtr != nullptr, "Value Ptr should not be null."); + LOG("In verify loop, get returned " + checkPtr->value() + " for key " + + key); + + if (strcmp(checkPtr->value().c_str(), value) != 0) { + testValueCnt++; + } else { + break; + } + ASSERT(testValueCnt < MAX, "Incorrect value found."); + } + dunit::sleep(SLEEP); + } +} + +#define verifyEntry(x, y, z) _verifyEntry(x, y, z, __LINE__) + +void _verifyEntry(const char* name, const char* key, const char* val, + int line) { + LOG(std::string("verifyEntry() called from ") + std::to_string(line) + "\n"); + _verifyEntry(name, key, val, false); + LOG("Entry verified."); +} + +void createRegion(const char* name, bool ackMode, const char* endpoints, + bool clientNotificationEnabled = false, + bool cachingEnable = true) { + LOG("createRegion() entered."); + std::cout << "Creating region -- " << name << " ackMode is " << ackMode + << "\n" + << std::flush; + auto regPtr = getHelper()->createRegion(name, ackMode, cachingEnable, nullptr, + endpoints, clientNotificationEnabled); + ASSERT(regPtr != nullptr, "Failed to create region."); + LOG("Region created."); +} +void createPooledRegion(const std::string& name, bool ackMode, + const std::string& locators, + const std::string& poolname, + bool clientNotificationEnabled = false, + bool cachingEnable = true) { + LOG("createRegion_Pool() entered."); + std::cout << "Creating region -- " << name << " ackMode is " << ackMode + << "\n" + << std::flush; + auto regPtr = + getHelper()->createPooledRegion(name, ackMode, locators, poolname, + cachingEnable, clientNotificationEnabled); + ASSERT(regPtr != nullptr, "Failed to create region."); + LOG("Pooled Region created."); +} + +void createPooledRegionSticky(const std::string& name, bool ackMode, + const std::string& locators, + const std::string& poolname, + bool clientNotificationEnabled = false, + bool cachingEnable = true) { + LOG("createRegion_Pool() entered."); + std::cout << "Creating region -- " << name << " ackMode is " << ackMode + << "\n" + << std::flush; + auto regPtr = getHelper()->createPooledRegionSticky( + name, ackMode, locators, poolname, cachingEnable, + clientNotificationEnabled); + ASSERT(regPtr != nullptr, "Failed to create region."); + LOG("Pooled Region created."); +} + +void createEntry(const char* name, const char* key, const char* value) { + LOG("createEntry() entered."); + std::cout << "Creating entry -- key: " << key << " value: " << value + << " in region " << name << "\n" + << std::flush; + // Create entry, verify entry is correct + auto keyPtr = CacheableKey::create(key); + auto valPtr = CacheableString::create(value); + + auto regPtr = getHelper()->getRegion(name); + ASSERT(regPtr != nullptr, "Region not found."); + + ASSERT(!regPtr->containsKey(keyPtr), + "Key should not have been found in region."); + ASSERT(!regPtr->containsValueForKey(keyPtr), + "Value should not have been found in region."); + + // regPtr->create( keyPtr, valPtr ); + regPtr->put(keyPtr, valPtr); + LOG("Created entry."); + + // verifyEntry( name, key, value ); + LOG("Entry created."); +} +void createEntryTwice(const char* name, const char* key, const char* value) { + LOG("createEntryTwice() entered."); + LOG(std::string("Creating entry -- key: ") + key + " value : " + value + + " in region " + name); + auto keyPtr = CacheableKey::create(key); + auto valPtr = CacheableString::create(value); + auto regPtr = getHelper()->getRegion(name); + regPtr->create(keyPtr, valPtr); + try { + regPtr->create(keyPtr, valPtr); + } catch (const EntryExistsException& geodeExcp) { + LOG(geodeExcp.what()); + LOG("createEntryTwice() Clean Exit."); + return; + } + ASSERT(false, + "Creating key twice is not allowed and while doing that exception was " + "not thrown"); + return; // This return will never reach +} + +void updateEntry(const char* name, const char* key, const char* value) { + LOG("updateEntry() entered."); + std::cout << "Updating entry -- key: " << key << " value: " << value + << " in region " << name << "\n" + << std::flush; + + // Update entry, verify entry is correct + auto keyPtr = CacheableKey::create(key); + auto valPtr = CacheableString::create(value); + + auto regPtr = getHelper()->getRegion(name); + ASSERT(regPtr != nullptr, "Region not found."); + + ASSERT(regPtr->containsKey(keyPtr), "Key should have been found in region."); + ASSERT(regPtr->containsValueForKey(keyPtr), + "Value should have been found in region."); + + regPtr->put(keyPtr, valPtr); + LOG("Put entry."); + + verifyEntry(name, key, value); + LOG("Entry updated."); +} + +void doGetAgain(const char* name, const char* key, const char* value) { + LOG("doGetAgain() entered."); + std::cout << "Netsearching for entry -- key: " << key << " value: " << value + << " in region " << name << "\n" + << std::flush; + + // Get entry created in Process A, verify entry is correct + auto keyPtr = CacheableKey::create(key); + + auto regPtr = getHelper()->getRegion(name); + std::cout << "netsearch region " << regPtr->getName() << "\n" << std::flush; + + ASSERT(regPtr != nullptr, "Region not found."); + + auto checkPtr = std::dynamic_pointer_cast( + regPtr->get(keyPtr)); // force a netsearch + + if (checkPtr != nullptr) { + LOG("checkPtr is not null"); + LOG(std::string("In doGetAgain, get returned ") + checkPtr->value() + + " for key " + key); + } else { + LOG("checkPtr is nullptr"); + } + verifyEntry(name, key, value); + LOG("GetAgain complete."); +} + +void doNetsearch(const char* name, const char* key, const char* value) { + LOG("doNetsearch() entered."); + std::cout << "Netsearching for entry -- key: " << key + << " expecting value: " << value << " in region " << name << "\n" + << std::flush; + + static int count = 0; + // Get entry created in Process A, verify entry is correct + auto keyPtr = CacheableKey::create(key); + + auto regPtr = getHelper()->getRegion(name); + std::cout << "netsearch region " << regPtr->getName() << "\n" << std::flush; + + ASSERT(regPtr != nullptr, "Region not found."); + + if (count == 0) { + ASSERT(!regPtr->containsKey(keyPtr), + "Key should not have been found in region."); + ASSERT(!regPtr->containsValueForKey(keyPtr), + "Value should not have been found in region."); + count++; + } + auto checkPtr = std::dynamic_pointer_cast( + regPtr->get(keyPtr)); // force a netsearch + + if (checkPtr != nullptr) { + LOG("checkPtr is not null"); + LOG(std::string("In net search, get returned ") + checkPtr->value() + + " for key " + key); + } else { + LOG("checkPtr is nullptr"); + } + verifyEntry(name, key, value); + LOG("Netsearch complete."); +} + +const char* keys[] = {"Key-1", "Key-2", "Key-3", "Key-4", + "Key-5", "Key-6", "Key-7"}; +const char* vals[] = {"Value-1", "Value-2", "Value-3", "Value-4", + "Value-5", "Value-6", "Value-7"}; +const char* nvals[] = {"New Value-1", "New Value-2", "New Value-3", + "New Value-4", "New Value-5", "New Value-6", + "New Value-7"}; + +const char* regionNames[] = {"DistRegionAck", "testregion"}; + +const bool USE_ACK = true; +const bool NO_ACK = false; +#include "LocatorHelper.hpp" +#define THREADERRORCHECK(x, y) \ + do { \ + if (!(x)) { \ + failed_ = true; \ + error_ = y; \ + return; \ + } \ + } while (0) + +class SuspendTransactionThread { + public: + SuspendTransactionThread(bool sleep, binary_semaphore& event) + : sleep_{sleep}, sem_{event} {} + + void run() { + LOG(" In SuspendTransactionThread"); + auto txManager = getHelper()->getCache()->getCacheTransactionManager(); + + txManager->begin(); + createEntry(regionNames[0], keys[4], vals[4]); + + tx_ = &txManager->getTransactionId(); + + if (sleep_) { + sem_.acquire(); + + std::this_thread::sleep_for(std::chrono::seconds{5}); + } + + tx_ = &txManager->suspend(); + LOG(" Out SuspendTransactionThread"); + + getHelper() + ->getCache() + ->getPoolManager() + .find("__TESTPOOL1_") + ->releaseThreadLocalConnection(); + } + + void start() { + thread_ = std::thread{[this]() { run(); }}; + } + + void stop() { + if (thread_.joinable()) { + thread_.join(); + } + } + + TransactionId& getSuspendedTx() { return *tx_; } + + protected: + bool sleep_; + std::thread thread_; + binary_semaphore& sem_; + TransactionId* tx_{nullptr}; +}; + +class ResumeTransactionThread { + public: + + ResumeTransactionThread(TransactionId& tx, bool commit, bool sleep, + binary_semaphore& event) + : tx_(tx), commit_{commit}, sleep_(sleep), failed_{false}, sem_{event} {} + + void run() { + LOG("In ResumeTransactionThread"); + + auto regPtr0 = getHelper()->getRegion(regionNames[0]); + THREADERRORCHECK(regPtr0 != nullptr, + "In ResumeTransactionThread - Region not found."); + + auto keyPtr4 = CacheableKey::create(keys[4]); + auto keyPtr5 = CacheableKey::create(keys[5]); + auto keyPtr6 = CacheableKey::create(keys[6]); + + THREADERRORCHECK(!regPtr0->containsKeyOnServer(keyPtr4), + "In ResumeTransactionThread - Key should not have been " + "found in region."); + + auto txManager = getHelper()->getCache()->getCacheTransactionManager(); + if (sleep_) { + THREADERRORCHECK(!txManager->isSuspended(tx_), + "In ResumeTransactionThread - the transaction should " + "NOT be in suspended state"); + } else { + THREADERRORCHECK(txManager->isSuspended(tx_), + "In ResumeTransactionThread - the transaction should be " + "in suspended state"); + } + + THREADERRORCHECK( + txManager->exists(tx_), + "In ResumeTransactionThread - the transaction should exist"); + + if (sleep_) { + sem_.release(); + txManager->tryResume(tx_, std::chrono::seconds(30)); + } else { + txManager->resume(tx_); + } + + THREADERRORCHECK( + regPtr0->containsKeyOnServer(keyPtr4), + "In ResumeTransactionThread - Key should have been found in region."); + + if (commit_) { + txManager->commit(); + THREADERRORCHECK( + regPtr0->containsKeyOnServer(keyPtr4), + "In ResumeTransactionThread - Key should have been found in region."); + + } else { + txManager->rollback(); + THREADERRORCHECK(!regPtr0->containsKeyOnServer(keyPtr4), + "In ResumeTransactionThread - Key should not have been " + "found in region."); + } + + if (commit_) { + regPtr0->destroy(keyPtr4); + + THREADERRORCHECK(!regPtr0->containsKeyOnServer(keyPtr4), + "In ResumeTransactionThread - Key should not have been " + "found in region."); + } else { + try { + regPtr0->destroy(keyPtr4); + FAIL("Should have got EntryNotFoundException for keyPtr4"); + } catch (EntryNotFoundException& /*ex*/) { + LOG("Got expected EntryNotFoundException for keyPtr4"); + } + } + + getHelper() + ->getCache() + ->getPoolManager() + .find("__TESTPOOL1_") + ->releaseThreadLocalConnection(); + LOG(" Out ResumeTransactionThread"); + } + + void start() { + thread_ = std::thread{[this]() { run(); }}; + } + + void stop() { + if (thread_.joinable()) { + thread_.join(); + } + } + + bool isFailed() { return failed_; } + + std::string getError() { return error_; } + + protected: + TransactionId& tx_; + bool commit_; + bool sleep_; + bool failed_; + std::string error_; + + std::thread thread_; + binary_semaphore& sem_; +}; + +DUNIT_TASK_DEFINITION(SERVER1, CreateServer1) + { + if (isLocalServer) CacheHelper::initServer(1); + LOG("SERVER1 started"); + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT1, SuspendResumeCommit) + { + auto txManager = getHelper()->getCache()->getCacheTransactionManager(); + auto regPtr0 = getHelper()->getRegion(regionNames[0]); + ASSERT(regPtr0 != nullptr, "In SuspendResumeCommit - Region not found."); + auto keyPtr4 = CacheableKey::create(keys[4]); + auto keyPtr5 = CacheableKey::create(keys[5]); + auto keyPtr6 = CacheableKey::create(keys[6]); + + txManager->begin(); + createEntry(regionNames[0], keys[4], vals[4]); + auto& m_suspendedTransaction = txManager->suspend(); + + ASSERT( + !regPtr0->containsKeyOnServer(keyPtr4), + "In SuspendResumeCommit - Key should not have been found in region."); + + ASSERT(txManager->isSuspended(m_suspendedTransaction), + "In SuspendResumeCommit - the transaction should be in suspended " + "state"); + ASSERT(txManager->exists(m_suspendedTransaction), + "In SuspendResumeCommit - the transaction should exist"); + + txManager->resume(m_suspendedTransaction); + + ASSERT( + !txManager->tryResume(m_suspendedTransaction), + "SuspendResumeRollback: Transaction shouldnt have been resumed again"); + ASSERT(regPtr0->containsKeyOnServer(keyPtr4), + "In SuspendResumeCommit - Key should have been found in region."); + + txManager->commit(); + ASSERT(regPtr0->containsKeyOnServer(keyPtr4), + "In SuspendResumeCommit - Key should have been found in region."); + + regPtr0->destroy(keyPtr4); + + ASSERT(!txManager->isSuspended(m_suspendedTransaction), + "In SuspendResumeCommit the transaction should NOT present"); + ASSERT(!txManager->exists(m_suspendedTransaction), + "In SuspendResumeCommit - the transaction should NOT exist"); + ASSERT(!txManager->tryResume(m_suspendedTransaction), + "In SuspendResumeCommit - the transaction should NOT have been " + "resumed"); + + bool resumeExc = false; + try { + txManager->resume(m_suspendedTransaction); + } catch (const IllegalStateException&) { + resumeExc = true; + } + + ASSERT(resumeExc, + "SuspendResumeCommit: Transaction shouldnt have been resumed"); + + bool threwTransactionException = false; + try { + txManager->suspend(); + } catch (const TransactionException&) { + threwTransactionException = true; + } + ASSERT(threwTransactionException, + "SuspendResumeCommit: Transaction shouldnt have been suspended"); + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT1, SuspendTimeOut) + { + auto txManager = getHelper()->getCache()->getCacheTransactionManager(); + auto keyPtr4 = CacheableKey::create(keys[4]); + auto keyPtr5 = CacheableKey::create(keys[5]); + + auto regPtr0 = getHelper()->getRegion(regionNames[0]); + ASSERT(regPtr0 != nullptr, "In SuspendTimeOut - Region not found."); + + txManager->begin(); + createEntry(regionNames[0], keys[4], vals[4]); + auto& tid1 = txManager->suspend(); + + txManager->begin(); + createEntry(regionNames[0], keys[5], vals[5]); + auto& tid2 = txManager->suspend(); + + txManager->resume(tid1); + createEntry(regionNames[0], keys[6], vals[6]); + txManager->commit(); + + ASSERT(txManager->isSuspended(tid2), + "In SuspendTimeOut the transaction should be present"); + ASSERT(txManager->exists(tid2), + "In SuspendTimeOut - the transaction should exist"); + + std::this_thread::sleep_for(std::chrono::seconds(65)); + ASSERT(!txManager->tryResume(tid2), + "In SuspendTimeOut - the transaction should NOT have been resumed"); + ASSERT(!txManager->isSuspended(tid2), + "In SuspendTimeOut the transaction should NOT present"); + ASSERT(!txManager->exists(tid2), + "In SuspendTimeOut - the transaction should NOT exist"); + ASSERT(regPtr0->containsKeyOnServer(keyPtr4), + "In SuspendTimeOut - Key should have been found in region."); + ASSERT(!regPtr0->containsKeyOnServer(keyPtr5), + "In SuspendTimeOut - Key should not have been found in region."); + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT1, SuspendResumeRollback) + { + auto txManager = getHelper()->getCache()->getCacheTransactionManager(); + auto keyPtr4 = CacheableKey::create(keys[4]); + auto keyPtr5 = CacheableKey::create(keys[5]); + auto keyPtr6 = CacheableKey::create(keys[6]); + + auto regPtr0 = getHelper()->getRegion(regionNames[0]); + ASSERT(regPtr0 != nullptr, "In SuspendResumeRollback - Region not found."); + + txManager->begin(); + createEntry(regionNames[0], keys[4], vals[4]); + auto& m_suspendedTransaction = txManager->suspend(); + + ASSERT( + !regPtr0->containsKeyOnServer(keyPtr4), + "In SuspendResumeRollback - Key should not have been found in region."); + + ASSERT(txManager->isSuspended(m_suspendedTransaction), + "In SuspendResumeRollback the transaction should be in suspended " + "state"); + ASSERT(txManager->exists(m_suspendedTransaction), + "In SuspendResumeRollback - the transaction should exist"); + + txManager->resume(m_suspendedTransaction); + + ASSERT(regPtr0->containsKeyOnServer(keyPtr4), + "In SuspendResumeRollback - Key should have been found in region."); + + txManager->rollback(); + ASSERT( + !regPtr0->containsKeyOnServer(keyPtr4), + "In SuspendResumeRollback - Key should not have been found in region."); + + try { + regPtr0->destroy(keyPtr4); + FAIL("Should have got EntryNotFoundException for keyPtr4"); + } catch (EntryNotFoundException& /*ex*/) { + LOG("Got expected EntryNotFoundException for keyPtr4"); + } + + ASSERT(!txManager->isSuspended(m_suspendedTransaction), + "In SuspendResumeRollback the transaction should NOT present"); + ASSERT(!txManager->exists(m_suspendedTransaction), + "In SuspendResumeRollback - the transaction should NOT exist"); + ASSERT(!txManager->tryResume(m_suspendedTransaction), + "In SuspendResumeRollback - the transaction should NOT have been " + "resumed"); + bool resumeExc = false; + try { + txManager->resume(m_suspendedTransaction); + } catch (const IllegalStateException&) { + resumeExc = true; + } + + ASSERT(resumeExc, + "SuspendResumeRollback: Transaction shouldnt have been resumed"); + } +END_TASK_DEFINITION +DUNIT_TASK_DEFINITION(CLIENT1, SuspendResumeInThread) + { + LOG("Start suspend thread and resume thread and rollback immedidately"); + + { + binary_semaphore event{0}; + SuspendTransactionThread suspend{false, event}; + suspend.start(); + + std::this_thread::sleep_for(std::chrono::seconds{2}); + ResumeTransactionThread resume{suspend.getSuspendedTx(), false, false, + event}; + resume.start(); + + suspend.stop(); + resume.stop(); + + ASSERT(!resume.isFailed(), resume.getError()); + } + + LOG("Start suspend thread and resume thread and commit immedidately"); + { + binary_semaphore event{0}; + SuspendTransactionThread suspend{false, event}; + + suspend.start(); + std::this_thread::sleep_for(std::chrono::seconds{2}); + ResumeTransactionThread resume{suspend.getSuspendedTx(), true, false, + event}; + resume.start(); + + suspend.stop(); + resume.stop(); + ASSERT(!resume.isFailed(), resume.getError()); + } + + LOG("Start suspend thread and tryresume thread with rollback. make " + "tryResume to sleep"); + { + binary_semaphore event{0}; + SuspendTransactionThread suspend{true, event}; + suspend.start(); + + std::this_thread::sleep_for(std::chrono::seconds{2}); + ResumeTransactionThread resume{suspend.getSuspendedTx(), false, true, + event}; + resume.start(); + + suspend.stop(); + resume.stop(); + + ASSERT(!resume.isFailed(), resume.getError()); + } + + LOG("Start suspend thread and tryresume thread with commit. make " + "tryResume to sleep"); + + { + binary_semaphore event{0}; + SuspendTransactionThread suspend{true, event}; + suspend.start(); + + std::this_thread::sleep_for(std::chrono::seconds{2}); + LOG("suspendTh->activate();"); + + ResumeTransactionThread resume{suspend.getSuspendedTx(), true, true, + event}; + resume.start(); + + suspend.stop(); + resume.stop(); + + ASSERT(!resume.isFailed(), resume.getError()); + } + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT1, CreateNonexistentServerRegion_Pooled_Locator) + { + initClient(true); + createPooledRegion("non-region", USE_ACK, locatorsG, "__TESTPOOL1_"); + try { + createEntry("non-region", keys[0], vals[0]); + FAIL( + "Expected exception when doing operations on a non-existent region."); + } catch (const CacheServerException& ex) { + std::cout << "Got expected CacheServerException when performing operation on a non-existent region: " << ex.what() << "\n"; + } + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT1, + CreateNonexistentServerRegion_Pooled_Locator_Sticky) + { + initClient(true); + createPooledRegionSticky("non-region", USE_ACK, locatorsG, "__TESTPOOL1_"); + try { + createEntry("non-region", keys[0], vals[0]); + FAIL( + "Expected exception when doing operations on a non-existent region."); + } catch (const CacheServerException& ex) { + std::cout << "Got expected CacheServerException when performing operation on a non-existent region: " << ex.what() << "\n"; + } + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT1, CreateClient1PooledRegionWithoutSticky) + { + createPooledRegion(regionNames[0], USE_ACK, locatorsG, "__TESTPOOL1_"); + LOG("StepOne_Pooled complete."); + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT1, CreateClient1PooledRegionWithSticky) + { + createPooledRegionSticky(regionNames[0], USE_ACK, locatorsG, + "__TESTPOOL1_"); + LOG("StepOne_Pooled_Locator_Sticky complete."); + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT2, CreateClient2PooledRegionWithoutSticky) + { + initClient(true); + createPooledRegion(regionNames[0], USE_ACK, locatorsG, "__TESTPOOL1_"); + LOG("StepTwo complete."); + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT2, CreateClient2PooledRegionWithSticky) + { + initClient(true); + createPooledRegionSticky(regionNames[0], USE_ACK, locatorsG, + "__TESTPOOL1_"); + LOG("StepTwo complete."); + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT1, CreateClient1Entries) + { + auto txManager = getHelper()->getCache()->getCacheTransactionManager(); + txManager->begin(); + createEntry(regionNames[0], keys[0], vals[0]); + txManager->commit(); + LOG("StepThree complete."); + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT2, CreateClient2Entries) + { + doNetsearch(regionNames[0], keys[0], vals[0]); + auto txManager = getHelper()->getCache()->getCacheTransactionManager(); + txManager->begin(); + createEntry(regionNames[0], keys[1], vals[1]); + txManager->commit(); + verifyEntry(regionNames[0], keys[1], vals[1]); + LOG("StepFour complete."); + } +END_TASK_DEFINITION +DUNIT_TASK_DEFINITION(CLIENT1, UpdateClient1Entries) + { + auto reg0 = getHelper()->getRegion(regionNames[0]); + auto vec0 = reg0->serverKeys(); + ASSERT(vec0.size() == 2, "Should have 2 keys in first region."); + std::string key0, key1; + key0 = vec0[0]->toString().c_str(); + key1 = vec0[1]->toString().c_str(); + ASSERT(key0 != key1, "The two keys should be different in first region."); + ASSERT(key0 == keys[0] || key0 == keys[1], + "Unexpected key in first region."); + ASSERT(key1 == keys[0] || key1 == keys[1], + "Unexpected key in first region."); + + doNetsearch(regionNames[0], keys[1], vals[1]); + updateEntry(regionNames[0], keys[0], nvals[0]); + LOG("StepFive complete."); + } +END_TASK_DEFINITION +DUNIT_TASK_DEFINITION(CLIENT2, UpdateClient2Entries) + { + doNetsearch(regionNames[0], keys[0], vals[0]); + auto txManager = getHelper()->getCache()->getCacheTransactionManager(); + txManager->begin(); + updateEntry(regionNames[0], keys[1], nvals[1]); + txManager->commit(); + LOG("StepSix complete."); + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT1, CreateClient1EntryTwice) + { createEntryTwice(regionNames[0], CREATE_TWICE_KEY, CREATE_TWICE_VALUE); } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT1, CreateClient1KeyThriceWithoutSticky) + { + createPooledRegion(regionNames[1], NO_ACK, locatorsG, "__TESTPOOL1_", false, + false); + auto reg = getHelper()->getRegion(regionNames[1]); + LOG("REGION Created with Caching Enabled false"); + auto keyPtr = CacheableKey::create(CREATE_TWICE_KEY); + auto valPtr = CacheableString::create(CREATE_TWICE_VALUE); + try { + reg->create(keyPtr, valPtr); + char message[200]; + LOG(std::string("First create on Key ") + CREATE_TWICE_KEY); + LOG(message); + reg->create(keyPtr, valPtr); + LOG(std::string("Second create on Key ") + CREATE_TWICE_KEY); + LOG(message); + reg->create(keyPtr, valPtr); + LOG(std::string("Third create on Key ") + CREATE_TWICE_KEY); + LOG(message); + } catch (const EntryExistsException& geodeExcp) { + LOG(geodeExcp.what()); + ASSERT(false, + "Creating KEY Twice on a caching-enabled false region should be " + "allowed."); + } + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT1, CreateClient1KeyThriceWithSticky) + { + createPooledRegionSticky(regionNames[1], NO_ACK, locatorsG, "__TESTPOOL1_", + false, false); + auto reg = getHelper()->getRegion(regionNames[1]); + LOG("REGION Created with Caching Enabled false"); + auto keyPtr = CacheableKey::create(CREATE_TWICE_KEY); + auto valPtr = CacheableString::create(CREATE_TWICE_VALUE); + + auto reg0 = getHelper()->getRegion(regionNames[0]); + reg0->localInvalidate(CacheableKey::create(keys[1])); + auto pool = getHelper()->getCache()->getPoolManager().find("__TESTPOOL1_"); + ASSERT(pool != nullptr, "Pool Should have been found"); + doNetsearch(regionNames[0], keys[1], nvals[1]); + pool->releaseThreadLocalConnection(); + updateEntry(regionNames[0], keys[0], nvals[0]); + pool->releaseThreadLocalConnection(); + try { + reg->create(keyPtr, valPtr); + char message[200]; + LOG(std::string("First create on Key ") + CREATE_TWICE_KEY); + LOG(message); + reg->create(keyPtr, valPtr); + LOG(std::string("Second create on Key ") + CREATE_TWICE_KEY); + LOG(message); + reg->create(keyPtr, valPtr); + LOG(std::string("Third create on Key ") + CREATE_TWICE_KEY); + LOG(message); + } catch (const EntryExistsException& geodeExcp) { + LOG(geodeExcp.what()); + ASSERT(false, + "Creating KEY Twice on a caching-enabled false region should be " + "allowed."); + } + } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT1, CloseCache1) + { cleanProc(); } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(CLIENT2, CloseCache2) + { cleanProc(); } +END_TASK_DEFINITION + +DUNIT_TASK_DEFINITION(SERVER1, CloseServer1) + { + if (isLocalServer) { + CacheHelper::closeServer(1); + LOG("SERVER1 stopped"); + } + } +END_TASK_DEFINITION + +} // namespace + +#endif // GEODE_INTEGRATION_TEST_THINCLIENTTRANSACTIONS_H_ diff --git a/cppcache/integration-test/testThinClientTransactionsWithSticky.cpp b/cppcache/integration-test/testThinClientTransactionsWithSticky.cpp index 7ab05b01cd..16f8c0ae79 100644 --- a/cppcache/integration-test/testThinClientTransactionsWithSticky.cpp +++ b/cppcache/integration-test/testThinClientTransactionsWithSticky.cpp @@ -15,7 +15,7 @@ * limitations under the License. */ -#include "ThinClientTransactions.hpp" +#include "ThinClientTransactionsWithSticky.hpp" DUNIT_MAIN {