geode-native.git
10 days agoGEODE-10291: Compile on ubuntu jammy (gcc/g++ 11.2.0) (#966) develop
M. Oleske [Thu, 15 Sep 2022 16:25:03 +0000 (12:25 -0400)] 
GEODE-10291: Compile on ubuntu jammy (gcc/g++ 11.2.0) (#966)

Authored-by: M. Oleske <michael@oleske.engineer>
12 days agoGEODE-10402: Fix FunctionException handling (#981)
Mario Salazar de Torres [Tue, 13 Sep 2022 11:16:34 +0000 (13:16 +0200)] 
GEODE-10402: Fix FunctionException handling (#981)

* GEODE-10402: Fix FunctionException handling

 - Fixed handling for FunctionException.
 - Added InternalFunctionInvocationTargetException and replaced
   GF_FUNCTION_EXCEPTION by
   GF_INTERNAL_FUNCTION_INVOCATION_TARGET_EXCEPTION, so function
   exceptions are properly handled.
 - Code modified to adapt to the above changes.

* GEODE-10402: Revision 1

 - Actually added handling for FunctionException exception in
   ThinClientRegion.
 - Removed InternalFunctionInvocationTargetException as part of the
   public API since this is part of the internal Java API.
 - Added a new IT to verify that FunctionExceptions thrown by the user
   are actually thrown in the native library as FunctionException's.

* GEODE-10402: Format code

* GEODE-10402: Revision 2

 - Fixed clang-tidy warnings.
 - Fixed format on one of the Java classes for testing.

5 weeks agoGEODE-10300: C++ native client: Allow locator responses greater than … (#970)
Alberto Gomez [Thu, 18 Aug 2022 07:39:47 +0000 (09:39 +0200)] 
GEODE-10300: C++ native client: Allow locator responses greater than … (#970)

* GEODE-10300: Fix locator response size limit in C++ client

If a response message from the locator to the C++ native client
is longer than 3000 bytes the C++ native client will only
read the first 3000 bytes.

* GEODE-10300: Updated after review

* GEODE-10300: Updated after review

* GEODE-10300: Updated after another review

* GEODE-10300: Updated after some more reviews

* GEODE-10300: Some more changes after review.

* GEODE-10300: Small change after review

* GEODE-10300: Remove unneeded space

5 weeks agoGEODE-10404: Fix javaobject compilation for Java 11 (#980)
Mario Salazar de Torres [Wed, 17 Aug 2022 06:26:01 +0000 (08:26 +0200)] 
GEODE-10404: Fix javaobject compilation for Java 11 (#980)

- After merging #973, javaobject compilation was broken for Java 11.
   The build passed since packer images uses Java 8, but as docker build
   images uses Java 11, compilation is failing there.
 - Root cause of the compilation issue is the difference in UTF-8 string
   handling between Java 8 and Java 11.
 - So, the issue was fixed by specifying UTF-8 encoding while compiling.

2 months agoMerge pull request #973 from Nordix/feature/GEODE-10076
Alberto Gomez [Fri, 8 Jul 2022 09:00:45 +0000 (11:00 +0200)] 
Merge pull request #973 from Nordix/feature/GEODE-10076

GEODE-10076. Align String serialization with Java client

2 months agoGEODE-10076: Fix test case in windows 973/head
Alberto Gomez [Thu, 7 Jul 2022 07:40:04 +0000 (09:40 +0200)] 
GEODE-10076: Fix test case in windows

2 months agoGEODE-10076. Align String serialization with Java client
Alberto Gomez [Wed, 18 May 2022 11:37:49 +0000 (13:37 +0200)] 
GEODE-10076. Align String serialization with Java client

2 months agoMerge pull request #974 from Nordix/feature/GEODE-10371
Alberto Gomez [Wed, 6 Jul 2022 06:54:15 +0000 (08:54 +0200)] 
Merge pull request #974 from Nordix/feature/GEODE-10371

GEODE-10371: C++ client: Improve dispersion on connections expiration

2 months agoGEODE-10089: Bump Geode version to 1.15.0 (#976)
Owen Nichols [Tue, 5 Jul 2022 08:16:41 +0000 (01:16 -0700)] 
GEODE-10089: Bump Geode version to 1.15.0 (#976)

Native client hardcodes Geode version to test with in several places.
Update those variables to latest-and-greatest apache-geode 1.15.0

3 months agoGEODE-10371: Changes after review 974/head
Alberto Gomez [Tue, 21 Jun 2022 17:15:56 +0000 (19:15 +0200)] 
GEODE-10371: Changes after review

3 months agoGEODE-10382: Fix Windows build in CI pipeline (#975)
Blake Bender [Thu, 16 Jun 2022 17:51:22 +0000 (10:51 -0700)] 
GEODE-10382: Fix Windows build in CI pipeline (#975)

* GEODE-10392: Remove unreferenced local variable (Windows build warning
due to new compiler)

3 months agoGEODE-10371: C++ client: Improve dispersion on connections expiration
Alberto Gomez [Thu, 9 Jun 2022 16:32:47 +0000 (18:32 +0200)] 
GEODE-10371: C++ client: Improve dispersion on connections expiration

4 months agoGEODE-10218: Allow server debug (#971)
Michael Martell [Fri, 13 May 2022 16:09:33 +0000 (09:09 -0700)] 
GEODE-10218: Allow server debug (#971)

* Remove unused List<Locator>
* Show withDebugAgent() usage in cli

4 months agoGEODE-10218: Allow server debug (#954)
Michael Martell [Thu, 12 May 2022 14:08:20 +0000 (07:08 -0700)] 
GEODE-10218: Allow server debug (#954)

* Support DebugAgent in clicache tests
* Support DebugAgent in cppcache tests
* Fix StartLocators bug
* Use builder pattern in Cluster, Locator and Server classes

4 months agoGEMNC-509: Rename .NET client to .NET Framework (#965)
Blake Bender [Wed, 4 May 2022 15:04:08 +0000 (08:04 -0700)] 
GEMNC-509: Rename .NET client to .NET Framework (#965)

Co-authored-by: Max Hufnagel <mhufnagel@vmware.com>
4 months agoGEODE-10259: Update protocol ordinal to 1.14.0 (125) (#962)
Jacob Barrett [Fri, 29 Apr 2022 19:56:22 +0000 (12:56 -0700)] 
GEODE-10259: Update protocol ordinal to 1.14.0 (125) (#962)

* Fixes ClientProxyMembershipID for 1.1.0+
* Fixes TcrConnection for 1.5.0+
* Fixes TXCommitMessage for 1.7.0+
* Updated ClientHealthStats for 1.9.0+
* Updated QueueConnectionRequestTest for 1.14.0+

5 months agoGEODE-10227: Remove redundant sendRequestForChunkedResponse (#957)
Michael Martell [Wed, 20 Apr 2022 04:57:44 +0000 (21:57 -0700)] 
GEODE-10227: Remove redundant sendRequestForChunkedResponse (#957)

* Remove redundant sendRequestForChunkedResponse
* Remove vestigial comment

5 months agoGEODE-10074: Remove ACE remainings (#928)
Mario Salazar de Torres [Tue, 19 Apr 2022 17:06:19 +0000 (19:06 +0200)] 
GEODE-10074: Remove ACE remainings (#928)

- Removed all the ACE includes along the repository
 - Removed ACE from CMake project.
 - Removed ACE dependency.
 - There were a couple of files in which ACE's strcmp was used.
   So, its usage was replaced to to std::string::operator==
 - Removed non-necessary ACE's getcwd calls.

5 months agoGEODE-9921: Rename .NET to .NET Framework (#960)
Max Hufnagel [Mon, 18 Apr 2022 14:10:52 +0000 (07:10 -0700)] 
GEODE-9921: Rename .NET to .NET Framework (#960)

5 months agoGEODE-10241: disable re-auth test until we can implement properly (#961)
Blake Bender [Fri, 15 Apr 2022 21:31:12 +0000 (14:31 -0700)] 
GEODE-10241: disable re-auth test until we can implement properly (#961)

5 months agoGEODE-10131: Remove set but unused variables (#947)
M. Oleske [Sat, 9 Apr 2022 15:42:17 +0000 (08:42 -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 <michael@oleske.engineer>
5 months agoFix lgtm ubuntu break (#958)
M. Oleske [Fri, 8 Apr 2022 17:46:09 +0000 (10:46 -0700)] 
Fix lgtm ubuntu break (#958)

- related to the earlier ubuntu break, so applying the same fix

Authored-by: M. Oleske <michael@oleske.engineer>
5 months agoBump puma from 5.6.2 to 5.6.4 in /docs/geode-native-book-dotnet (#956)
dependabot[bot] [Thu, 7 Apr 2022 20:42:12 +0000 (13:42 -0700)] 
Bump puma from 5.6.2 to 5.6.4 in /docs/geode-native-book-dotnet (#956)

Bumps [puma](https://github.com/puma/puma) from 5.6.2 to 5.6.4.
- [Release notes](https://github.com/puma/puma/releases)
- [Changelog](https://github.com/puma/puma/blob/master/History.md)
- [Commits](https://github.com/puma/puma/compare/v5.6.2...v5.6.4)

---
updated-dependencies:
- dependency-name: puma
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
5 months agoBump puma from 5.6.2 to 5.6.4 in /docs/geode-native-book-cpp (#955)
dependabot[bot] [Thu, 7 Apr 2022 20:41:42 +0000 (13:41 -0700)] 
Bump puma from 5.6.2 to 5.6.4 in /docs/geode-native-book-cpp (#955)

Bumps [puma](https://github.com/puma/puma) from 5.6.2 to 5.6.4.
- [Release notes](https://github.com/puma/puma/releases)
- [Changelog](https://github.com/puma/puma/blob/master/History.md)
- [Commits](https://github.com/puma/puma/compare/v5.6.2...v5.6.4)

---
updated-dependencies:
- dependency-name: puma
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
5 months agoGEODE-10120: Bump ruby dependencies (#945)
M. Oleske [Thu, 7 Apr 2022 20:30:59 +0000 (13:30 -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] <support@github.com>
* 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] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
5 months agoRemove spurious comma (#951)
Michael Martell [Wed, 6 Apr 2022 21:07:03 +0000 (14:07 -0700)] 
Remove spurious comma (#951)

Packer builds for Windows were failing because JSON parsing failed.

5 months agoUbuntu build break fix, 2nd attempt (#953)
Blake Bender [Wed, 6 Apr 2022 20:05:05 +0000 (13:05 -0700)] 
Ubuntu build break fix, 2nd attempt (#953)

* Fix Ubuntu build break due to cmake upgrade
- New cmake version changes treatment of CMAKE_INSTALL_LIBDIR variable,
  which now evaluates to different things for xerces build and main
  client build.  As a result, we build Xerces, then go looking to link
  to it in the wrong place.  Hard-coding the path to just 'lib' for now
  appears to fix the Ubuntu issue without breaking anything else.

* 2nd attempt to fix Ubuntu build break
- First attempt broke RHEL flavors, and only fixed the first dependent
  library on Ubuntu - gtest was still looked for in the wrong place
- It appears cmake has special code on Ubuntu to use the wrong lib dir
  iff CMAKE_INSTALL_PREFIX is set to /usr or /usr/local, so we'll set it
  to something else and see if that does or doesn't break anything else.

5 months agoFix Ubuntu build break due to cmake upgrade (#952)
Blake Bender [Tue, 5 Apr 2022 21:37:34 +0000 (14:37 -0700)] 
Fix Ubuntu build break due to cmake upgrade (#952)

- New cmake version changes treatment of CMAKE_INSTALL_LIBDIR variable,
  which now evaluates to different things for xerces build and main
  client build.  As a result, we build Xerces, then go looking to link
  to it in the wrong place.   It appears cmake has special code on Ubuntu to use the wrong lib dir
  iff CMAKE_INSTALL_PREFIX is set to /usr or /usr/local, so setting it to something else seems the safest approach.

5 months agoGEODE-10058: Remove netcore and cbindings (#950)
Michael Martell [Wed, 30 Mar 2022 17:21:08 +0000 (10:21 -0700)] 
GEODE-10058: Remove netcore and cbindings (#950)

* Remove c-bindings
* Remove netcore
* Remove dotnetcore sdk
* Remove c-bindings and netcore from CMakeLists.txt
* Remove netcore tests from CI

Co-authored-by: Blake Bender <bblake@vmware.com>
5 months agoGEODE-10101: Bump Geode version to 1.14.4 (#948)
Dick Cavender [Mon, 28 Mar 2022 16:12:13 +0000 (09:12 -0700)] 
GEODE-10101: Bump Geode version to 1.14.4 (#948)

Native client hardcodes Geode version to test with in several places.
Update those variables to latest-and-greatest apache-geode 1.14.4

6 months agoFix regex used to read server handshake requests (#946)
Blake Bender [Fri, 11 Mar 2022 20:42:04 +0000 (12:42 -0800)] 
Fix regex used to read server handshake requests (#946)

6 months agoFix regexes in server messages (#944)
Blake Bender [Mon, 7 Mar 2022 19:49:00 +0000 (11:49 -0800)] 
Fix regexes in server messages (#944)

6 months agoGnmsg parse server handshake response (#939)
Blake Bender [Mon, 7 Mar 2022 15:55:03 +0000 (07:55 -0800)] 
Gnmsg parse server handshake response (#939)

* Add timestamp, tid, direction, and Type to server handshake request

* Parsing server handshake response in gnmsg
- Added logging to TcrConnection to dump handshake response bytes
- Cleaned up some code in TcrConnection::initTcrConnection, to help
  clarify what's going on
- Renamed TcrConnection member variables to conform to standard

6 months agoAdd shorts and longs to read_cacheable (#940)
Michael Martell [Thu, 3 Mar 2022 15:47:53 +0000 (07:47 -0800)] 
Add shorts and longs to read_cacheable (#940)

6 months agoGEODE-10085: Don't start JmxManager if already running (#936)
Michael Martell [Wed, 2 Mar 2022 15:48:48 +0000 (07:48 -0800)] 
GEODE-10085: Don't start JmxManager if already running (#936)

* Don't start JmxManager if already started
* Fix broken StartLocators() and StartServers()

6 months agoGEODE-10082: Fixes mixing of DSCode and DSFid enums. (#933)
Jacob Barrett [Mon, 28 Feb 2022 16:49:17 +0000 (08:49 -0800)] 
GEODE-10082: Fixes mixing of DSCode and DSFid enums. (#933)

Moves InterestResultPolicy and ClientProxyMembershipId to DSFid.

Replaces eroneous CacheableEnum with PDX_ENUM.

7 months agoAllow --handshake and --messages flags to coexist in gnmsg (#935)
Blake Bender [Fri, 25 Feb 2022 00:07:42 +0000 (16:07 -0800)] 
Allow --handshake and --messages flags to coexist in gnmsg (#935)

7 months agoGEODE-10062: Update Native Client Docs to minimize redirects - additions and correcti...
Dave Barnes [Thu, 24 Feb 2022 21:46:14 +0000 (13:46 -0800)] 
GEODE-10062: Update Native Client Docs to minimize redirects - additions and corrections (#934)

7 months agognmsg now parses all observed locator requests and responses (#932)
Blake Bender [Thu, 24 Feb 2022 19:04:03 +0000 (11:04 -0800)] 
gnmsg now parses all observed locator requests and responses (#932)

- Also remove duplicate values from ds_codes enum
- geode-native has these wrong right now in the C++ code, so they came
  across to Python this way.

7 months agoGEODE-10062: Update Native Client Docs to minimize redirects (#931)
Dave Barnes [Thu, 24 Feb 2022 17:03:30 +0000 (09:03 -0800)] 
GEODE-10062: Update Native Client Docs to minimize redirects (#931)

7 months agoGEODE-10073: Fix regex issue on Linux systems (#929)
Blake Bender [Tue, 22 Feb 2022 21:12:58 +0000 (13:12 -0800)] 
GEODE-10073: Fix regex issue on Linux systems (#929)

- Hex dump of a pointer on Linux is prefixed with '0x', so need to
  allow x or X character

7 months agoGEODE-10073: Print bytes of ClientConnectionRequest/Response in logs (#927)
Blake Bender [Tue, 22 Feb 2022 20:48:49 +0000 (12:48 -0800)] 
GEODE-10073: Print bytes of ClientConnectionRequest/Response in logs (#927)

* Print bytes of ClientConnectionRequest/Response in logs
* Dump the bytes at debug log level, so we can parse
* Also added parsing of these to gnmsg tool

Co-authored-by: Blake Bender <bblake@vmware.com>
7 months agoGEODE-9325: Replace old ITs worker spawner (#813)
Mario Salazar de Torres [Tue, 22 Feb 2022 19:51:13 +0000 (20:51 +0100)] 
GEODE-9325: Replace old ITs worker spawner (#813)

- Replaced references to ACE_Process realated to old ITs process
   spawners.
 - Added Boost program_options project, given that was needed to fulfill
   above changes.
 - Fix Windows execution whenever a test failed. Apparently if not all
 the processes have terminated ctest does not exit its execution and due
 to the fact that there is a worker in charge of managing the cluster
 processes, if the coordinator terminates all of the workers, it's not
 guaranteed that the cluster is stopped.
 - Reversed order in which cleanups are called in order to ensure proper
 teardown.

7 months agoGEODE-10065: Add missing colon in regexp (#926)
Michael Martell [Fri, 18 Feb 2022 18:36:55 +0000 (10:36 -0800)] 
GEODE-10065: Add missing colon in regexp (#926)

7 months agoGEODE-9327: Remove ACE networking code references (#815)
Mario Salazar de Torres [Fri, 18 Feb 2022 16:51:04 +0000 (17:51 +0100)] 
GEODE-9327: Remove ACE networking code references (#815)

 - Replaced method implementation to get a random port in old ITs framework.
 - Replaced convertHostToCanonicalForm implementation.
 - Changed ClientProxyMembershipID address serialization.
 - Removed unused PoolFactory::addCheck method.
 - Modified QueueConnectionRequestTest test to pass after the changes.
 - Fixed min/max macros windows compilcation
 - Changed ClientProxyMembershipID member attribute names to match style
   guidelines.
 - Changed HostStatSampler member attribute names to match style
 guidelines.
 - Changed Utils::convertHostToCanonicalForm to make the method more
   readable.

7 months agoGEODE-9326: Replace ACE_Get_Opt (#814)
Mario Salazar de Torres [Thu, 17 Feb 2022 22:22:07 +0000 (23:22 +0100)] 
GEODE-9326: Replace ACE_Get_Opt (#814)

 - Replaced ACE_GET_Opt by Boost alternative.
 - Due to the include changes there was a compilation error in fw_spawn,
   which is solved by this commit.
 - Fixed FD_SETSIZE to 1024 for Win32 to avoid coredump in ACE_Process

7 months agoGEODE-9324: Remove ACE_Task references (#812)
Mario Salazar de Torres [Thu, 17 Feb 2022 21:24:58 +0000 (22:24 +0100)] 
GEODE-9324: Remove ACE_Task references (#812)

 - Removed all references to ACE_Task
 - Changed variable names to comply with styleguides

7 months agoGEODE-9268: Fix PdxInstance handling after cluster restart (#806)
Mario Salazar de Torres [Wed, 16 Feb 2022 20:19:25 +0000 (21:19 +0100)] 
GEODE-9268: Fix PdxInstance handling after cluster restart (#806)

* GEODE-9268: Fix PdxInstance handling after restart

 - In scenarios where an PdxInstance is obtained and later used, it
   might happen a coredump if the PdxTypeRegistry is cleaned up after the
   PdxInstance is obtained. This happens on those scenarios where
   redundancy is completly lost.
 - This change refactors PdxInstance handling so in all cases, the PdxType
   used is the one owned by the PdxInstance.
   Whenever the PdxInstance is to be written, a check is executed to
   ensure the cluster is aware of the PdxType and if not, register it.
 - Removed PdxInstance serialization retries, as with the new approach
   is not needed anymore.
 - Removed UnknownPdxTypeException exception. Instead whenever a PdxType
   is requested and not present, an IllegalStateException exception is
   thrown, as it happens in the Java client.
 - Now PdxInstance is not serialized whenever created. Instead, it's PDX
   byte stream is generated on-demand. Note that the PdxInstance will be
   serialized before being put into a server, as it was done before this
   change.
 - Fixed TcrMessage deserialization whenever a PdxType is requested by
   its ID and no PdxType was found.
 - Fixed incPdxInstanceCreations so it's incremented strictly whenver a
   PdxInstance is created, and not whenever a PdxInstance is
   deserialized.
 - Fixed IT PdxTypeRegistry cleanupOnClusterRestart logic and renamed to
   cleanupOnClusterRestartAndPut. This test was supposed to verify that
   if a PdxInstance is created, and after that the cluster is restarted,
   there is no coredump while writting it to a region. Instead it was
   creating a PdxInstance before and after the cluster, but it has been
   fixed to work as initially intended.
 - Created a new IT PdxTypeRegistry cleanupOnClusterRestartAndFetchFields
   to verify the issue described in the first bullet is not causing a coredump.
 - Removed old IT testThinClientPdxInstance TS as there is a equivalent new
   IT TS named PdxInstanceTest.
 - Fixed some ITs to work accordingly the the new code.
 - Fixed PdxInstanceImplTest.updatePdxStream to work accordingly with
   the code changes.

* GEODE-9268: Revision 1

 - Refactored member attributes for NestedPdxObject classes in order to
   follow style guidelines.
 - Refactored member attributes for PdxInstanceImpl class in order to
 follow style guidelines.
 - Moved default bytes variables from static to constants inside an anon
   namespace within PdxInstanceImpl cpp file.
 - Renamed 'pft' variables to 'field' inside PdxInstanceImpl in order to
   make the code more readable.
 - Changed return of PdxLocalWriter::getPdxStream to use the explicit
   constructor of std::vector<uint8_t>
 - Solved PdxInstanceTest.testNestedPdxInstance asserts description
   mismatch.
 - Used binary_semaphore inside PdxTypeRegistryTest insteda of an
   in-place CacheListener in order to listener for cluster start/stop
   events.
 - Revert back changes to the OQL used inside
   PdxTypeRegistryTest.cleanupOnClusterRestartAndPut
 - Added test descriptions for both tests inside PdxTypeRegistryTest.

* GEODE-9268: Revision 2

 - Addressed several others assert descriptions mismatch.

7 months agoGEODE-10040: Increase wait timeout for gfsh (#924)
Michael Martell [Wed, 16 Feb 2022 18:27:23 +0000 (10:27 -0800)] 
GEODE-10040: Increase wait timeout for gfsh (#924)

7 months agoGEODE-10004: support older geode versions (#917)
Michael Martell [Tue, 15 Feb 2022 16:13:45 +0000 (08:13 -0800)] 
GEODE-10004: support older geode versions (#917)

* Check for class existence instead of version.

7 months agoGEODE-100043: Solve TransactionCleaningTest errors (#923)
Mario Salazar de Torres [Fri, 11 Feb 2022 19:21:09 +0000 (20:21 +0100)] 
GEODE-100043: Solve TransactionCleaningTest errors (#923)

- From time to time this test was failing with the error "Concurrent
   modification in the cache". Thing is this test was not intended to
   check concurrent modifications.
 - In GEODE-9322 this test was modified to avoid a race-condition
   whenever restarting the server. To do so, it was needed to enable
   notification subscription and change the region shortcut to
   CACHE_PROXY. In doing so, concurrent checks are also enabled, and due
   to some strange race-condition this is occasionally leading to the exception.
 - Concurrency checks were disabled for the region to avoid the
   race-condition to happen at all

7 months agoAdd license text to some doc sources (#922)
Dave Barnes [Wed, 9 Feb 2022 23:16:16 +0000 (15:16 -0800)] 
Add license text to some doc sources (#922)

7 months agoGEODE-9959: Add FQDN in log upon SSL missconfig (#904)
Mario Salazar de Torres [Wed, 9 Feb 2022 20:13:01 +0000 (21:13 +0100)] 
GEODE-9959: Add FQDN in log upon SSL missconfig (#904)

- Whenever there is an SSL missoconfiguration while trying to reach a
   locator, an AuthenticationRequiredException is thrown and a log is
   written. Currently FQDN of the locator is not logged.
 - This commit adds the locator FQDN in order to further troubleshoot
   this kind of scenarios.
 - Also, in order to ease implementation, toString method was
   implemented for ServerLocation.

7 months agoGEODE-10030: Remove obsolete cross-reference in geode-native user docs (#921)
Dave Barnes [Wed, 9 Feb 2022 19:05:03 +0000 (11:05 -0800)] 
GEODE-10030: Remove obsolete cross-reference in geode-native user docs (#921)

7 months agoGEODE-9960: Remove strong name signing support for .net assemblies (#905)
Blake Bender [Wed, 9 Feb 2022 18:25:06 +0000 (10:25 -0800)] 
GEODE-9960: Remove strong name signing support for .net assemblies (#905)

* Remove strong name signing support for .net assemblies
- This has caused problems with build pipelines, and is no longer used by .net runtimes
- Make internals visible to test DLL

7 months agoGEODE-10016: Add map of threadId to threadName (#918)
Michael Martell [Tue, 8 Feb 2022 14:48:17 +0000 (06:48 -0800)] 
GEODE-10016: Add map of threadId to threadName (#918)

* Save threadName to thread_local storage
* Per review, get/setThreadName doesn't need tid
* Add threadName in parens after tid
* Update gnmsg for threadName
* Move accessors to Log class
* Inline threadNames instead of constants
* Remove getThreadName()

7 months agoGEODE-9323: Remove ACE from tests/cpp projects (#811)
Mario Salazar de Torres [Mon, 7 Feb 2022 16:16:05 +0000 (17:16 +0100)] 
GEODE-9323: Remove ACE from tests/cpp projects (#811)

* GEODE-9323: Remove ACE from tests/cpp projects

 - Removed all ACE references from tests/cpp projects.
 - Also moved SerializationRegistry inline functions to the
   source file in order to solve compilation.

* GEODE-9323: Empty commit to trigger CI

7 months agoGEODE-9896: bump sqlite-fx to 1.0.115.5 (#916)
M. Oleske [Mon, 7 Feb 2022 15:57:51 +0000 (07:57 -0800)] 
GEODE-9896: bump sqlite-fx to 1.0.115.5 (#916)

7 months agoGEODE-9896: Bump sqlite to 3.37.2 (#914)
M. Oleske [Mon, 7 Feb 2022 15:55:37 +0000 (07:55 -0800)] 
GEODE-9896: Bump sqlite to 3.37.2 (#914)

Authored-by: M. Oleske <michael@oleske.engineer>
7 months agoGEODE-9896: Bump ace to 7.0.6 (#913)
M. Oleske [Mon, 7 Feb 2022 15:49:35 +0000 (07:49 -0800)] 
GEODE-9896: Bump ace to 7.0.6 (#913)

- fix building on macos monterry

Authored-by: M. Oleske <michael@oleske.engineer>
7 months agoBumping Geode version to 1.14.3 for CI (#912)
Dick Cavender [Thu, 3 Feb 2022 17:42:41 +0000 (09:42 -0800)] 
Bumping Geode version to 1.14.3 for CI (#912)

7 months agoFix typo in gfsh command ('hreshold' vs 'threshold') (#911)
Blake Bender [Thu, 3 Feb 2022 17:29:54 +0000 (09:29 -0800)] 
Fix typo in gfsh command ('hreshold' vs 'threshold') (#911)

Co-authored-by: Blake Bender <bblake@vmware.com>
8 months agoEliminate duplicate REGISTER_INTEREST message (#906)
Blake Bender [Tue, 18 Jan 2022 15:13:05 +0000 (07:13 -0800)] 
Eliminate duplicate REGISTER_INTEREST message (#906)

- first endpoint in m_redundantEndpoints was originally supposed to
be left out of the for loop.  This broke during a refactor roughly three years ago, moving from old-school iterator to a ranged for loop.

Co-authored-by: Blake Bender <bblake@vmware.com>
9 months agoBumping Geode version to 1.14.2 for CI (#902)
Owen Nichols [Thu, 16 Dec 2021 10:05:31 +0000 (02:05 -0800)] 
Bumping Geode version to 1.14.2 for CI (#902)

9 months agoBumping Geode version to 1.14.1 for CI (#900)
Owen Nichols [Sat, 11 Dec 2021 21:59:55 +0000 (13:59 -0800)] 
Bumping Geode version to 1.14.1 for CI (#900)

9 months agoFix to disable auto updates on rhel8 (#898)
Michael Martell [Thu, 2 Dec 2021 17:29:38 +0000 (09:29 -0800)] 
Fix to disable auto updates on rhel8 (#898)

10 months agognmsg - decode REGISTER_INTEREST message (#895)
Blake Bender [Mon, 22 Nov 2021 20:40:17 +0000 (12:40 -0800)] 
gnmsg - decode REGISTER_INTEREST message (#895)

* Make log-rolling work with relative paths
* Don't report individual chunk stats, just stats for the complete
response.
* Parse REGISTER_INTEREST client message details
* Tiny performance improvement to regex parsing
  - Really need to spend time tuning these

10 months agoGEODE-9804: new register interest tests (#894)
Michael Martell [Sat, 20 Nov 2021 01:30:22 +0000 (17:30 -0800)] 
GEODE-9804: new register interest tests (#894)

* Fixes incorrect fetching of initial entries.
* Makes InterestResultPolicy a proper enum.
* Adds integration benchmark for register interest.
* Fixes for MSVC errors.
* Don't ask for keys if not caching locally.
* Replace sleep with CacheListener events.
* Use latches for syncrhonization with listener.
* Change boost loglevel to warning.
* Encapsulate synchronization in CountdownCacheListener.

Co-authored-by: Jacob Barrett <jbarrett@pivotal.io>
10 months agoGEODE-9741: Fixes Boost boostrap command. (#890)
Jacob Barrett [Tue, 2 Nov 2021 22:52:38 +0000 (15:52 -0700)] 
GEODE-9741: Fixes Boost boostrap command. (#890)

10 months agoGEODE-9741: Pass full toolkit version to external build systems. (#889)
Jacob Barrett [Tue, 2 Nov 2021 22:04:44 +0000 (15:04 -0700)] 
GEODE-9741: Pass full toolkit version to external build systems. (#889)

* Require at least CMake 3.19.7 for 3 component toolkit version support.
* LGTM downloads latest CMake.

10 months agoGEODE-9741: Use v142 Toolchain in Visual Studio 2019 CI Pipeline (#884)
Michael Martell [Tue, 26 Oct 2021 19:27:06 +0000 (12:27 -0700)] 
GEODE-9741: Use v142 Toolchain in Visual Studio 2019 CI Pipeline (#884)

* Add v142 to VS 2019 and build with it
* Use explicit versions
* Use lowest versions for tools & sdk

11 months agomake sure geode version matches from start to fan-in (#887)
Owen Nichols [Mon, 25 Oct 2021 18:07:41 +0000 (11:07 -0700)] 
make sure geode version matches from start to fan-in (#887)

11 months agoFix Ubuntu images in CI (#886)
Blake Bender [Sat, 23 Oct 2021 15:20:53 +0000 (08:20 -0700)] 
Fix Ubuntu images in CI (#886)

* Don't perform upgrade when creating instance
- This appears to be breaking SSH connectivity

11 months agoGEODE-9553: remove all remaining printf variations from C++ code (#885)
Blake Bender [Fri, 22 Oct 2021 13:57:24 +0000 (06:57 -0700)] 
GEODE-9553: remove all remaining printf variations from C++ code (#885)

11 months agoGEODE-9639: Removed the use of std::unary_function. (#874)
Matthew Reddington [Fri, 22 Oct 2021 13:54:54 +0000 (08:54 -0500)] 
GEODE-9639: Removed the use of std::unary_function. (#874)

* Removed remaining deprecations that can be removed.

11 months agoGEODE-9719: Start/Stop cluster in a test fixture (#881)
Michael Martell [Mon, 18 Oct 2021 21:01:14 +0000 (14:01 -0700)] 
GEODE-9719: Start/Stop cluster in a test fixture (#881)

* Start/Stop cluster in a test fixture
* Create an authRegion in same cluster
* Turn off cluster output after all tests run

11 months agoGEODE-9553: remove sprintf usage in test utility code (/tests/cpp) (#883)
Blake Bender [Mon, 18 Oct 2021 19:33:15 +0000 (12:33 -0700)] 
GEODE-9553: remove sprintf usage in test utility code (/tests/cpp) (#883)

11 months agoGEODE-9688: net-core builds only for x64 (#878)
Ernie Burghardt [Tue, 12 Oct 2021 22:09:17 +0000 (15:09 -0700)] 
GEODE-9688: net-core builds only for x64 (#878)

* GEODE-9688: net-core builds only for x64
* Fix path for install
* Specifiy platform for test runs

11 months agoFix build break (#882)
Blake Bender [Tue, 12 Oct 2021 22:08:36 +0000 (15:08 -0700)] 
Fix build break (#882)

- changes to Cluster ctor crossed paths on the way into develop with a
  change that added some new tests

11 months agoGEODE-9600: Add Cluster Support to NetCore Test Framework (#880)
Michael Martell [Tue, 12 Oct 2021 15:44:25 +0000 (08:44 -0700)] 
GEODE-9600: Add Cluster Support to NetCore Test Framework (#880)

* Add cluster support to netcore-integration-test and netcore-session integration-test
* Add top level solution for netcore
* Add programmatic cluster files in shared folder for use by both test projects
* Address Rat check issue: Add using statement to new Process
* Remove gfsh based cluster scripts, now supported in cluster.cs, etc.
* Update refs to Cluster.cs, etc. in project files
* cmake configure Config.cs.in so environment vars are not required
* Remove old netcore solution file
* Cleanup per review: Geode .Net Core, not .net Core

11 months agoGEODE-9634: Workaround for failing test. (#872)
Jacob Barrett [Tue, 12 Oct 2021 14:40:39 +0000 (07:40 -0700)] 
GEODE-9634: Workaround for failing test. (#872)

* Updates test.
* Fixes hostname issues with tests.
* Fix default locator address init.

11 months agoGEODE-9591: Don't retry function when Function.isHA=false (#866)
Jakov Varenina [Tue, 12 Oct 2021 14:32:30 +0000 (16:32 +0200)] 
GEODE-9591: Don't retry function when Function.isHA=false (#866)

* GEODE-9591: Don't retry function when Function.isHA=false

Fault:
Client clears result by calling ResultCollector::clearResult and
re-executes the function even thought Function.isHa() is set to false.

Fix:
Client will not clear result and retry the function, but it will fail with the
received exception from server.

Added integration test:
1.FunctionExecutionTest.testThatFunctionExecutionThrowsExceptionNonHAWithFilter
2.FunctionExecutionTest.testThatFunctionExecutionThrowsExceptionNonHA

The integration test reproduces the problem in a following way:

1. Populate region so that all buckets are created
2. Shutdown one of the three available servers
3. Peforme rebalance operation, so that buckets are recovered on running
servers
4. Execute the function. Function execution will fail with the
InternalFunctionInvocationTargetException exception, because client will
try to execute single hop function with old PR meatada information (PR
metadata received before the rebalance operation).

* ASSERT_THROW used instead of try-catch

* Check whether isHA==false before clearing results

It is possible that function exeuction fail on multiple servers, and in
that case only first exception will be thrown, and any subsequent
exeception will be ignored. Additionally result should not be cleared in
any case when isHa==false.

* Rebalance cmd was not executed

Test cases are changed to do rebalance operation
after new server is introduced in cluster.

* Client metadata wasn't complete when function is executed

Since client metadata was not complete the function was not exectued in
a single hop manner, and therfore exeption is not reproduced.

This commit is fixed in a way that now client is connected to only
one server. This way client metadata will be updated more frequently,
and client metadata will be complete when function is executed.

11 months agoGEODE-9553: remove usage of sprintf from tests (#879)
Blake Bender [Tue, 12 Oct 2021 14:22:50 +0000 (07:22 -0700)] 
GEODE-9553: remove usage of sprintf from tests (#879)

11 months agoGEODE-9631: New tests for reauthentication (#876)
Michael Martell [Wed, 6 Oct 2021 19:47:06 +0000 (12:47 -0700)] 
GEODE-9631: New tests for reauthentication (#876)

* New tests for reauthentication and durability
* Add new SimulatedExpirationSecurityManager for reauthentication testing

11 months agoGEODE-9608: Remove Release config, add Opimization and pdb (#877)
Michael Martell [Mon, 4 Oct 2021 22:23:00 +0000 (15:23 -0700)] 
GEODE-9608: Remove Release config, add Opimization and pdb (#877)

11 months agoAdd a fan-in job and print out the properties of the Geode build used for testing... 868/head
Owen Nichols [Fri, 1 Oct 2021 23:39:06 +0000 (16:39 -0700)] 
Add a fan-in job and print out the properties of the Geode build used for testing (#875)

11 months agoGEODE-9569: Native client UG build script keeps doc after generating it (#873)
Alberto Bustamante Reyes [Fri, 1 Oct 2021 08:21:30 +0000 (10:21 +0200)] 
GEODE-9569: Native client UG build script keeps doc after generating it (#873)

11 months agoGEODE-9553: remove sprintf and snprintf from geode native (#871)
Blake Bender [Thu, 30 Sep 2021 17:50:42 +0000 (10:50 -0700)] 
GEODE-9553: remove sprintf and snprintf from geode native (#871)

* GEODE-9553: Replace instances of sprintf and snprintf in geode native
- Occasionally one of these will pop up as an error in a new compiler,
  and really they have no place in a modern C++ codebase.
- Just the product code (mostly) for now, we'll do test code in another PR

11 months agoGEODE-9576: Fix for single-hop function execution (#864)
Jakov Varenina [Thu, 30 Sep 2021 15:45:21 +0000 (17:45 +0200)] 
GEODE-9576: Fix for single-hop function execution (#864)

* GEODE-9576: Fix for single-hop function execution

Fault:
"InternalFunctionInvocationTargetException: Multiple target nodes
found for single hop operation" occurs on server when executing
function in a single hop manner for all buckets during the period when
client bucket metadata doesn't contains all buckets locations.

Fix:
The client will execute function in a non single-hop manner until it
recevies all buckets locations. This solution is aligned with java
client.

* Updates after comments

* Fix for clang-tidy errors

* Fix for clang-tidy

Use old MOCK_METHOD1 because clang-tidy reports error for the new macro.

11 months agoGEODE-9609: Fix Warning in SessionState (#869)
Michael Martell [Wed, 29 Sep 2021 19:52:19 +0000 (12:52 -0700)] 
GEODE-9609: Fix Warning in SessionState (#869)

12 months agoGEODE-9637: Disable Wan deserialization test (#870)
Blake Bender [Fri, 24 Sep 2021 18:40:17 +0000 (11:40 -0700)] 
GEODE-9637: Disable Wan deserialization test (#870)

- Just want a green CI until GEODE-9634 is resolved.

Co-authored-by: Blake Bender <bblake@vmware.com>
12 months agoGEODE-9549: enable netcore tests in CI (#867)
Blake Bender [Wed, 22 Sep 2021 14:18:09 +0000 (07:18 -0700)] 
GEODE-9549: enable netcore tests in CI (#867)

* GEODE-9549: Enable .net core tests in CI
- Enable test running job in yml
* GEODE-9549: Build NetCore and NetCore.Test projects individually
- This simplifies the cmake config, and makes the post-build file copy
  steps more reliable
* GEODE-9549: Copy C bindings lib to binary dir for tests
- Give Java project a reasonable name
* GEODE-9549: Use ctest to run netcore tests
- Eliminates need for start/stop scripts, all done via ctest now
* GEODE-9549: Make .net core project build in sequence
- Still getting sharing violations on parallel builds, so need to ensure
  none of these build at the same time.

Co-authored-by: Jacob Barrett <jabarrett@vmware.com>
12 months agoGEODE-9189: Upgrade dependencies.
Jacob Barrett [Wed, 18 Aug 2021 21:34:44 +0000 (14:34 -0700)] 
GEODE-9189: Upgrade dependencies.

Boost to 1.76.
ACE to 7.0.3.
GTest 1.11.0.
Benchmark 1.6.0.
SQLite to 3.36.0

Removed unused colored_printing, which depends on internals of gtest.

12 months agoGEODE-7282: Geode Native Client .NET guide: Fix capitalization issues (#863)
Dave Barnes [Wed, 8 Sep 2021 15:43:00 +0000 (08:43 -0700)] 
GEODE-7282: Geode Native Client .NET guide: Fix capitalization issues (#863)

12 months agoBumping Geode version to 1.14.0 for CI (#865)
Nabarun Nag [Sat, 4 Sep 2021 00:40:46 +0000 (17:40 -0700)] 
Bumping Geode version to 1.14.0 for CI (#865)

12 months agoGEODE-9551: Fixes API return and parameter types. (#860)
Jacob Barrett [Fri, 27 Aug 2021 19:13:18 +0000 (12:13 -0700)] 
GEODE-9551: Fixes API return and parameter types. (#860)

13 months agoGEODE-9464: Add Asp.Net Core session state sample application (#856)
Michael Martell [Thu, 26 Aug 2021 20:33:36 +0000 (13:33 -0700)] 
GEODE-9464: Add Asp.Net Core session state sample application  (#856)

* Add AspNetCore GeodeSession Sample
* Create region for sessionState in startserver
* Add Apache license header

13 months agoFix buffer overflow gripes in RHEL-8 build (#861)
Blake Bender [Thu, 26 Aug 2021 20:20:33 +0000 (13:20 -0700)] 
Fix buffer overflow gripes in RHEL-8 build (#861)

- RHEL-8 CI images appear to have picked up a new compiler that's
  flagging some broken sprintf code in a couple of legacy tests.

13 months agoGEODE-9550: Native Client user guide: warn against floating point values in keys...
Dave Barnes [Wed, 25 Aug 2021 21:28:39 +0000 (14:28 -0700)] 
GEODE-9550: Native Client user guide: warn against floating point values in keys (#857) (#859)

13 months agoFixes disable of LTO on RHEL8. (#858)
Jacob Barrett [Wed, 25 Aug 2021 21:14:18 +0000 (14:14 -0700)] 
Fixes disable of LTO on RHEL8. (#858)