[geode-native] branch develop updated: GEODE-10291: Compile on ubuntu jammy (gcc/g++ 11.2.0) (#966)
This is an automated email from the ASF dual-hosted git repository. moleske 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 35913aeb9 GEODE-10291: Compile on ubuntu jammy (gcc/g++ 11.2.0) (#966) 35913aeb9 is described below commit 35913aeb93606738efad89d6408a3020eafd14c5 Author: M. Oleske AuthorDate: Thu Sep 15 12:25:03 2022 -0400 GEODE-10291: Compile on ubuntu jammy (gcc/g++ 11.2.0) (#966) Authored-by: M. Oleske --- cppcache/integration/test/PdxTypeRegistryTest.cpp | 1 + cppcache/test/ConnectionQueueTest.cpp | 1 + 2 files changed, 2 insertions(+) diff --git a/cppcache/integration/test/PdxTypeRegistryTest.cpp b/cppcache/integration/test/PdxTypeRegistryTest.cpp index f87fbd62b..a16b9e9a7 100644 --- a/cppcache/integration/test/PdxTypeRegistryTest.cpp +++ b/cppcache/integration/test/PdxTypeRegistryTest.cpp @@ -13,6 +13,7 @@ * See the License for the specific language governing permissions and * limitations under the License. */ +#include #include diff --git a/cppcache/test/ConnectionQueueTest.cpp b/cppcache/test/ConnectionQueueTest.cpp index cae1da265..b92c7e6dc 100644 --- a/cppcache/test/ConnectionQueueTest.cpp +++ b/cppcache/test/ConnectionQueueTest.cpp @@ -18,6 +18,7 @@ #include #include +#include #include
[geode-native] branch develop updated: GEODE-10131: Remove set but unused variables (#947)
This is an automated email from the ASF dual-hosted git repository. moleske 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 d46ed2a6c GEODE-10131: Remove set but unused variables (#947) d46ed2a6c is described below commit d46ed2a6c8b9148322a992a184807dac231d30a1 Author: M. Oleske AuthorDate: Sat Apr 9 08:42:17 2022 -0700 GEODE-10131: Remove set but unused variables (#947) * Remove set but unused variables - On AppleClang 13.1.6.13160021, the warning -Wunused-but-set-variable is appears as an error. This allows compilation on AppleClang on MacOS 12.3 Authored-by: M. Oleske --- cppcache/src/ClientMetadata.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/cppcache/src/ClientMetadata.cpp b/cppcache/src/ClientMetadata.cpp index dd090d686..881ea4690 100644 --- a/cppcache/src/ClientMetadata.cpp +++ b/cppcache/src/ClientMetadata.cpp @@ -50,10 +50,8 @@ ClientMetadata::ClientMetadata( "%zu ", totalNumBuckets, fpaSet->size()); if (!fpaSet->empty()) { - int totalFPABuckets = 0; for (int i = 0; i < static_cast(fpaSet->size()); i++) { std::vector attList; -totalFPABuckets += fpaSet->at(i)->getNumBuckets(); attList.push_back(fpaSet->at(i)->getNumBuckets()); attList.push_back(fpaSet->at(i)->getStartingBucketID()); m_fpaMap[fpaSet->at(i)->getPartitionName()] = attList;
[geode-native] branch develop updated: GEODE-10120: Bump ruby dependencies (#945)
This is an automated email from the ASF dual-hosted git repository. moleske 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 99df0109c GEODE-10120: Bump ruby dependencies (#945) 99df0109c is described below commit 99df0109cd434a22a3ddf0186c5150a7756f7bac Author: M. Oleske AuthorDate: Thu Apr 7 13:30:59 2022 -0700 GEODE-10120: Bump ruby dependencies (#945) * Bump addressable from 2.7.0 to 2.8.0 in /docs/geode-native-book-cpp Bumps [addressable](https://github.com/sporkmonger/addressable) from 2.7.0 to 2.8.0. - [Release notes](https://github.com/sporkmonger/addressable/releases) - [Changelog](https://github.com/sporkmonger/addressable/blob/main/CHANGELOG.md) - [Commits](https://github.com/sporkmonger/addressable/compare/addressable-2.7.0...addressable-2.8.0) --- updated-dependencies: - dependency-name: addressable dependency-type: indirect ... Signed-off-by: dependabot[bot] * Bump addressable from 2.7.0 to 2.8.0 in /docs/geode-native-book-dotnet Bumps [addressable](https://github.com/sporkmonger/addressable) from 2.7.0 to 2.8.0. - [Release notes](https://github.com/sporkmonger/addressable/releases) - [Changelog](https://github.com/sporkmonger/addressable/blob/main/CHANGELOG.md) - [Commits](https://github.com/sporkmonger/addressable/compare/addressable-2.7.0...addressable-2.8.0) --- updated-dependencies: - dependency-name: addressable dependency-type: indirect ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> --- docs/geode-native-book-cpp/Gemfile.lock| 78 -- docs/geode-native-book-dotnet/Gemfile.lock | 78 -- 2 files changed, 82 insertions(+), 74 deletions(-) diff --git a/docs/geode-native-book-cpp/Gemfile.lock b/docs/geode-native-book-cpp/Gemfile.lock index d7df3492c..fa694de86 100644 --- a/docs/geode-native-book-cpp/Gemfile.lock +++ b/docs/geode-native-book-cpp/Gemfile.lock @@ -6,11 +6,11 @@ GEM minitest (~> 5.1) thread_safe (~> 0.3, >= 0.3.4) tzinfo (~> 1.1) -addressable (2.7.0) +addressable (2.8.0) public_suffix (>= 2.0.2, < 5.0) ansi (1.5.0) -backports (3.21.0) -bookbindery (10.1.17) +backports (3.23.0) +bookbindery (10.1.18) ansi (~> 1.4) css_parser elasticsearch @@ -28,7 +28,6 @@ GEM redcarpet (~> 3.2.3) rouge (!= 1.9.1) sprockets (= 3.7.2) - therubyracer thor (= 0.19.1) builder (3.2.4) chunky_png (1.4.0) @@ -48,42 +47,52 @@ GEM sass (>= 3.3.0, < 3.5) compass-import-once (1.0.5) sass (>= 3.2, < 3.5) -concurrent-ruby (1.1.8) +concurrent-ruby (1.1.9) contracts (0.13.0) -css_parser (1.9.0) +css_parser (1.11.0) addressable dotenv (2.7.6) -elasticsearch (7.13.0) - elasticsearch-api (= 7.13.0) - elasticsearch-transport (= 7.13.0) -elasticsearch-api (7.13.0) - multi_json -elasticsearch-transport (7.13.0) +elastic-transport (8.0.0) faraday (~> 1) multi_json -em-websocket (0.5.2) +elasticsearch (8.1.0) + elastic-transport (= 8.0.0) + elasticsearch-api (= 8.1.0) +elasticsearch-api (8.1.0) + multi_json +em-websocket (0.5.3) eventmachine (>= 0.12.9) - http_parser.rb (~> 0.6.0) + http_parser.rb (~> 0) erubis (2.7.0) eventmachine (1.2.7) -excon (0.81.0) +excon (0.91.0) execjs (2.8.1) -faraday (1.4.2) +faraday (1.10.0) faraday-em_http (~> 1.0) faraday-em_synchrony (~> 1.0) faraday-excon (~> 1.1) + faraday-httpclient (~> 1.0) + faraday-multipart (~> 1.0) faraday-net_http (~> 1.0) - faraday-net_http_persistent (~> 1.1) - multipart-post (>= 1.2, < 3) + faraday-net_http_persistent (~> 1.0) + faraday-patron (~> 1.0) + faraday-rack (~> 1.0) + faraday-retry (~> 1.0) ruby2_keywords (>= 0.0.4) faraday-em_http (1.0.0) faraday-em_synchrony (1.0.0) faraday-excon (1.1.0) +faraday-httpclient (1.0.1) +faraday-multipart (1.0.3) + multipart-post (>= 1.2, < 3) faraday-net_http (1.0.1) -faraday-net_http_persistent (1.1.0) -fast_blank (1.0.0) -fastimage (2.2.3) -ffi (1.15.1) +faraday-net_http_persistent (1.2.0) +faraday-patron (1.0.0) +faraday-rack (1.0.0) +faraday-retry (1.0.3) +fast_blank (1.0.1) +fastimage (2.2.6) +ffi (1.15.5) fog-aws (0.7.6) fog-core (~> 1.27) fog-json (~> 1.0) @@ -96,24 +105,23 @@ GEM fog-json (1.2.0) fog-core
[geode] branch develop updated (8d99d08 -> f6fee95)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 8d99d08 GEODE-8977: improve thread monitor to log lock information (#6377) add f6fee95 GEODE-9161: Use not deperacted property in gradle build (#6388) No new revisions were added by this update. Summary of changes: extensions/geode-modules-assembly/build.gradle | 2 +- geode-assembly/build.gradle| 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
[geode] branch develop updated (0b3eb04 -> 1ce8170)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 0b3eb04 GEODE-9192: test-container fixes (#6381) add 1ce8170 GEODE-9161: Fix some more gradle 7 warnings (#6369) No new revisions were added by this update. Summary of changes: extensions/geode-modules-assembly/build.gradle | 8 extensions/geode-modules-session/build.gradle | 2 +- geode-assembly/build.gradle| 6 +++--- geode-core/build.gradle| 6 +++--- 4 files changed, 11 insertions(+), 11 deletions(-)
[geode] branch develop updated (e65228f -> 5f7abc9)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from e65228f GEODE-8899: Temporarily restore dunitParallelForks (#6335) add 5f7abc9 GEODE-9161: Fix simple Gradle 7 warnings (#6315) No new revisions were added by this update. Summary of changes: buildSrc/build.gradle | 2 +- extensions/geode-modules-session/build.gradle | 6 +++--- extensions/session-testing-war/build.gradle | 4 ++-- geode-apis-compatible-with-redis/build.gradle | 6 +++--- geode-assembly/build.gradle | 10 +- geode-assembly/geode-assembly-test/build.gradle | 2 +- geode-concurrency-test/build.gradle | 2 +- geode-connectors/build.gradle | 2 +- geode-core/build.gradle | 14 +++--- geode-dunit/build.gradle| 4 ++-- geode-http-service/build.gradle | 2 +- geode-junit/build.gradle| 4 ++-- geode-logging/build.gradle | 4 ++-- geode-lucene/geode-lucene-test/build.gradle | 4 ++-- geode-management/build.gradle | 2 +- geode-membership/build.gradle | 4 ++-- geode-pulse/build.gradle| 2 +- geode-pulse/geode-pulse-test/build.gradle | 10 +- geode-rebalancer/build.gradle | 4 ++-- geode-serialization/build.gradle| 2 +- geode-wan/build.gradle | 2 +- geode-web-api/build.gradle | 2 +- geode-web-management/build.gradle | 2 +- geode-web/build.gradle | 4 ++-- gradle/java.gradle | 4 ++-- gradle/test.gradle | 2 +- 26 files changed, 53 insertions(+), 53 deletions(-)
[geode] 01/01: Fix self assignment
This is an automated email from the ASF dual-hosted git repository. moleske pushed a commit to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode.git commit 8dcef12679cf3384cf58d3d13632f5eaa7e88e9c Author: M. Oleske AuthorDate: Wed Mar 31 15:20:25 2021 -0700 Fix self assignment - https://github.com/apache/geode/pull/6222 added this and it the lgtm review was missed --- .../org/apache/geode/internal/cache/wan/GatewaySenderEventImpl.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderEventImpl.java b/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderEventImpl.java index 67b241e..1626a5e 100644 --- a/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderEventImpl.java +++ b/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderEventImpl.java @@ -1390,6 +1390,6 @@ public class GatewaySenderEventImpl } public void setAcked(boolean acked) { -this.isAcked = isAcked; +this.isAcked = acked; } }
[geode] branch moleske-patch-1 created (now 8dcef12)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode.git. at 8dcef12 Fix self assignment This branch includes the following new commits: new 8dcef12 Fix self assignment The 1 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.
[geode] branch develop updated (0c57fe5 -> bd20837)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 0c57fe5 GEODE-8946: User Guide - add Upgrade link to Getting Started page (#6159) add bd20837 GEODE-6143: Remove powermock from gfsh and lucene (#6161) No new revisions were added by this update. Summary of changes: geode-gfsh/build.gradle | 2 -- geode-lucene/build.gradle | 3 --- 2 files changed, 5 deletions(-)
[geode] branch develop updated (1fb1dc1 -> f30b891)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 1fb1dc1 GEODE-9003: Remove the member from replicates as GII candidate if it'… (#6093) add f30b891 GEODE-6143: remove powermock from LinuxSystemStatsTest (#6125) No new revisions were added by this update. Summary of changes: .../internal/statistics/LinuxSystemStatsTest.java | 129 - .../internal/statistics/100PercentStealTime.txt| 2 + .../internal/statistics/25PercentStealTime.txt | 2 + .../internal/statistics/66PercentStealTime.txt | 2 + .../geode/internal/statistics/FlatStealTime.txt| 2 + .../geode/internal/statistics/mockNetstatStats.txt | 2 + .../statistics/platform/LinuxProcFsStatistics.java | 59 +- 7 files changed, 61 insertions(+), 137 deletions(-) create mode 100644 geode-core/src/integrationTest/resources/org/apache/geode/internal/statistics/100PercentStealTime.txt create mode 100644 geode-core/src/integrationTest/resources/org/apache/geode/internal/statistics/25PercentStealTime.txt create mode 100644 geode-core/src/integrationTest/resources/org/apache/geode/internal/statistics/66PercentStealTime.txt create mode 100644 geode-core/src/integrationTest/resources/org/apache/geode/internal/statistics/FlatStealTime.txt create mode 100644 geode-core/src/integrationTest/resources/org/apache/geode/internal/statistics/mockNetstatStats.txt
[geode-kafka-connector] branch master updated: Correcting grammar of the connector description (#13)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/geode-kafka-connector.git The following commit(s) were added to refs/heads/master by this push: new 5056824 Correcting grammar of the connector description (#13) 5056824 is described below commit 50568245dc04c08895b80372dd982bb2c307da34 Author: Ashish Choudhary AuthorDate: Fri Jan 15 06:58:59 2021 +0530 Correcting grammar of the connector description (#13) * correcting kafka spelling * correcting grammar of description --- pom.xml | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/pom.xml b/pom.xml index 8c6956b..1d279b2 100644 --- a/pom.xml +++ b/pom.xml @@ -223,12 +223,11 @@ Kafka Connect Apache Geode https://geode.apache.org/docs/ -The Apache Geode connector can be used to move data from Kafka to Geode and vice versa. The Sink take data from a Kafka topic and puts in a region in Geode, while the Source will move any data inserted into Geode region, to Kafka topics . +The Apache Geode connector can be used to move data from Kafka to Geode and vice versa. The Sink takes data from a Kafka topic and puts it in a region in Geode, while the Source will move any data inserted into Geode region, to Kafka topics. -Apache Geode is an in-memory data grid which stores data in a key-value format. When the Geode acts as a Sink, the key value pair is extracted from the Sink Record from the Kafka topic and that key-value pair is stored in Geode regions. When Geode acts as a Source, whenever a key-value pair is inserted into the region, an event is sent to connector containing the data. This data is then placed into the Kafka topic. +Apache Geode is an in-memory data grid that stores data in a key-value format. When the Geode acts as a Sink, the key-value pair is extracted from the Sink Record from the Kafka topic and that key-value pair is stored in Geode regions. When Geode acts as a Source, whenever a key-value pair is inserted into the region, an event is sent to the connector containing the data. This data is then placed into the Kafka topic. -The mapping between the region and topics must be provided by the user. They must also ensure that the Apache Geode cluster is up and running along with all the required regions along with the topics in Kafka. The Apache Geode cluster must consist of at least one locator and one server. - +The mapping between the region and topics must be provided by the user. The user must also ensure that the Apache Geode cluster is up and running along with all the required regions along with the topics in Kafka. The Apache Geode cluster must consist of at least one locator and one server. logos/PoweredBYlogo.png Apache Software Foundation / VMware
[geode-native] branch develop updated: GEODE-8676: Update Bookbindery (#685)
This is an automated email from the ASF dual-hosted git repository. moleske 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 0791032 GEODE-8676: Update Bookbindery (#685) 0791032 is described below commit 07910325960691ab6774bedbf6e1a96f693e85d1 Author: Dave Barnes AuthorDate: Wed Nov 4 14:03:54 2020 -0800 GEODE-8676: Update Bookbindery (#685) --- docs/docker/view-docs.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/docker/view-docs.sh b/docs/docker/view-docs.sh index b91ffad..0a1d8bc 100755 --- a/docs/docker/view-docs.sh +++ b/docs/docker/view-docs.sh @@ -53,7 +53,7 @@ docker run -i -t \ -v "$PWD:${DOCS_DIR}" \ -p 127.0.0.1:${PORT}:${PORT} \ ${IMAGE_NAME}-${USER_NAME} \ - /bin/bash -c "bundle install; rackup --host 0.0.0.0 -p ${PORT}" + /bin/bash -c "bundle install; bundle exec rackup --host 0.0.0.0 -p ${PORT}" popd 1> /dev/null
[geode-native] branch develop updated: GEODE-8469: Enable no-missing-variable-declarations (#645)
This is an automated email from the ASF dual-hosted git repository. moleske 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 4617f22 GEODE-8469: Enable no-missing-variable-declarations (#645) 4617f22 is described below commit 4617f22a1fa91464e8f9a38b08eeff206c1677d4 Author: M. Oleske AuthorDate: Mon Sep 28 17:56:29 2020 -0700 GEODE-8469: Enable no-missing-variable-declarations (#645) * Enable no-missing-variable-declarations * ignore the warning in integration-test. We don't care about the old test code. Authored-by: M. Oleske Co-authored-by: Blake Bender Co-authored-by: Blake Bender --- CMakeLists.txt | 1 - cppcache/benchmark/GeodeLoggingBM.cpp | 28 ++-- cppcache/integration-test/CMakeLists.txt | 6 + cppcache/src/DataOutput.cpp| 3 +- cppcache/src/Exception.cpp | 2 +- cppcache/src/ExceptionTypes.cpp| 169 ++--- cppcache/src/Log.cpp | 28 ++-- cppcache/src/PdxFieldType.cpp | 4 +- cppcache/test/CacheXmlParserTest.cpp | 10 +- tests/cpp/security/XmlAuthzCredentialGenerator.hpp | 18 +-- 10 files changed, 135 insertions(+), 134 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 5842f61..e71dd1b 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -255,7 +255,6 @@ elseif(CMAKE_CXX_COMPILER_ID MATCHES "Clang") -Wno-float-equal #TODO fix -Wno-header-hygiene #TODO fix -Wno-conversion #TODO fix --Wno-missing-variable-declarations #TODO fix -Wno-reserved-id-macro #TODO fix -Wno-documentation # TODO fix -Wno-range-loop-analysis diff --git a/cppcache/benchmark/GeodeLoggingBM.cpp b/cppcache/benchmark/GeodeLoggingBM.cpp index ac49e31..3dd3ff7 100644 --- a/cppcache/benchmark/GeodeLoggingBM.cpp +++ b/cppcache/benchmark/GeodeLoggingBM.cpp @@ -35,9 +35,9 @@ using apache::geode::client::internal::geode_hash; const int STRING_ARRAY_LENGTH = 3; -int g_iteration = 0; +static int g_iteration = 0; -std::array logStrings{ +static std::array logStrings{ "Short test string", "Slightly longer test string", "Very long string: " "AA" @@ -106,17 +106,17 @@ void GeodeLogToFile(benchmark::State& state) { } } -auto LogStringsToConsole = GeodeLogToConsole; -auto LogIntsToConsole = GeodeLogToConsole; -auto LogComboToConsole = GeodeLogToConsole; +static const auto kLogStringsToConsole = GeodeLogToConsole; +static const auto kLogIntsToConsole = GeodeLogToConsole; +static const auto kLogComboToConsole = GeodeLogToConsole; -auto LogStringsToFile = GeodeLogToFile; -auto LogIntsToFile = GeodeLogToFile; -auto LogComboToFile = GeodeLogToFile; +static const auto kLogStringsToFile = GeodeLogToFile; +static const auto kLogIntsToFile = GeodeLogToFile; +static const auto kLogComboToFile = GeodeLogToFile; -BENCHMARK(LogStringsToConsole)->Range(8, 8 << 10); -BENCHMARK(LogIntsToConsole)->Range(8, 8 << 10); -BENCHMARK(LogComboToConsole)->Range(8, 8 << 10); -BENCHMARK(LogStringsToFile)->Range(8, 8 << 10); -BENCHMARK(LogIntsToFile)->Range(8, 8 << 10); -BENCHMARK(LogComboToFile)->Range(8, 8 << 10); +BENCHMARK(kLogStringsToConsole)->Range(8, 8 << 10); +BENCHMARK(kLogIntsToConsole)->Range(8, 8 << 10); +BENCHMARK(kLogComboToConsole)->Range(8, 8 << 10); +BENCHMARK(kLogStringsToFile)->Range(8, 8 << 10); +BENCHMARK(kLogIntsToFile)->Range(8, 8 << 10); +BENCHMARK(kLogComboToFile)->Range(8, 8 << 10); diff --git a/cppcache/integration-test/CMakeLists.txt b/cppcache/integration-test/CMakeLists.txt index 6dd225e..fb5170a 100644 --- a/cppcache/integration-test/CMakeLists.txt +++ b/cppcache/integration-test/CMakeLists.txt @@ -105,6 +105,12 @@ foreach(FILE ${SOURCES}) FOLDER cpp/test/integration/legacy ) +if(CMAKE_CXX_COMPILER_ID MATCHES "Clang") + target_compile_options(_WarningsAsError INTERFACE + -Wno-missing-variable-declarations + ) +endif() + target_link_libraries(${TEST} PRIVATE ACE::ACE diff --git a/cppcache/src/DataOutput.cpp b/cppcache/src/DataOutput.cpp index 12afb19..6b234ce 100644 --- a/cppcache/src/DataOutput.cpp +++ b/cppcache/src/DataOutput.cpp @@ -18,7 +18,6 @@ #include #include -#include #include "CacheImpl.hpp" #include "CacheRegionHelper.hpp" @@ -31,7 +30,7 @@ namespace apache { namespace geode { namespace client { -std::recursive_mutex globalBigBufferMutex; +static std::recursive_mutex globalBigBufferMutex; size_t DataOutput::m_highWaterMark = 50 * 102
[geode-native] branch develop updated: GEODE-8469: Enable no-missing-variable-declarations (#645)
This is an automated email from the ASF dual-hosted git repository. moleske 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 4617f22 GEODE-8469: Enable no-missing-variable-declarations (#645) 4617f22 is described below commit 4617f22a1fa91464e8f9a38b08eeff206c1677d4 Author: M. Oleske AuthorDate: Mon Sep 28 17:56:29 2020 -0700 GEODE-8469: Enable no-missing-variable-declarations (#645) * Enable no-missing-variable-declarations * ignore the warning in integration-test. We don't care about the old test code. Authored-by: M. Oleske Co-authored-by: Blake Bender Co-authored-by: Blake Bender --- CMakeLists.txt | 1 - cppcache/benchmark/GeodeLoggingBM.cpp | 28 ++-- cppcache/integration-test/CMakeLists.txt | 6 + cppcache/src/DataOutput.cpp| 3 +- cppcache/src/Exception.cpp | 2 +- cppcache/src/ExceptionTypes.cpp| 169 ++--- cppcache/src/Log.cpp | 28 ++-- cppcache/src/PdxFieldType.cpp | 4 +- cppcache/test/CacheXmlParserTest.cpp | 10 +- tests/cpp/security/XmlAuthzCredentialGenerator.hpp | 18 +-- 10 files changed, 135 insertions(+), 134 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 5842f61..e71dd1b 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -255,7 +255,6 @@ elseif(CMAKE_CXX_COMPILER_ID MATCHES "Clang") -Wno-float-equal #TODO fix -Wno-header-hygiene #TODO fix -Wno-conversion #TODO fix --Wno-missing-variable-declarations #TODO fix -Wno-reserved-id-macro #TODO fix -Wno-documentation # TODO fix -Wno-range-loop-analysis diff --git a/cppcache/benchmark/GeodeLoggingBM.cpp b/cppcache/benchmark/GeodeLoggingBM.cpp index ac49e31..3dd3ff7 100644 --- a/cppcache/benchmark/GeodeLoggingBM.cpp +++ b/cppcache/benchmark/GeodeLoggingBM.cpp @@ -35,9 +35,9 @@ using apache::geode::client::internal::geode_hash; const int STRING_ARRAY_LENGTH = 3; -int g_iteration = 0; +static int g_iteration = 0; -std::array logStrings{ +static std::array logStrings{ "Short test string", "Slightly longer test string", "Very long string: " "AA" @@ -106,17 +106,17 @@ void GeodeLogToFile(benchmark::State& state) { } } -auto LogStringsToConsole = GeodeLogToConsole; -auto LogIntsToConsole = GeodeLogToConsole; -auto LogComboToConsole = GeodeLogToConsole; +static const auto kLogStringsToConsole = GeodeLogToConsole; +static const auto kLogIntsToConsole = GeodeLogToConsole; +static const auto kLogComboToConsole = GeodeLogToConsole; -auto LogStringsToFile = GeodeLogToFile; -auto LogIntsToFile = GeodeLogToFile; -auto LogComboToFile = GeodeLogToFile; +static const auto kLogStringsToFile = GeodeLogToFile; +static const auto kLogIntsToFile = GeodeLogToFile; +static const auto kLogComboToFile = GeodeLogToFile; -BENCHMARK(LogStringsToConsole)->Range(8, 8 << 10); -BENCHMARK(LogIntsToConsole)->Range(8, 8 << 10); -BENCHMARK(LogComboToConsole)->Range(8, 8 << 10); -BENCHMARK(LogStringsToFile)->Range(8, 8 << 10); -BENCHMARK(LogIntsToFile)->Range(8, 8 << 10); -BENCHMARK(LogComboToFile)->Range(8, 8 << 10); +BENCHMARK(kLogStringsToConsole)->Range(8, 8 << 10); +BENCHMARK(kLogIntsToConsole)->Range(8, 8 << 10); +BENCHMARK(kLogComboToConsole)->Range(8, 8 << 10); +BENCHMARK(kLogStringsToFile)->Range(8, 8 << 10); +BENCHMARK(kLogIntsToFile)->Range(8, 8 << 10); +BENCHMARK(kLogComboToFile)->Range(8, 8 << 10); diff --git a/cppcache/integration-test/CMakeLists.txt b/cppcache/integration-test/CMakeLists.txt index 6dd225e..fb5170a 100644 --- a/cppcache/integration-test/CMakeLists.txt +++ b/cppcache/integration-test/CMakeLists.txt @@ -105,6 +105,12 @@ foreach(FILE ${SOURCES}) FOLDER cpp/test/integration/legacy ) +if(CMAKE_CXX_COMPILER_ID MATCHES "Clang") + target_compile_options(_WarningsAsError INTERFACE + -Wno-missing-variable-declarations + ) +endif() + target_link_libraries(${TEST} PRIVATE ACE::ACE diff --git a/cppcache/src/DataOutput.cpp b/cppcache/src/DataOutput.cpp index 12afb19..6b234ce 100644 --- a/cppcache/src/DataOutput.cpp +++ b/cppcache/src/DataOutput.cpp @@ -18,7 +18,6 @@ #include #include -#include #include "CacheImpl.hpp" #include "CacheRegionHelper.hpp" @@ -31,7 +30,7 @@ namespace apache { namespace geode { namespace client { -std::recursive_mutex globalBigBufferMutex; +static std::recursive_mutex globalBigBufferMutex; size_t DataOutput::m_highWaterMark = 50 * 102
[geode-native] branch develop updated: GEODE-8508: Update LGTM to use Geode 1.13 (#652)
This is an automated email from the ASF dual-hosted git repository. moleske 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 983036f GEODE-8508: Update LGTM to use Geode 1.13 (#652) 983036f is described below commit 983036fc86b5254ca1acc3ec10af9a2a82cff693 Author: Alberto Bustamante Reyes AuthorDate: Sat Sep 19 01:31:06 2020 +0200 GEODE-8508: Update LGTM to use Geode 1.13 (#652) --- .lgtm.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.lgtm.yml b/.lgtm.yml index 652e079..f9b8cb5 100644 --- a/.lgtm.yml +++ b/.lgtm.yml @@ -4,9 +4,9 @@ extraction: command: - mkdir _lgtm_build_dir - cd _lgtm_build_dir -- wget -O apache-geode.tgz http://mirror.transip.net/apache/geode/1.12.0/apache-geode-1.12.0.tgz +- wget -O apache-geode.tgz http://mirror.transip.net/apache/geode/1.13.0/apache-geode-1.13.0.tgz - tar xzf apache-geode.tgz -- cmake -DGEODE_ROOT="`pwd`/apache-geode-1.12.0" .. +- cmake -DGEODE_ROOT="`pwd`/apache-geode-1.13.0" .. - cd dependencies && cmake --build . -- -j2 index: build_command:
[geode-native] branch moleske-patch-2 created (now 89ce2a8)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-2 in repository https://gitbox.apache.org/repos/asf/geode-native.git. at 89ce2a8 Remove GemFireVersion.getBuildDate() This branch includes the following new commits: new 89ce2a8 Remove GemFireVersion.getBuildDate() The 1 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.
[geode-native] 01/01: Remove GemFireVersion.getBuildDate()
This is an automated email from the ASF dual-hosted git repository. moleske pushed a commit to branch moleske-patch-2 in repository https://gitbox.apache.org/repos/asf/geode-native.git commit 89ce2a8fcc9d338118c537958e82708d17ac224e Author: M. Oleske AuthorDate: Sat Aug 29 18:38:36 2020 -0700 Remove GemFireVersion.getBuildDate() This [commit](https://github.com/apache/geode/commit/d9d13100ae11e2eba8a6428fad29007388680b43#diff-a377b15d2036a2fc9072c1915c7f0ed4L83) removed getBuildDate() in the java code, causing geode native develop to fail to compile with geode develop --- tests/javaobject/LatestProp.java | 1 - 1 file changed, 1 deletion(-) diff --git a/tests/javaobject/LatestProp.java b/tests/javaobject/LatestProp.java index 4dc2daf..3bdb944 100644 --- a/tests/javaobject/LatestProp.java +++ b/tests/javaobject/LatestProp.java @@ -24,7 +24,6 @@ import org.apache.geode.internal.*; public class LatestProp { public static void main(String[] args){ -System.out.println("build.date=" + GemFireVersion.getBuildDate()); System.out.println("build.jdk=" + GemFireVersion.getBuildJavaVersion()); System.out.println("build.version=" + GemFireVersion.getBuildJavaVersion()); try{
[geode-native] 01/01: Add note that docker compose is needed
This is an automated email from the ASF dual-hosted git repository. moleske pushed a commit to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode-native.git commit 6019702c10843752c1f1448b8efc894887235b31 Author: M. Oleske AuthorDate: Sat Aug 29 18:35:53 2020 -0700 Add note that docker compose is needed Docker Compose is installed on Mac and Windows when using Docker Desktop. On Linux you install Docker and Docker Compose separately, so it should be called out in our docs --- BUILDING.md | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/BUILDING.md b/BUILDING.md index 7ee9d41..db793c5 100644 --- a/BUILDING.md +++ b/BUILDING.md @@ -7,7 +7,8 @@ * [OpenSSL](https://www.openssl.org) *(for building source documentation)* * Java 8 JDK *(for building server side java functions used in some of the integration tests)* * [Apache Geode](http://geode.apache.org/releases/) binaries installed or available to link against -* [Docker](https://www.docker.com/ (for running SNI Test) +* [Docker](https://www.docker.com/) (for running SNI Test) +* [Docker Compose](https://docs.docker.com/compose/install/) (for running SNI Test) ### Platform-Specific Prerequisites
[geode-native] branch moleske-patch-1 created (now 6019702)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode-native.git. at 6019702 Add note that docker compose is needed This branch includes the following new commits: new 6019702 Add note that docker compose is needed The 1 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.
[geode-native] branch develop updated: GEODE-8340: Enforce Switch compiler warnings as errors (#625)
This is an automated email from the ASF dual-hosted git repository. moleske 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 ea1bb1f GEODE-8340: Enforce Switch compiler warnings as errors (#625) ea1bb1f is described below commit ea1bb1f350ab29a94c5e374842fe39cc77166d04 Author: M. Oleske AuthorDate: Sat Aug 29 08:50:31 2020 -0700 GEODE-8340: Enforce Switch compiler warnings as errors (#625) - GEODE-8340: Enforce no-covered-switch-default - GEODE-8340: Enforce no-switch-enum - GEODE-8340: Enforce no-implicit-fallthrough - GEODE-8340: Use 4 way if else - GEODE-8340: Replace switch statement for enum with direct lookup - GEODE-8340: Use map instead of switch Co-authored-by: M. Oleske Co-authored-by: Blake Bender --- CMakeLists.txt | 3 - cppcache/include/geode/DataInput.hpp | 27 +- cppcache/include/geode/PdxFieldTypes.hpp | 42 +- .../integration-test/ThinClientSecurityHelper.hpp | 1 + cppcache/integration-test/testThinClientCq.cpp | 10 +- .../integration-test/testThinClientCqDurable.cpp | 5 +- .../testThinClientSecurityAuthorization.cpp| 1 + .../testThinClientSecurityAuthorizationMU.cpp | 1 + .../testThinClientSecurityCQAuthorizationMU.cpp| 5 +- ...tThinClientSecurityDurableCQAuthorizationMU.cpp | 5 +- cppcache/integration/framework/TestConfig.cpp.in | 2 +- cppcache/integration/test/SimpleCqListener.cpp | 5 +- cppcache/src/CqQueryImpl.cpp | 5 +- cppcache/src/CqService.cpp | 4 +- cppcache/src/EntryExpiryHandler.cpp| 2 +- cppcache/src/ExceptionTypes.cpp| 743 + cppcache/src/LRUAction.cpp | 3 +- cppcache/src/LocalRegion.cpp | 26 +- cppcache/src/Log.cpp | 10 +- cppcache/src/PdxFieldType.cpp | 53 +- cppcache/src/PdxInstanceImpl.cpp | 106 ++- cppcache/src/RegionExpiryHandler.cpp | 2 +- cppcache/src/SerializationRegistry.cpp | 193 +- cppcache/src/TcrConnection.cpp | 51 +- cppcache/src/TcrMessage.cpp| 30 + cppcache/src/TransactionalOperation.cpp| 3 - cppcache/src/statistics/AtomicStatisticsImpl.cpp | 5 +- cppcache/src/statistics/OsStatisticsImpl.cpp | 6 +- cppcache/src/statistics/StatArchiveWriter.cpp | 2 - .../src/statistics/StatisticDescriptorImpl.cpp | 13 +- tests/cpp/fwklib/IpcHandler.cpp| 6 +- tests/cpp/security/XmlAuthzCredentialGenerator.hpp | 16 +- 32 files changed, 986 insertions(+), 400 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index baeab6e..2a6ecda 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -256,11 +256,8 @@ elseif(CMAKE_CXX_COMPILER_ID MATCHES "Clang") -Wno-header-hygiene #TODO fix -Wno-conversion #TODO fix -Wno-missing-variable-declarations #TODO fix --Wno-switch-enum #TODO fix -Wno-reserved-id-macro #TODO fix -Wno-documentation # TODO fix --Wno-implicit-fallthrough #TODO fix --Wno-covered-switch-default #TODO fix -Wno-range-loop-analysis -Wno-weak-vtables -Wno-weak-template-vtables diff --git a/cppcache/include/geode/DataInput.hpp b/cppcache/include/geode/DataInput.hpp index c9f7bb8..f8bb967 100644 --- a/cppcache/include/geode/DataInput.hpp +++ b/cppcache/include/geode/DataInput.hpp @@ -296,25 +296,14 @@ class APACHE_GEODE_EXPORT DataInput { inline std::basic_string readString() { std::basic_string value; auto type = static_cast(read()); -switch (type) { - case internal::DSCode::CacheableString: -readJavaModifiedUtf8(value); -break; - case internal::DSCode::CacheableStringHuge: -readUtf16Huge(value); -break; - case internal::DSCode::CacheableASCIIString: -readAscii(value); -break; - case internal::DSCode::CacheableASCIIStringHuge: -readAsciiHuge(value); -break; - case internal::DSCode::CacheableNullString: -// empty string -break; - // TODO: What's the right response here? - default: -break; +if (type == internal::DSCode::CacheableString) { + readJavaModifiedUtf8(value); +} else if (type == internal::DSCode::CacheableStringHuge) { + readUtf16Huge(value); +} else if (type == internal::DSCode::CacheableASCIIString) { + readAscii(value); +} else if (type == internal::DSCode::CacheableASCIIStringHuge) { + readAsciiHuge(value); } return value; } diff --git a/cppcache/include/geode/PdxFieldTypes.hpp b/cppcache/include/geode/PdxFieldTypes.hpp ind
[geode] branch develop updated (48d2ee5 -> 15cb323)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 48d2ee5 GEODE-7702: bulkOp from accessor or NORMAL should sync with clear (#5012) add 960f33b GEODE-7707: Tab completing '--url' on 'connect' gives two default values add 1a87e32 GEODE-7707: Align examples in doc and code add 15cb323 GEODE-7707: Tab completing '--url' on 'connect' gives two default values (#5061) No new revisions were added by this update. Summary of changes: .../geode/management/internal/i18n/CliStrings.java | 7 +- .../gfsh/command-pages/connect.html.md.erb | 25 ++ 2 files changed, 12 insertions(+), 20 deletions(-)
[geode-native] branch moleske-patch-1 updated (05a95ae -> a53a129)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode-native.git. discard 05a95ae Update .lgtm.yml to latest Geode release (1.12) add 11a9bc2 GEODE-7868: Enhance ReflectionBasedAutoSerializer (#585) add 2e0fe62 GEODE-7930: Fix endpoint name truncation bug add 42fb6fb Introduced fixes: add 7dbcae0 Merge pull request #588 from Nordix/feature/GEODE-7930 add 71ae4c6 Bump doc version to 1.13 add 597b1a8 Fix broken Geode Image Link (#587) add a53a129 Update .lgtm.yml to latest Geode release (1.12) This update added new revisions after undoing existing revisions. That is to say, some revisions that were in the old version of the branch are not in the new version. This situation occurs when a user --force pushes a change and generates a repository containing something like this: * -- * -- B -- O -- O -- O (05a95ae) \ N -- N -- N refs/heads/moleske-patch-1 (a53a129) You should already have received notification emails for all of the O revisions, and so the following emails describe only the N revisions from the common base, B. Any revisions marked "omit" are not gone; other references still refer to them. Any revisions marked "discard" are gone forever. No new revisions were added by this update. Summary of changes: README.md |2 +- .../integration-test2/AutoSerializationTests.cs| 1639 clicache/integration-test2/CMakeLists.txt |1 + clicache/src/ReflectionBasedAutoSerializer.cpp | 615 clicache/src/ReflectionBasedAutoSerializer.hpp | 12 +- clicache/src/TypeRegistry.cpp |2 +- clicache/src/impl/DotNetTypes.hpp | 10 + cppcache/src/ThinClientPoolDM.cpp |9 +- docs/geode-native-book-cpp/config.yml |6 +- docs/geode-native-book-cpp/redirects.rb|8 +- docs/geode-native-book-dotnet/config.yml |6 +- docs/geode-native-book-dotnet/redirects.rb |8 +- docs/manual-build/README.md|2 + 13 files changed, 2030 insertions(+), 290 deletions(-) create mode 100644 clicache/integration-test2/AutoSerializationTests.cs
[geode-native] branch develop updated: Fix broken Geode Image Link (#587)
This is an automated email from the ASF dual-hosted git repository. moleske 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 597b1a8 Fix broken Geode Image Link (#587) 597b1a8 is described below commit 597b1a822d31a1d718b06c0a3e97c90d17a0b9f1 Author: M. Oleske AuthorDate: Mon Apr 20 12:03:35 2020 -0700 Fix broken Geode Image Link (#587) --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 86256ef..f333bef 100644 --- a/README.md +++ b/README.md @@ -1,4 +1,4 @@ -[https://geode.apache.org/img/apache_geode_logo.png; align="center"/>](http://geode.apache.org) +[https://geode.apache.org/img/Apache_Geode_logo.png; align="center"/>](http://geode.apache.org) [![Build Status](https://travis-ci.org/apache/geode-native.svg?branch=develop)](https://travis-ci.org/apache/geode-native) [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://www.apache.org/licenses/LICENSE-2.0) [![Total alerts](https://img.shields.io/lgtm/alerts/g/apache/geode-native.svg?logo=lgtm=18)](https://lgtm.com/projects/g/apache/geode-native/alerts/) [![Language grade: C/C++](https://img.shields.io/lgtm/grade/cpp/g/apache/geode-native.svg?l [...]
[geode] branch develop updated (d4b7c14 -> 1b1ad6b)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from d4b7c14 GEODE-7864: Closing the query statements after execution. (#4873) add 1b1ad6b GEODE-7947 Implement tests for EXPIRE-related functionality (#4904) No new revisions were added by this update. Summary of changes: ...Test.java => ExpireAtDockerAcceptanceTest.java} | 20 ++-- ...ceTest.java => ExpireDockerAcceptanceTest.java} | 20 ++-- .../geode/redis/GeoDockerAcceptanceTest.java | 2 +- .../geode/redis/HashesDockerAcceptanceTest.java| 2 +- .../geode/redis/ListsDockerAcceptanceTest.java | 2 +- ...eTest.java => PexpireDockerAcceptanceTest.java} | 20 ++-- .../geode/redis/PubSubDockerAcceptanceTest.java| 2 +- .../geode/redis/RenameDockerAcceptanceTest.java| 2 +- .../geode/redis/SetsDockerAcceptanceTest.java | 2 +- .../redis/SortedSetsDockerAcceptanceTest.java | 2 +- .../geode/redis/StringsDockerAcceptanceTest.java | 2 +- .../redis/general/ExpireAtIntegrationTest.java | 128 + .../geode/redis/general/ExpireIntegrationTest.java | 14 +-- .../redis/general/PexpireIntegrationTest.java | 12 +- .../redis/internal/executor/ExpireAtExecutor.java | 46 +--- .../redis/internal/executor/ExpireExecutor.java| 21 ++-- ...nitTest.java => ExpireAtExecutorJUnitTest.java} | 53 + .../executor/general/ExpireExecutorJUnitTest.java | 2 +- ...itTest.java => PExpireAtExecutorJUnitTest.java} | 54 - 19 files changed, 263 insertions(+), 143 deletions(-) copy geode-redis/src/acceptanceTest/java/org/apache/geode/redis/{ListsDockerAcceptanceTest.java => ExpireAtDockerAcceptanceTest.java} (68%) mode change 100755 => 100644 copy geode-redis/src/acceptanceTest/java/org/apache/geode/redis/{ListsDockerAcceptanceTest.java => ExpireDockerAcceptanceTest.java} (68%) mode change 100755 => 100644 copy geode-redis/src/acceptanceTest/java/org/apache/geode/redis/{ListsDockerAcceptanceTest.java => PexpireDockerAcceptanceTest.java} (68%) mode change 100755 => 100644 create mode 100644 geode-redis/src/integrationTest/java/org/apache/geode/redis/general/ExpireAtIntegrationTest.java copy geode-redis/src/test/java/org/apache/geode/redis/internal/executor/general/{ExpireExecutorJUnitTest.java => ExpireAtExecutorJUnitTest.java} (66%) copy geode-redis/src/test/java/org/apache/geode/redis/internal/executor/general/{ExpireExecutorJUnitTest.java => PExpireAtExecutorJUnitTest.java} (66%)
[geode-native] 01/01: Fix broken Geode Image Link
This is an automated email from the ASF dual-hosted git repository. moleske pushed a commit to branch moleske-patch-2 in repository https://gitbox.apache.org/repos/asf/geode-native.git commit df791c5bfd084b0d8dc307da6346a7b60cefab30 Author: M. Oleske AuthorDate: Thu Apr 2 10:38:48 2020 -0700 Fix broken Geode Image Link --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 86256ef..f333bef 100644 --- a/README.md +++ b/README.md @@ -1,4 +1,4 @@ -[https://geode.apache.org/img/apache_geode_logo.png; align="center"/>](http://geode.apache.org) +[https://geode.apache.org/img/Apache_Geode_logo.png; align="center"/>](http://geode.apache.org) [![Build Status](https://travis-ci.org/apache/geode-native.svg?branch=develop)](https://travis-ci.org/apache/geode-native) [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://www.apache.org/licenses/LICENSE-2.0) [![Total alerts](https://img.shields.io/lgtm/alerts/g/apache/geode-native.svg?logo=lgtm=18)](https://lgtm.com/projects/g/apache/geode-native/alerts/) [![Language grade: C/C++](https://img.shields.io/lgtm/grade/cpp/g/apache/geode-native.svg?l [...]
[geode-native] branch moleske-patch-2 created (now df791c5)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-2 in repository https://gitbox.apache.org/repos/asf/geode-native.git. at df791c5 Fix broken Geode Image Link This branch includes the following new commits: new df791c5 Fix broken Geode Image Link The 1 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.
[geode-native] branch moleske-patch-1 created (now 05a95ae)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode-native.git. at 05a95ae Update .lgtm.yml to latest Geode release (1.12) This branch includes the following new commits: new 05a95ae Update .lgtm.yml to latest Geode release (1.12) The 1 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.
[geode-native] 01/01: Update .lgtm.yml to latest Geode release (1.12)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a commit to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode-native.git commit 05a95aeb58e8eb3aa578214e30e18a06af1bafdd Author: M. Oleske AuthorDate: Thu Apr 2 10:20:53 2020 -0700 Update .lgtm.yml to latest Geode release (1.12) --- .lgtm.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.lgtm.yml b/.lgtm.yml index cda763f..652e079 100644 --- a/.lgtm.yml +++ b/.lgtm.yml @@ -4,9 +4,9 @@ extraction: command: - mkdir _lgtm_build_dir - cd _lgtm_build_dir -- wget -O apache-geode.tgz http://mirror.transip.net/apache/geode/1.10.0/apache-geode-1.10.0.tgz +- wget -O apache-geode.tgz http://mirror.transip.net/apache/geode/1.12.0/apache-geode-1.12.0.tgz - tar xzf apache-geode.tgz -- cmake -DGEODE_ROOT="`pwd`/apache-geode-1.10.0" .. +- cmake -DGEODE_ROOT="`pwd`/apache-geode-1.12.0" .. - cd dependencies && cmake --build . -- -j2 index: build_command:
[geode] branch develop updated (232cfa1 -> faece9a)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 232cfa1 GEODE-7923: update release scripts to handle support branches (#4874) add faece9a GEODE-7927: create tests to ensure feature-parity with Redis PEXPIRE command (#4886) No new revisions were added by this update. Summary of changes: .../PexpireIntegrationTest.java} | 51 +- .../executor/general/PexpireExecutorJUnitTest.java | 25 +-- 2 files changed, 30 insertions(+), 46 deletions(-) copy geode-redis/src/integrationTest/java/org/apache/geode/redis/{RedisLockServiceIntegrationTest.java => general/PexpireIntegrationTest.java} (50%) copy geode-assembly/src/acceptanceTest/java/org/apache/geode/management/internal/cli/shell/GfshDisconnectWithinScript.java => geode-redis/src/test/java/org/apache/geode/redis/internal/executor/general/PexpireExecutorJUnitTest.java (62%)
[geode] branch develop updated (50d2b47 -> 5fda2a6)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 50d2b47 add 1.11.0 to old versions (#4554) add 5fda2a6 GEODE-7651: Provide docker target for geode-assembly (#4573) No new revisions were added by this update. Summary of changes: build.gradle| 13 +++-- .../Dockerfile | 11 --- geode-assembly/build.gradle | 7 +++ gradle.properties | 2 ++ 4 files changed, 20 insertions(+), 13 deletions(-) copy {ci/resource-types/concourse-metadata-resource => geode-assembly}/Dockerfile (84%)
[geode] branch develop updated (7c9392b -> 4220611)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 7c9392b GEODE-7575: Add concourse team specific fly targets in the deploy scripts add 4220611 Add LGTM Badges (#4406) No new revisions were added by this update. Summary of changes: README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
[geode] branch moleske-patch-2 updated (4dc032a -> 4fb4c6e)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-2 in repository https://gitbox.apache.org/repos/asf/geode.git. from 4dc032a Force ci to run again for a readme change add 4fb4c6e What is going on with ci? No new revisions were added by this update. Summary of changes:
[geode] branch moleske-patch-2 updated (67aefe6 -> 4dc032a)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-2 in repository https://gitbox.apache.org/repos/asf/geode.git. from 67aefe6 Add LGTM Badges add 4dc032a Force ci to run again for a readme change No new revisions were added by this update. Summary of changes:
[geode] branch develop updated (1fd8745 -> c5c27e9)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 1fd8745 GEODE-7478: Register interest in appropriate cases. (#4366) add c5c27e9 Fix Geode Logo (#4405) No new revisions were added by this update. Summary of changes: README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
[geode] branch moleske-patch-2 created (now 67aefe6)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-2 in repository https://gitbox.apache.org/repos/asf/geode.git. at 67aefe6 Add LGTM Badges This branch includes the following new commits: new 67aefe6 Add LGTM Badges The 1 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.
[geode] branch moleske-patch-1 created (now 6dcbe1e)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode.git. at 6dcbe1e Fix Geode Logo This branch includes the following new commits: new 6dcbe1e Fix Geode Logo The 1 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.
[geode-native] branch develop updated: Add note about BasicIPv6Test (#549)
This is an automated email from the ASF dual-hosted git repository. moleske 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 35bf2fc Add note about BasicIPv6Test (#549) 35bf2fc is described below commit 35bf2fc13cb435531b2332856ca191569a498f36 Author: Alberto Bustamante Reyes AuthorDate: Thu Nov 14 23:45:54 2019 +0100 Add note about BasicIPv6Test (#549) * Add note about how to enable IPv6 --- BUILDING.md | 8 CONTRIBUTING.md | 2 ++ 2 files changed, 10 insertions(+) diff --git a/BUILDING.md b/BUILDING.md index 56f2d78..9b03b36 100644 --- a/BUILDING.md +++ b/BUILDING.md @@ -103,6 +103,14 @@ $ cmake --build . -- /m ### Optional Configuration + IPv6 support + +IPv6 support can be enabled by adding `-DWITH_IPV6=ON` to the CMake [Generator](#generator) command. + +```console +$ cmake … -DWITH_IPV6=ON … +``` + Code Coverage If building with GCC or Clang you can enable C++ code coverage by adding `-DUSE_CPP_COVERAGE=ON` to the CMake [Generator](#generator) command. diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index a1a6831..83f0fa7 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -58,6 +58,8 @@ $ ctest -R -j1 ``` For example: `$ ctest -R AuthInitializeTest.putGetWithBasicAuth -j1` +Notice that `BasicIPv6Test` test is expected to fail due to IPv6 support is disabled by default. [BUILDING.md](BUILDING.md) explains how to enable it. + ## Style
[geode-native] branch develop updated: GEODE-7418: Fixes issue deserializing unregistered PDX types. (#546)
This is an automated email from the ASF dual-hosted git repository. moleske 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 c932a1d GEODE-7418: Fixes issue deserializing unregistered PDX types. (#546) c932a1d is described below commit c932a1d765809dcd8d7a0c0f14a3aa6c98e18a5c Author: Jacob Barrett AuthorDate: Tue Nov 12 09:39:50 2019 -0800 GEODE-7418: Fixes issue deserializing unregistered PDX types. (#546) * GEODE-7418: Fixes issue deserializing unregistered PDX types. * If a PDX type does not have a domain class or registered serializer then we now fall back to PdxInstance objects. * Add test case to reproduce crash deserializing JSON values * Add test case for serializing non-PDX types * Test macro cleanup - Fixes test macros to properly handle ; --- cppcache/integration-test/.clang-tidy | 4 +- cppcache/integration-test/ThinClientDistOps.hpp| 8 +- cppcache/integration-test/ThinClientFailover.hpp | 2 +- cppcache/integration-test/ThinClientFailover2.hpp | 2 +- .../ThinClientFailoverInterest.hpp | 2 +- .../ThinClientFailoverInterest2.hpp| 2 +- .../integration-test/ThinClientFailoverRegex.hpp | 2 +- cppcache/integration-test/ThinClientHeapLRU.hpp| 2 +- .../ThinClientLocalCacheLoader.hpp | 2 +- cppcache/integration-test/ThinClientSSL.hpp| 2 +- .../integration-test/ThinClientSSLWithPassword.hpp | 2 +- cppcache/integration-test/ThinClientTXFailover.hpp | 2 +- .../integration-test/ThinClientTransactionsXA.hpp | 2 +- cppcache/integration-test/fw_dunit.hpp | 6 +- cppcache/integration-test/testCacheless.cpp| 2 +- .../integration-test/testCreateAndDestroyPool.cpp | 12 +- .../integration-test/testPdxMetadataCheckTest.cpp | 23 +- cppcache/integration-test/testThinClientCq.cpp | 33 +- .../integration-test/testThinClientCqDelta.cpp | 2 +- .../integration-test/testThinClientCqDurable.cpp | 4 +- .../testThinClientDeltaWithNotification.cpp| 2 +- .../integration-test/testThinClientDistOps2.cpp| 4 +- .../integration-test/testThinClientFailover3.cpp | 2 +- .../testThinClientHAMixedRedundancy.cpp| 2 +- .../testThinClientHAQueryFailover.cpp | 2 +- .../integration-test/testThinClientHeapLRU.cpp | 2 +- .../integration-test/testThinClientInterest1.cpp | 2 +- .../testThinClientInterest1_Bug1001.cpp| 2 +- .../testThinClientInterest2Pooled.cpp | 2 +- .../integration-test/testThinClientInterest3.cpp | 2 +- .../testThinClientInterest3Cacheless.cpp | 2 +- .../testThinClientInterestList.cpp | 2 +- .../testThinClientInterestList2.cpp| 2 +- .../testThinClientListenerCallbackArgTest.cpp | 2 +- .../testThinClientListenerInit.cpp | 6 +- ...ThinClientNotificationWithDeltaWithoutcache.cpp | 2 +- .../testThinClientPdxDeltaWithNotification.cpp | 2 +- .../integration-test/testThinClientPdxInstance.cpp | 6 +- .../integration-test/testThinClientPdxTests.cpp| 595 +++-- .../testThinClientPoolRedundancy.cpp | 38 +- .../testThinClientPutAllTimeout.cpp| 2 +- .../testThinClientPutWithDelta.cpp | 4 +- cppcache/integration-test/testThinClientRegex.cpp | 2 +- cppcache/integration-test/testThinClientRegex2.cpp | 2 +- cppcache/integration-test/testThinClientRegex3.cpp | 2 +- .../testThinClientRegionQueryExclusiveness.cpp | 12 +- .../testThinClientRemoteQueryFailoverPdx.cpp | 3 +- .../testThinClientRemoteQueryRS.cpp| 41 +- .../testThinClientRemoteQuerySS.cpp| 20 +- .../testThinClientRemoteQueryTimeout.cpp | 51 +- .../testThinClientRemoteRegionQuery.cpp| 47 +- .../integration-test/testThinClientRemoveOps.cpp | 2 +- .../testThinClientSSLAuthCorrupt.cpp | 2 +- .../integration-test/testThinClientSSLAuthFail.cpp | 2 +- .../testThinClientSSLAuthUntrusted.cpp | 2 +- .../testThinClientTransactionsWithSticky.cpp | 2 +- .../testThinClientTransactionsWithoutSticky.cpp| 2 +- cppcache/integration/test/CMakeLists.txt | 1 + cppcache/integration/test/PdxInstanceTest.cpp | 26 + cppcache/integration/test/PdxSerializerTest.cpp| 165 ++ cppcache/src/PdxHelper.cpp | 100 ++-- cppcache/src/PdxHelper.hpp | 7 +- cppcache/src/PdxWrapper.cpp| 4 +- cppcache/src/SerializationRegistry.cpp | 26 +- cppcache/src/SerializationRegistry.hpp | 4 +- 65 files changed, 776 insertions(+), 550 deletions(-) diff --git
[geode-native] branch develop updated: GEODE-7419: Complete info about geode-native integration tests (#544)
This is an automated email from the ASF dual-hosted git repository. moleske 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 e167371 GEODE-7419: Complete info about geode-native integration tests (#544) e167371 is described below commit e167371c5862fa0afcb1fb6302723709bb6b24c8 Author: Alberto Bustamante Reyes AuthorDate: Sat Nov 9 01:32:41 2019 +0100 GEODE-7419: Complete info about geode-native integration tests (#544) --- CONTRIBUTING.md | 44 ++-- 1 file changed, 26 insertions(+), 18 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 9510ca9..a1a6831 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -6,12 +6,13 @@ see [BUILDING.md](BUILDING.md) ## Next steps * Make your changes/add your feature/fix a bug. -* Test your feature branch changes -* Check your formatting -* Submit a pull request +* Test your feature branch changes. +* Check your formatting. +* Submit a pull request. ## Testing -Before submitting a pull request the unit and integration tests must all pass. We are using CTest, (Please see [the CTest documentation](https://cmake.org/Wiki/CMake/Testing_With_CTest) for further information.) +Before submitting a pull request the unit and integration tests must all pass. We are using CTest for running tests (please see [the CTest documentation](https://cmake.org/Wiki/CMake/Testing_With_CTest) for further information) and [Google Test](https://github.com/google/googletest) as testing framework. + ### Running unit tests ```bash $ cd @@ -20,35 +21,42 @@ $ cd cppcache/test/ $ ./apache-geode_unittests ``` -### Running old integration tests -```bash -$ cd -$ cd build -$ cmake --build . --target run-integration-tests -``` +### Running integration tests +There are two test suites of integration tests based on different testing frameworks. The old integration tests (stored in `geode-native/cppcache/integration-test`) are based on a custom testing framework, and the newer ones (stored in `geode-native/cppcache/integration/test`) are based on [Google Test](https://github.com/google/googletest). + +Old integration tests are deprecated. If your changes include the implementation of new integration test/s to be verified, they should be written using Google Test. If your change implies a significant change in one or more old test cases, you should create the equivalent test case/s using Google Test to substitute the old one/s instead of adapting them. -Which is equivalent to running ctest directly: +Both integration test suites can be executed together using CTest. + Running old integration test suite ```bash $ cd build/cppcache/integration-test -$ ctest --timeout 2000 -L STABLE -C -R . -j1 +$ ctest --timeout 2000 -L STABLE -C -j1 ``` -This will take ~ 2 hours, YMMV... you can up the jobs to 4 and run in parallel, but you may end up with test failures that will need to be re-run sequentially. Like so: +Execution will take 2 hours approximately. It is possible to increase the number of jobs changing the value of `-j` parameter (up to `4`) for running tests in parallel, although it may end up with failed tests that will need to be re-run sequentially. Standalone tests can also be run as follows: ```bash $ cd build/cppcache/integration-test $ ctest -R -C ``` +For example: `$ ctest --timeout 2000 -L STABLE -C Release -R testCacheless -j1` + .NET integration tests can be executed similarly from `build/clicache/integration-test`. -### Running new integration tests + Running Google Test integration test suite ```bash $ cd -$ cd build -$ cd cppcache/integration/test -$ ./cpp-integration-test [] +$ cd build/cppcache/integration/test +$ ctest -j1 +``` +It is possible to increase the number of jobs changing the value of `-j` parameter for running tests in parallel, although it may end up with failed tests that will need to be re-run sequentially. Standalone tests can also be run as follows: + +```bash +$ cd +$ cd build/cppcache/integration/test +$ ctest -R -j1 ``` -Note that are gtest options that may be passed to the test executable, like for example the test cases to be run. Use --help to get all the available options. +For example: `$ ctest -R AuthInitializeTest.putGetWithBasicAuth -j1` ## Style
[geode-native] branch develop updated: GEODE-7426: Fixes segfault in log message. (#545)
This is an automated email from the ASF dual-hosted git repository. moleske 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 55da853 GEODE-7426: Fixes segfault in log message. (#545) 55da853 is described below commit 55da853760c200c53568fe2e6549c912ec26cc27 Author: Jacob Barrett AuthorDate: Fri Nov 8 11:31:47 2019 -0800 GEODE-7426: Fixes segfault in log message. (#545) --- cppcache/src/ThinClientPoolDM.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/cppcache/src/ThinClientPoolDM.cpp b/cppcache/src/ThinClientPoolDM.cpp index c9b9475..67eb9b3 100644 --- a/cppcache/src/ThinClientPoolDM.cpp +++ b/cppcache/src/ThinClientPoolDM.cpp @@ -493,9 +493,8 @@ void ThinClientPoolDM::cleanStaleConnections(std::atomic& isRunning) { m_connManageTaskId, _nextIdle)) { LOGERROR("Failed to reschedule connection manager"); } else { -LOGFINEST( -"Rescheduled next connection manager run after %z seconds", -std::chrono::duration_cast(_nextIdle).count()); +LOGFINEST("Rescheduled next connection manager run after %s", + to_string(_nextIdle).c_str()); } LOGDEBUG("Pool size is %d, pool counter is %d", size(), m_poolSize.load());
[geode-native] branch develop updated: Resolve LGTM issues in ThinClientPoolDM (#543)
This is an automated email from the ASF dual-hosted git repository. moleske 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 24c5c78 Resolve LGTM issues in ThinClientPoolDM (#543) 24c5c78 is described below commit 24c5c785aa7fb8df92f14397068c29db8679df95 Author: M. Oleske AuthorDate: Thu Oct 31 17:30:36 2019 -0700 Resolve LGTM issues in ThinClientPoolDM (#543) * LGTM issues in ThinClientPoolDM, among other things * Clean up some warnings from CLion analysis * Remove unused functions * Use do-while instead of removing `;` * Use acquire and block instead of while loop and tryacquire * Use unique pointer for resultProcessor * Add override to be more clear * Don't check for error twice Authored-by: Michael Oleske --- cppcache/include/geode/internal/geode_base.hpp | 4 +- cppcache/src/TcrConnection.hpp | 4 +- cppcache/src/ThinClientPoolDM.cpp | 178 +++-- cppcache/src/ThinClientPoolDM.hpp | 2 - cppcache/src/util/exception.hpp| 4 +- 5 files changed, 53 insertions(+), 139 deletions(-) diff --git a/cppcache/include/geode/internal/geode_base.hpp b/cppcache/include/geode/internal/geode_base.hpp index 86ab9e5..1e222c9 100644 --- a/cppcache/include/geode/internal/geode_base.hpp +++ b/cppcache/include/geode/internal/geode_base.hpp @@ -55,10 +55,10 @@ /** Deletes x only if it exists */ #define _GEODE_SAFE_DELETE(x) \ - { \ + do {\ delete x; \ x = nullptr; \ - } + } while (0) /** Deletes array x only if it exists */ #define _GEODE_SAFE_DELETE_ARRAY(x) \ diff --git a/cppcache/src/TcrConnection.hpp b/cppcache/src/TcrConnection.hpp index 713c364..c3d09cf 100644 --- a/cppcache/src/TcrConnection.hpp +++ b/cppcache/src/TcrConnection.hpp @@ -56,11 +56,11 @@ /** Closes and Deletes connection only if it exists */ #define GF_SAFE_DELETE_CON(x) \ - { \ + do {\ x->close(); \ delete x; \ x = nullptr; \ - } + } while (0) namespace apache { namespace geode { diff --git a/cppcache/src/ThinClientPoolDM.cpp b/cppcache/src/ThinClientPoolDM.cpp index 617343d..c9b9475 100644 --- a/cppcache/src/ThinClientPoolDM.cpp +++ b/cppcache/src/ThinClientPoolDM.cpp @@ -31,7 +31,6 @@ #include "ExecutionImpl.hpp" #include "ExpiryHandler_T.hpp" #include "ExpiryTaskManager.hpp" -#include "NonCopyable.hpp" #include "TcrConnectionManager.hpp" #include "TcrEndpoint.hpp" #include "ThinClientRegion.hpp" @@ -57,7 +56,6 @@ class GetAllWork : public PooledWork, std::shared_ptr m_serverLocation; TcrMessage* m_request; TcrMessageReply* m_reply; - MapOfUpdateCounters m_mapOfUpdateCounters; bool m_attemptFailover; bool m_isBGThread; bool m_addToLocalCache; @@ -106,7 +104,7 @@ class GetAllWork : public PooledWork, m_reply->setChunkedResultHandler(m_resultCollector); } - ~GetAllWork() { + ~GetAllWork() override { delete m_request; delete m_reply; delete m_resultCollector; @@ -115,7 +113,7 @@ class GetAllWork : public PooledWork, TcrMessage* getReply() { return m_reply; } void init() {} - GfErrType execute(void) { + GfErrType execute() override { GuardUserAttributes gua; if (m_userAttribute != nullptr) { @@ -191,8 +189,7 @@ ThinClientPoolDM::ThinClientPoolDM(const char* name, m_memId = cacheImpl->getClientProxyMembershipIDFactory().create( hostName, driver, hostPort, clientDurableId.c_str(), durableTimeOut); - if (m_attrs->m_initLocList.size() == 0 && - m_attrs->m_initServList.size() == 0) { + if (m_attrs->m_initLocList.empty() && m_attrs->m_initServList.empty()) { std::string msg = "No locators or servers provided for pool named "; msg += name; throw IllegalStateException(msg); @@ -205,7 +202,7 @@ ThinClientPoolDM::ThinClientPoolDM(const char* name, cacheImpl->getStatisticsManager().forceSample(); if (!sysProp.isEndpointShufflingDisabled()) { -if (m_attrs->m_initServList.size() > 0) { +if (!m_attrs->m_initServList.empty()) { RandGen randgen; m_server = randgen(static_cast(m_attrs->m_initServList.size())); } @@ -220,7 +217,6 @@ ThinClientPoolDM::ThinClientPoolDM(const char* name, void ThinClientPoolDM::init() { LOGDEBUG("ThinClientPoolDM::init: Starting pool initialization"); auto cacheImpl = m_connManager.getCacheImpl(); - auto& sysProp = cacheImpl->getDistributedSystem().getSystemProperties(); m_isMultiUserMode = getMultiuserAuthentication
[geode-native] branch moleske-patch-1 deleted (was 2e042a4)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode-native.git. was 2e042a4 proper javascript link The revisions that were on this branch are still contained in other references; therefore, this change does not discard any commits from the repository.
[geode-native] branch develop updated (85aa0fe -> b2cd5b6)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode-native.git. from 85aa0fe GEODE-7262: Split Geode Native Client guide into language-specific editions (#532) new fae080a Add LGTM Badges new b2cd5b6 proper javascript link The 2 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: README.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
[geode-native] 02/02: proper javascript link
This is an automated email from the ASF dual-hosted git repository. moleske pushed a commit to branch develop in repository https://gitbox.apache.org/repos/asf/geode-native.git commit b2cd5b6b8ab4ce45f072f0d379a4b5e1426f9ed7 Author: M. Oleske AuthorDate: Sat Oct 5 14:32:14 2019 -0700 proper javascript link --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index ff7931a..86256ef 100644 --- a/README.md +++ b/README.md @@ -1,6 +1,6 @@ [https://geode.apache.org/img/apache_geode_logo.png; align="center"/>](http://geode.apache.org) -[![Build Status](https://travis-ci.org/apache/geode-native.svg?branch=develop)](https://travis-ci.org/apache/geode-native) [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://www.apache.org/licenses/LICENSE-2.0) [![Total alerts](https://img.shields.io/lgtm/alerts/g/apache/geode-native.svg?logo=lgtm=18)](https://lgtm.com/projects/g/apache/geode-native/alerts/) [![Language grade: C/C++](https://img.shields.io/lgtm/grade/cpp/g/apache/geode-native.svg?l [...] +[![Build Status](https://travis-ci.org/apache/geode-native.svg?branch=develop)](https://travis-ci.org/apache/geode-native) [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://www.apache.org/licenses/LICENSE-2.0) [![Total alerts](https://img.shields.io/lgtm/alerts/g/apache/geode-native.svg?logo=lgtm=18)](https://lgtm.com/projects/g/apache/geode-native/alerts/) [![Language grade: C/C++](https://img.shields.io/lgtm/grade/cpp/g/apache/geode-native.svg?l [...] **[Overview](#overview)** **[Building from Source](#building)**
[geode-native] 01/02: Add LGTM Badges
This is an automated email from the ASF dual-hosted git repository. moleske pushed a commit to branch develop in repository https://gitbox.apache.org/repos/asf/geode-native.git commit fae080af2246d702bfc8cd8f3c83bfa695debcac Author: M. Oleske AuthorDate: Sat Oct 5 14:30:00 2019 -0700 Add LGTM Badges - remove bad introduction link --- README.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/README.md b/README.md index 3e63a8e..ff7931a 100644 --- a/README.md +++ b/README.md @@ -1,10 +1,9 @@ [https://geode.apache.org/img/apache_geode_logo.png; align="center"/>](http://geode.apache.org) -[![Build Status](https://travis-ci.org/apache/geode-native.svg?branch=develop)](https://travis-ci.org/apache/geode-native) [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://www.apache.org/licenses/LICENSE-2.0) +[![Build Status](https://travis-ci.org/apache/geode-native.svg?branch=develop)](https://travis-ci.org/apache/geode-native) [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://www.apache.org/licenses/LICENSE-2.0) [![Total alerts](https://img.shields.io/lgtm/alerts/g/apache/geode-native.svg?logo=lgtm=18)](https://lgtm.com/projects/g/apache/geode-native/alerts/) [![Language grade: C/C++](https://img.shields.io/lgtm/grade/cpp/g/apache/geode-native.svg?l [...] **[Overview](#overview)** **[Building from Source](#building)** -**[Introduction](#started)** **[Application Development](#development)** ## Overview
[geode-native] branch moleske-patch-1 updated (4dde1f2 -> 2e042a4)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode-native.git. from 4dde1f2 Add LGTM Badges add 2e042a4 proper javascript link No new revisions were added by this update. Summary of changes: README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
[geode-native] branch moleske-patch-1 created (now 4dde1f2)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode-native.git. at 4dde1f2 Add LGTM Badges This branch includes the following new commits: new 4dde1f2 Add LGTM Badges The 1 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.
[geode-native] 01/01: Add LGTM Badges
This is an automated email from the ASF dual-hosted git repository. moleske pushed a commit to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode-native.git commit 4dde1f23bebb6dc9e9171c92edba46ebab0c3be2 Author: M. Oleske AuthorDate: Sat Oct 5 14:30:00 2019 -0700 Add LGTM Badges - remove bad introduction link --- README.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/README.md b/README.md index 3e63a8e..ff7931a 100644 --- a/README.md +++ b/README.md @@ -1,10 +1,9 @@ [https://geode.apache.org/img/apache_geode_logo.png; align="center"/>](http://geode.apache.org) -[![Build Status](https://travis-ci.org/apache/geode-native.svg?branch=develop)](https://travis-ci.org/apache/geode-native) [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://www.apache.org/licenses/LICENSE-2.0) +[![Build Status](https://travis-ci.org/apache/geode-native.svg?branch=develop)](https://travis-ci.org/apache/geode-native) [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://www.apache.org/licenses/LICENSE-2.0) [![Total alerts](https://img.shields.io/lgtm/alerts/g/apache/geode-native.svg?logo=lgtm=18)](https://lgtm.com/projects/g/apache/geode-native/alerts/) [![Language grade: C/C++](https://img.shields.io/lgtm/grade/cpp/g/apache/geode-native.svg?l [...] **[Overview](#overview)** **[Building from Source](#building)** -**[Introduction](#started)** **[Application Development](#development)** ## Overview
[geode-native] branch moleske-patch-1 created (now 5545210)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode-native.git. at 5545210 Update lgtm to point to current Geode (1.10.0) This branch includes the following new commits: new 5545210 Update lgtm to point to current Geode (1.10.0) The 1 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.
[geode-native] 01/01: Update lgtm to point to current Geode (1.10.0)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a commit to branch moleske-patch-1 in repository https://gitbox.apache.org/repos/asf/geode-native.git commit 554521022a9c84ff32fcea3c553439fa44a50c00 Author: M. Oleske AuthorDate: Fri Oct 4 08:46:12 2019 -0700 Update lgtm to point to current Geode (1.10.0) - 1.9.0 no longer exists so lgtm doesn't actually build. Updated it for 1.10.0 --- .lgtm.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.lgtm.yml b/.lgtm.yml index 5135970..cda763f 100644 --- a/.lgtm.yml +++ b/.lgtm.yml @@ -4,9 +4,9 @@ extraction: command: - mkdir _lgtm_build_dir - cd _lgtm_build_dir -- wget -O apache-geode.tgz http://mirror.transip.net/apache/geode/1.9.0/apache-geode-1.9.0.tgz +- wget -O apache-geode.tgz http://mirror.transip.net/apache/geode/1.10.0/apache-geode-1.10.0.tgz - tar xzf apache-geode.tgz -- cmake -DGEODE_ROOT="`pwd`/apache-geode-1.9.0" .. +- cmake -DGEODE_ROOT="`pwd`/apache-geode-1.10.0" .. - cd dependencies && cmake --build . -- -j2 index: build_command:
[geode] branch develop updated (21ae51e -> 91176d6)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 21ae51e GEODE-7193: add entry-idle-time and entry-time-to-live to region attributes add 91176d6 GEODE-7178: Check operation if instance of Byte and Destroy for older… (#4041) No new revisions were added by this update. Summary of changes: .../internal/cache/tier/sockets/BaseCommand.java | 23 --- .../cache/tier/sockets/command/Destroy65.java | 27 - .../internal/cache/tier/sockets/command/Put65.java | 21 ++- .../cache/tier/sockets/BaseCommandJUnitTest.java | 1 - .../cache/tier/sockets/command/Destroy65Test.java | 70 +- 5 files changed, 115 insertions(+), 27 deletions(-)
[geode-native] branch develop updated: Update the build instructions for Mac
This is an automated email from the ASF dual-hosted git repository. moleske 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 0ed4ed4 Update the build instructions for Mac new 672f1db Merge pull request #514 from charliemblack/develop 0ed4ed4 is described below commit 0ed4ed49e5ed7dc400f3fb234a3884feec717fb0 Author: Charlie Black AuthorDate: Fri Sep 6 16:17:48 2019 -0700 Update the build instructions for Mac --- BUILDING.md | 27 --- 1 file changed, 24 insertions(+), 3 deletions(-) diff --git a/BUILDING.md b/BUILDING.md index 0bf6d16..8ffcbb0 100644 --- a/BUILDING.md +++ b/BUILDING.md @@ -49,13 +49,34 @@ The recommended generator for most unix platforms is 'Makefiles' (default): $ cmake .. ``` - Xcode -The recommended generator for Xcode is `Xcode`: + Mac OSX Xcode + +Install X Code from the App Store +* You have to run XCode once to get it initialize properly (software agreement). +* Install the command line tools for xcode - run `xcode-select --install` from terminal + +Install the required dependancies through homebbrew. If you use another package manager for your mac feel free to use that. ```bash -$ cmake .. -G "Xcode" +$ brew install geode +$ brew install openssl +$ brew install doxygen +$ brew install cmake ``` +Follow these steps to build the geode native client. The recommended code generator is `Xcode`. + +```bash +$ cd +$ mkdir build +$ cd build +$ cmake .. -G "Xcode" -DOPENSSL_ROOT_DIR=/usr/local/opt/openssl -DCMAKE_INSTALL_PREFIX=`pwd`/install +$ cmake --build . --target docs +$ cmake --build . --target install -j8 +``` + +At the end of the process the geode native client will be in the `/build/install` directory. + Windows / Visual Studio When running cmake commands on Windows, be sure to use [Visual Studio Native Tools Command Prompt](https://msdn.microsoft.com/en-us/library/f35ctcxw.aspx) so environment variables are set properly.
[geode-native] branch develop updated: GEODE-7056: Update Geode version in CI
This is an automated email from the ASF dual-hosted git repository. moleske 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 9655181 GEODE-7056: Update Geode version in CI new ea9c590 Merge pull request #509 from Nordix/feature/GEODE-7056 9655181 is described below commit 9655181e1973951ab233d8a61ed21c9c2a2c5ac5 Author: Alberto Bustamante Reyes AuthorDate: Wed Aug 7 17:40:32 2019 +0200 GEODE-7056: Update Geode version in CI --- docker/Dockerfile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docker/Dockerfile b/docker/Dockerfile index c60d3c1..7a153ba 100644 --- a/docker/Dockerfile +++ b/docker/Dockerfile @@ -41,7 +41,7 @@ RUN apt-get update && \ update-alternatives --install /usr/bin/clang-tidyclang-tidy /usr/bin/clang-tidy-${CLANG_VERSION} 999 && \ update-alternatives --install /usr/bin/clang-format clang-format /usr/bin/clang-format-${CLANG_VERSION} 999 -ENV GEODE_VERSION 1.6.0 +ENV GEODE_VERSION 1.9.0 RUN wget "https://www.apache.org/dyn/closer.cgi?action=download=geode/${GEODE_VERSION}/apache-geode-${GEODE_VERSION}.tgz; --quiet -O - | \ tar xzf -
[geode] branch GEODE-7001-region-entry-count updated (9d4bd47 -> 4e8ce22)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch GEODE-7001-region-entry-count in repository https://gitbox.apache.org/repos/asf/geode.git. from 9d4bd47 concourse didn't trigger add 4e8ce22 GEODE-7001: Use LongAdder instead of AtomicLong No new revisions were added by this update. Summary of changes: .../org/apache/geode/internal/cache/RegionPerfStats.java | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-)
[geode] branch GEODE-7001-region-entry-count updated (8993596 -> 9d4bd47)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch GEODE-7001-region-entry-count in repository https://gitbox.apache.org/repos/asf/geode.git. from 8993596 Add region entry count gauge add 9d4bd47 concourse didn't trigger No new revisions were added by this update. Summary of changes:
[geode] branch develop updated (b9168fe -> cb8b36c)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from b9168fe GEODE-6862: Extract Patterns from LogConsumer add cb8b36c GEODE-6887: Stop routing wrapper meters to int stats (#3733) No new revisions were added by this update. Summary of changes: .../internal/cache/wan/GatewayReceiverStats.java | 4 +- .../statistics/meters/IntStatisticBinding.java | 46 .../statistics/meters/LegacyStatCounter.java | 10 .../cache/wan/GatewayReceiverStatsTest.java| 10 ++-- .../statistics/meters/IntStatisticBindingTest.java | 61 -- .../statistics/meters/LegacyStatCounterTest.java | 29 -- 6 files changed, 7 insertions(+), 153 deletions(-) delete mode 100644 geode-core/src/main/java/org/apache/geode/internal/statistics/meters/IntStatisticBinding.java delete mode 100644 geode-core/src/test/java/org/apache/geode/internal/statistics/meters/IntStatisticBindingTest.java
[geode] branch develop updated (7693689 -> 3d7113f)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 7693689 GEODE-6829: Fixes synchronization in AttributesFactory (#3670) add 3d7113f GEODE-6763: Send GatewayReceiver events received to Micrometer (#3573) No new revisions were added by this update. Summary of changes: .../geode/metrics/GatewayReceiverMetricsTest.java | 301 + .../metrics/SimpleMetricsPublishingService.java| 33 ++- .../stats/GatewayReceiverStatsIntegrationTest.java | 5 +- .../internal/cache/tier/sockets/AcceptorImpl.java | 5 +- .../internal/cache/wan/GatewayReceiverStats.java | 112 ++-- .../cache/tier/sockets/AcceptorImplTest.java | 2 + .../cache/wan/GatewayReceiverStatsTest.java| 199 ++ .../geode/test/junit/categories/MetricsTest.java} | 3 +- 8 files changed, 617 insertions(+), 43 deletions(-) create mode 100644 geode-assembly/src/acceptanceTest/java/org/apache/geode/metrics/GatewayReceiverMetricsTest.java copy geode-junit/src/main/java/com/examples/snapshot/MyObjectDataSerializable.java => geode-assembly/src/acceptanceTest/java/org/apache/geode/metrics/SimpleMetricsPublishingService.java (54%) create mode 100644 geode-core/src/test/java/org/apache/geode/internal/cache/wan/GatewayReceiverStatsTest.java copy geode-junit/src/{test/java/org/apache/geode/test/junit/categories/CategoryOne.java => main/java/org/apache/geode/test/junit/categories/MetricsTest.java} (96%)
[geode] branch develop updated (c63097d -> a3304e9)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from c63097d GEODE-6757: fix input gating for meta add a3304e9 GEODE-6754: Remove micrometer timers from LocalRegion (#3567) No new revisions were added by this update. Summary of changes: .../apache/geode/internal/cache/LocalRegion.java | 270 + .../internal/cache/LocalRegionMetricsTest.java | 430 - 2 files changed, 90 insertions(+), 610 deletions(-) delete mode 100644 geode-core/src/test/java/org/apache/geode/internal/cache/LocalRegionMetricsTest.java
[geode] branch develop updated: GEODE-6606: Support added MeterSubregistries in reconnect (#3418)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a commit to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git The following commit(s) were added to refs/heads/develop by this push: new f1550e3 GEODE-6606: Support added MeterSubregistries in reconnect (#3418) f1550e3 is described below commit f1550e3cd0388a1fc0beb898d982ec15866db02a Author: Kirk Lund AuthorDate: Mon Apr 8 13:39:02 2019 -0700 GEODE-6606: Support added MeterSubregistries in reconnect (#3418) --- .../MeterSubregistryReconnectDistributedTest.java | 194 + .../geode/cache/CacheFactoryIntegrationTest.java | 13 +- .../internal/InternalDistributedSystem.java| 12 +- .../geode/internal/cache/GemFireCacheImpl.java | 10 +- .../apache/geode/internal/cache/InternalCache.java | 2 +- .../geode/internal/cache/InternalCacheBuilder.java | 11 +- .../cache/InternalCacheForClientAccess.java| 5 + .../geode/internal/cache/ReconnectableCache.java | 24 +++ .../internal/cache/xmlcache/CacheCreation.java | 5 + ...ernalCacheBuilderAllowsMultipleSystemsTest.java | 13 +- .../internal/cache/InternalCacheBuilderTest.java | 19 +- 11 files changed, 284 insertions(+), 24 deletions(-) diff --git a/geode-core/src/distributedTest/java/org/apache/geode/internal/metrics/MeterSubregistryReconnectDistributedTest.java b/geode-core/src/distributedTest/java/org/apache/geode/internal/metrics/MeterSubregistryReconnectDistributedTest.java new file mode 100644 index 000..f688fe3 --- /dev/null +++ b/geode-core/src/distributedTest/java/org/apache/geode/internal/metrics/MeterSubregistryReconnectDistributedTest.java @@ -0,0 +1,194 @@ +/* + * 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. + */ +package org.apache.geode.internal.metrics; + +import static java.util.concurrent.TimeUnit.MILLISECONDS; +import static org.apache.geode.distributed.ConfigurationProperties.DISABLE_AUTO_RECONNECT; +import static org.apache.geode.distributed.ConfigurationProperties.ENABLE_CLUSTER_CONFIGURATION; +import static org.apache.geode.distributed.ConfigurationProperties.LOCATORS; +import static org.apache.geode.distributed.ConfigurationProperties.MAX_WAIT_TIME_RECONNECT; +import static org.apache.geode.distributed.ConfigurationProperties.MEMBER_TIMEOUT; +import static org.apache.geode.distributed.ConfigurationProperties.NAME; +import static org.apache.geode.distributed.internal.membership.gms.MembershipManagerHelper.getMembershipManager; +import static org.apache.geode.test.awaitility.GeodeAwaitility.await; +import static org.apache.geode.test.awaitility.GeodeAwaitility.getTimeout; +import static org.apache.geode.test.dunit.IgnoredException.addIgnoredException; +import static org.apache.geode.test.dunit.VM.getController; +import static org.apache.geode.test.dunit.VM.getVM; +import static org.apache.geode.test.dunit.VM.toArray; +import static org.assertj.core.api.Assertions.assertThat; + +import java.io.File; +import java.io.Serializable; +import java.util.Properties; + +import io.micrometer.core.instrument.MeterRegistry; +import io.micrometer.core.instrument.composite.CompositeMeterRegistry; +import io.micrometer.core.instrument.simple.SimpleMeterRegistry; +import org.junit.After; +import org.junit.Before; +import org.junit.Rule; +import org.junit.Test; + +import org.apache.geode.ForcedDisconnectException; +import org.apache.geode.cache.CacheFactory; +import org.apache.geode.distributed.LocatorLauncher; +import org.apache.geode.distributed.internal.InternalDistributedSystem; +import org.apache.geode.distributed.internal.membership.gms.mgr.GMSMembershipManager; +import org.apache.geode.internal.cache.InternalCache; +import org.apache.geode.test.dunit.VM; +import org.apache.geode.test.dunit.rules.DistributedRule; +import org.apache.geode.test.junit.rules.serializable.SerializableTemporaryFolder; +import org.apache.geode.test.junit.rules.serializable.SerializableTestName; + +public class MeterSubregistryReconnectDistributedTest implements Serializable { + + private static final long TIMEOUT = getTimeout().getValueInMS(); + + private static LocatorLauncher locatorLauncher;
[geode] branch develop updated (189d6fb -> 5b19591)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 189d6fb GEODE-6322: refactor createRegionCommand to use RegionConfig only (#3156) new 8d0bd64 Revert "GEODE-6334: Change cache operation stats to longs (#3142)" new 5b19591 Revert "GEODE-6345: Change jvmPauses stat to long (#3143)" The 7378 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: .../DistributedNoAckRegionCCEDUnitTest.java| 4 +- .../geode/internal/cache/GIIDeltaDUnitTest.java| 2 +- .../java/org/apache/geode/TXJUnitTest.java | 54 +-- .../apache/geode/cache/CacheStatisticsTest.java| 42 -- .../org/apache/geode/cache/ProxyJUnitTest.java | 33 +- ...itionedRegionSingleNodeOperationsJUnitTest.java | 12 +- .../bean/stats/MemberLevelStatsJUnitTest.java | 2 +- .../codeAnalysis/sanctionedDataSerializables.txt | 8 +- .../geode/internal/admin/ClientStatsManager.java | 16 +- .../internal/admin/remote/ClientHealthStats.java | 90 ++-- .../geode/internal/cache/CachePerfStats.java | 273 ++-- .../geode/internal/cache/DummyCachePerfStats.java | 36 +- .../apache/geode/internal/cache/LocalRegion.java | 52 +-- .../internal/statistics/StatSamplerStats.java | 61 +-- .../geode/management/ClientHealthStatus.java | 24 +- .../org/apache/geode/management/MemberMXBean.java | 2 +- .../management/internal/beans/MemberMBean.java | 2 +- .../internal/beans/MemberMBeanBridge.java | 4 +- .../geode/internal/cache/CachePerfStatsTest.java | 467 + .../internal/statistics/StatSamplerStatsTest.java | 80 .../cq/dunit/CqQueryOptimizedExecuteDUnitTest.java | 77 ++-- ...onedRegionCqQueryOptimizedExecuteDUnitTest.java | 68 +-- .../org/apache/geode/cache30/RegionTestCase.java | 6 +- .../sockets/ClientServerMiscDUnitTestBase.java | 5 + 24 files changed, 719 insertions(+), 701 deletions(-) delete mode 100644 geode-core/src/integrationTest/java/org/apache/geode/cache/CacheStatisticsTest.java delete mode 100644 geode-core/src/test/java/org/apache/geode/internal/statistics/StatSamplerStatsTest.java
[geode] branch revert-3143-GEODE-6345-statsamplerstats deleted (was 52843f1)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a change to branch revert-3143-GEODE-6345-statsamplerstats in repository https://gitbox.apache.org/repos/asf/geode.git. was 52843f1 Revert "GEODE-6345: Change jvmPauses stat to long (#3143)" This change permanently discards the following revisions: discard 52843f1 Revert "GEODE-6345: Change jvmPauses stat to long (#3143)"
[geode] branch develop updated: GEODE-6345: Change jvmPauses stat to long (#3143)
This is an automated email from the ASF dual-hosted git repository. moleske pushed a commit to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git The following commit(s) were added to refs/heads/develop by this push: new 5e97425 GEODE-6345: Change jvmPauses stat to long (#3143) 5e97425 is described below commit 5e974258bcf80cf0a5470938d2dddac7f8adda82 Author: M. Oleske AuthorDate: Tue Feb 5 12:23:29 2019 -0800 GEODE-6345: Change jvmPauses stat to long (#3143) Renamed a function per request. Changed a couple map lookups to array accesses Keeping JvmPausesIsALong to validate type Co-authored-by: Michael Oleske Co-authored-by: Kirk Lund Co-authored-by: Mark Hanson --- .../internal/statistics/StatSamplerStats.java | 61 - .../internal/statistics/StatSamplerStatsTest.java | 80 ++ 2 files changed, 107 insertions(+), 34 deletions(-) diff --git a/geode-core/src/main/java/org/apache/geode/internal/statistics/StatSamplerStats.java b/geode-core/src/main/java/org/apache/geode/internal/statistics/StatSamplerStats.java index 7224aa4..7609c92 100644 --- a/geode-core/src/main/java/org/apache/geode/internal/statistics/StatSamplerStats.java +++ b/geode-core/src/main/java/org/apache/geode/internal/statistics/StatSamplerStats.java @@ -24,24 +24,25 @@ import org.apache.geode.StatisticsTypeFactory; * Statistics related to the statistic sampler. */ public class StatSamplerStats { - public static final String SAMPLE_COUNT = "sampleCount"; // int - public static final String SAMPLE_TIME = "sampleTime"; // long - public static final String DELAY_DURATION = "delayDuration"; // int - public static final String STAT_RESOURCES = "statResources"; // int - public static final String JVM_PAUSES = "jvmPauses"; // int - public static final String SAMPLE_CALLBACKS = "sampleCallbacks"; // int - public static final String SAMPLE_CALLBACK_ERRORS = "sampleCallbackErrors"; // int - public static final String SAMPLE_CALLBACK_DURATION = "sampleCallbackDuration"; // long + static final String SAMPLE_COUNT = "sampleCount"; + private static final String SAMPLE_TIME = "sampleTime"; + private static final String DELAY_DURATION = "delayDuration"; + private static final String STAT_RESOURCES = "statResources"; + private static final String JVM_PAUSES = "jvmPauses"; + private static final String SAMPLE_CALLBACKS = "sampleCallbacks"; + private static final String SAMPLE_CALLBACK_ERRORS = "sampleCallbackErrors"; + private static final String SAMPLE_CALLBACK_DURATION = "sampleCallbackDuration"; private static final StatisticsType samplerType; private static final int sampleCountId; private static final int sampleTimeId; private static final int delayDurationId; private static final int statResourcesId; - private static final int jvmPausesId; + static final int jvmPausesId; private static final int sampleCallbacksId; private static final int sampleCallbackErrorsId; private static final int sampleCallbackDurationId; + static { StatisticsTypeFactory f = StatisticsTypeFactoryImpl.singleton(); samplerType = f.createType("StatSampler", "Stats on the statistic sampler.", @@ -56,7 +57,7 @@ public class StatSamplerStats { f.createIntGauge(STAT_RESOURCES, "Current number of statistic resources being sampled by this sampler.", "resources", false), -f.createIntCounter(JVM_PAUSES, +f.createLongCounter(JVM_PAUSES, "Total number of JVM pauses (which may or may not be full GC pauses) detected by this sampler. A JVM pause is defined as a system event which kept the statistics sampler thread from sampling for 3000 or more milliseconds. This threshold can be customized by setting the system property gemfire.statSamplerDelayThreshold (units are milliseconds).", "jvmPauses", false), f.createIntGauge(SAMPLE_CALLBACKS, @@ -79,54 +80,46 @@ public class StatSamplerStats { private final Statistics samplerStats; - public StatSamplerStats(StatisticsFactory f, long id) { + StatSamplerStats(StatisticsFactory f, long id) { this.samplerStats = f.createStatistics(samplerType, "statSampler", id); } - public void tookSample(long nanosSpentWorking, int statResources, long nanosSpentSleeping) { + static StatisticsType getStatisticsType() { +return samplerType; + } + + void tookSample(long nanosSpentWorking, int statResources, long nanosSpentSleeping) { this.samplerStats.incInt(sampleCountId, 1); this.samplerStats.incLong(sampleTimeId, nanosSpentWorking / 100); this.samplerStats.set
[geode] branch develop updated: GEODE-6325: Clarify start time assertion
This is an automated email from the ASF dual-hosted git repository. moleske pushed a commit to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git The following commit(s) were added to refs/heads/develop by this push: new 556ac6e GEODE-6325: Clarify start time assertion new 054687a Merge pull request #3133 from demery-pivotal/feature/GEODE-6325 556ac6e is described below commit 556ac6e76ab64d2c9a20ad534305dda6907a90ef Author: Dale Emery AuthorDate: Mon Jan 28 17:04:32 2019 -0800 GEODE-6325: Clarify start time assertion - Convert all assertions to AssertJ and make them more informative. - Convert all polling loops to Awaitility. Co-authored-by: Dale Emery Co-authored-by: Michael Oleske --- .../GemFireStatSamplerIntegrationTest.java | 546 ++--- 1 file changed, 272 insertions(+), 274 deletions(-) diff --git a/geode-core/src/integrationTest/java/org/apache/geode/internal/statistics/GemFireStatSamplerIntegrationTest.java b/geode-core/src/integrationTest/java/org/apache/geode/internal/statistics/GemFireStatSamplerIntegrationTest.java index a5d840b..143289c 100644 --- a/geode-core/src/integrationTest/java/org/apache/geode/internal/statistics/GemFireStatSamplerIntegrationTest.java +++ b/geode-core/src/integrationTest/java/org/apache/geode/internal/statistics/GemFireStatSamplerIntegrationTest.java @@ -14,6 +14,12 @@ */ package org.apache.geode.internal.statistics; +import static java.io.File.separator; +import static java.lang.Byte.MAX_VALUE; +import static java.lang.System.currentTimeMillis; +import static java.lang.System.getProperty; +import static java.lang.System.setProperty; +import static java.util.Arrays.fill; import static org.apache.geode.distributed.ConfigurationProperties.ARCHIVE_DISK_SPACE_LIMIT; import static org.apache.geode.distributed.ConfigurationProperties.ARCHIVE_FILE_SIZE_LIMIT; import static org.apache.geode.distributed.ConfigurationProperties.ENABLE_TIME_STATISTICS; @@ -22,24 +28,29 @@ import static org.apache.geode.distributed.ConfigurationProperties.MCAST_PORT; import static org.apache.geode.distributed.ConfigurationProperties.STATISTIC_ARCHIVE_FILE; import static org.apache.geode.distributed.ConfigurationProperties.STATISTIC_SAMPLE_RATE; import static org.apache.geode.distributed.ConfigurationProperties.STATISTIC_SAMPLING_ENABLED; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertFalse; -import static org.junit.Assert.assertNotNull; -import static org.junit.Assert.assertNull; -import static org.junit.Assert.assertTrue; -import static org.junit.Assume.assumeFalse; +import static org.apache.geode.internal.GemFireVersion.getBuildId; +import static org.apache.geode.internal.GemFireVersion.getGemFireVersion; +import static org.apache.geode.internal.GemFireVersion.getSourceDate; +import static org.apache.geode.internal.PureJavaMode.osStatsAreAvailable; +import static org.apache.geode.internal.cache.control.HeapMemoryMonitor.getTenuredMemoryPoolMXBean; +import static org.apache.geode.internal.cache.control.HeapMemoryMonitor.getTenuredPoolStatistics; +import static org.apache.geode.internal.net.SocketCreator.getHostName; +import static org.apache.geode.internal.net.SocketCreator.getLocalHost; +import static org.apache.geode.internal.statistics.HostStatSampler.TEST_FILE_SIZE_LIMIT_IN_KB_PROPERTY; +import static org.apache.geode.test.awaitility.GeodeAwaitility.await; +import static org.assertj.core.api.Assertions.assertThat; +import static org.assertj.core.api.Assumptions.assumeThat; import java.io.File; import java.lang.reflect.Method; -import java.util.Arrays; import java.util.List; +import java.util.Objects; import java.util.Properties; import java.util.concurrent.atomic.AtomicBoolean; import java.util.concurrent.atomic.AtomicInteger; import org.apache.logging.log4j.Logger; import org.junit.After; -import org.junit.Assert; import org.junit.Before; import org.junit.Rule; import org.junit.Test; @@ -51,16 +62,10 @@ import org.apache.geode.Statistics; import org.apache.geode.StatisticsType; import org.apache.geode.distributed.DistributedSystem; import org.apache.geode.distributed.internal.InternalDistributedSystem; -import org.apache.geode.internal.GemFireVersion; -import org.apache.geode.internal.PureJavaMode; -import org.apache.geode.internal.cache.control.HeapMemoryMonitor; import org.apache.geode.internal.logging.LogService; -import org.apache.geode.internal.net.SocketCreator; import org.apache.geode.internal.statistics.GemFireStatSampler.LocalStatListenerImpl; -import org.apache.geode.internal.statistics.platform.OsStatisticsFactory; import org.apache.geode.internal.statistics.platform.ProcessStats; import org.apache.geode.internal.stats50.VMStats50; -import org.apache.geode.internal.util.StopWatch; import org.apache.geode.test.junit.categories.StatisticsTest; /** @@ -86,8 +91,8 @@ public class
[geode-native] branch develop updated: Fix extra console in BUILDING for rendering
This is an automated email from the ASF dual-hosted git repository. moleske 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 63a6b81 Fix extra console in BUILDING for rendering 63a6b81 is described below commit 63a6b8196353debcd382a1c54a02208e914c227e Author: M. Oleske AuthorDate: Tue Sep 11 21:42:36 2018 -0700 Fix extra console in BUILDING for rendering --- BUILDING.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/BUILDING.md b/BUILDING.md index 29fd4c3..79f2eae 100644 --- a/BUILDING.md +++ b/BUILDING.md @@ -30,7 +30,7 @@ $ cd build $ cmake .. # build step $ cmake --build . -- -```console +``` To explicitly specify the location in which the Native Client will be installed, add `-DCMAKE_INSTALL_PREFIX=/path/to/installation/destination` to this initial `cmake` execution command.