You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by jb...@apache.org on 2017/09/22 21:09:57 UTC

[geode-native] branch develop updated (594265d -> be340c3)

This is an automated email from the ASF dual-hosted git repository.

jbarrett pushed a change to branch develop
in repository https://gitbox.apache.org/repos/asf/geode-native.git.


    from 594265d  GEODE-3611: Upgrade boost dependency to version 1.65.1
     new cf16d30  GEODE-3674: Fix bug in usage of shared pointer
     new e3fd0a6  GEODE-3674: Workaround possible threading bug in testcase framework
     new be340c3  GEODE-3674: Remove testThinClientSecurityMultiUserTest from flaky list

The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 cppcache/integration-test/CMakeLists.txt                          | 1 -
 cppcache/integration-test/testThinClientSecurityMultiUserTest.cpp | 4 ++--
 cppcache/src/ProxyCache.cpp                                       | 3 +--
 3 files changed, 3 insertions(+), 5 deletions(-)

-- 
To stop receiving notification emails like this one, please contact
['"commits@geode.apache.org" <co...@geode.apache.org>'].

[geode-native] 02/03: GEODE-3674: Workaround possible threading bug in testcase framework

Posted by jb...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

jbarrett pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode-native.git

commit e3fd0a6e43f0e307e406ae31eccf90f4d22df8da
Author: David Kimura <dk...@pivotal.io>
AuthorDate: Wed Sep 20 13:36:50 2017 -0700

    GEODE-3674: Workaround possible threading bug in testcase framework
---
 cppcache/integration-test/testThinClientSecurityMultiUserTest.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/cppcache/integration-test/testThinClientSecurityMultiUserTest.cpp b/cppcache/integration-test/testThinClientSecurityMultiUserTest.cpp
index 75a741b..f0a1972 100644
--- a/cppcache/integration-test/testThinClientSecurityMultiUserTest.cpp
+++ b/cppcache/integration-test/testThinClientSecurityMultiUserTest.cpp
@@ -359,8 +359,8 @@ DUNIT_TASK_DEFINITION(CLIENT_1, StepOne)
                               false, -1, true, 0);
       LOG("Region created successfully");
       PoolPtr pool = getPool(regionNamesAuth[0]);
-      int m_numberOfUsers = 10;
-      int m_numberOfOps = 100;
+      int m_numberOfUsers = 3;
+      int m_numberOfOps = 10;
       UserThread* uthreads = new UserThread[m_numberOfUsers];
 
       for (int i = 0; i < m_numberOfUsers; i++) {

-- 
To stop receiving notification emails like this one, please contact
"commits@geode.apache.org" <co...@geode.apache.org>.

[geode-native] 03/03: GEODE-3674: Remove testThinClientSecurityMultiUserTest from flaky list

Posted by jb...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

jbarrett pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode-native.git

commit be340c305bd5021bab573c95071e17965279a883
Author: David Kimura <dk...@pivotal.io>
AuthorDate: Wed Sep 20 14:28:46 2017 -0700

    GEODE-3674: Remove testThinClientSecurityMultiUserTest from flaky list
---
 cppcache/integration-test/CMakeLists.txt | 1 -
 1 file changed, 1 deletion(-)

diff --git a/cppcache/integration-test/CMakeLists.txt b/cppcache/integration-test/CMakeLists.txt
index 0df3bcb..dd43783 100644
--- a/cppcache/integration-test/CMakeLists.txt
+++ b/cppcache/integration-test/CMakeLists.txt
@@ -174,7 +174,6 @@ set_property(TEST testThinClientPutAll PROPERTY LABELS FLAKY)
 set_property(TEST testThinClientPutAllPRSingleHop PROPERTY LABELS FLAKY)
 set_property(TEST testThinClientPutAllWithCallBackArgWithoutConcurrency PROPERTY LABELS FLAKY)
 set_property(TEST testThinClientSecurityCQAuthorizationMU PROPERTY LABELS FLAKY)
-set_property(TEST testThinClientSecurityMultiUserTest PROPERTY LABELS FLAKY)
 set_property(TEST testThinClientTXFailover PROPERTY LABELS FLAKY)
 set_property(TEST testThinClientTransactionsWithSticky PROPERTY LABELS FLAKY)
 set_property(TEST testThinClientTransactionsXA PROPERTY LABELS FLAKY)

-- 
To stop receiving notification emails like this one, please contact
"commits@geode.apache.org" <co...@geode.apache.org>.

[geode-native] 01/03: GEODE-3674: Fix bug in usage of shared pointer

Posted by jb...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

jbarrett pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode-native.git

commit cf16d3071cb92bce3e87238146927316a67db1fd
Author: David Kimura <dk...@pivotal.io>
AuthorDate: Wed Sep 20 13:25:40 2017 -0700

    GEODE-3674: Fix bug in usage of shared pointer
---
 cppcache/src/ProxyCache.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/cppcache/src/ProxyCache.cpp b/cppcache/src/ProxyCache.cpp
index 2d03a8b..d304609 100644
--- a/cppcache/src/ProxyCache.cpp
+++ b/cppcache/src/ProxyCache.cpp
@@ -63,8 +63,7 @@ void ProxyCache::close() {
       prqs->closeCqs(false);
     }
 
-    ProxyCachePtr pcp(this);
-    GuardUserAttribures gua(pcp);
+    GuardUserAttribures gua(shared_from_this());
     m_isProxyCacheClosed = true;
     m_userAttributes->unSetCredentials();
     // send message to server

-- 
To stop receiving notification emails like this one, please contact
"commits@geode.apache.org" <co...@geode.apache.org>.