GEODE-10294: Compare invalid token during putIfAbsent retry. (#7679)

 * During putIfAbsent retry, comparing invalid token value when
   putIfAbsent of a null value instead.

GEODE-10306 Fixing an order issue that can lead to problems when stopping (#7682)

When stopping the cache server, the acceptor is last which should not be the case. It should be first so new data stops coming in.

GEODE-10302: Increase call stack timeout for upgrade tests (#7691)

To be 3h45m, 15 minutes shy of the newly increased task timeout.

GEODE-10307: Doc changes needed for enable security-manager property (#7687)

* GEODE-10307: Doc changes needed for enable security-manager property

GEODE-10303: Increase memory for windows-gfsh-distributed-test (#7685)

Authored-by: Robert Houghton <rhoughton@pivotal.io>

GEODE-10302: Increase upgrade test timeout to 4h (#7683)

GEODE-10299: update examples for creating dynamic regions (#7681)

GEODE-10271: ConnectionProxyJUnitTest cleanup (#7652)

GEODE-10283: Remove problematic buildSrc project, replacing with includeBuild projects (#7600)

* Extract DependencyConstraints plugin from buildSrc
* Extract japicmp plugin customization from buildSrc
* Test isolation from buildSrc
* Disable jacoco on all test facets by default

Co-authored-by: Your Name <you@example.com>

GEODE-10288: Fix property assignment syntax (#7678)

GEODE-10289: Argument file for JDK 17 (#7673)

* GEODE-10289: Argument file for JDK 17

The argument file was generated on Linux using OpenJDK 17.0.2

* Add arg file to assembly_content.txt

GEODE-10293: Modify .asf.yaml to require CodeQL, drop LGTM (#7674)

GEODE-10293: Modify .asf.yaml to require CodeQL, drop LGTM

GEODE-10288: Define JDK 8, 11, 17 homes for upgrade tests (#7675)

GEODE-10282: Migrate from springfox to springdoc (#7659)

springfox swagger is no longer maintained, so springdoc is the best choice going forward.

Enable GitHub CodeQL workflow

GEODE-10284: Add partition-listener option to gfsh create region command (#7666)

* Update documentation

Co-authored-by: Dave Barnes <daveba@vmware.com>

GEODE-10089: Roll develop to 1.16.0 (#7668)

Now that support/1.15 has been created, develop advances.

GEODE‑10046: fix classgraph version in LICENSE (#7663)

GEODE-10206: clean up product references to CMS (#7646)

* the command line validator no longer warns if CMS is not configured if the JDK does not support CMS
* cleaned up comments that refer to CMS

Revert "GEODE-9484: Improve sending message to multy destinations (#7381)" (#7655)

This reverts commit 62cd12c7f0bbb3d092011555e714e57ce041791a.

GEODE-10278: Remove the geode-for-redis module

Removing geode-for-redis module and all of the references to it
from CI, scripts, docs, expected assembly and pom context.

GEODE-10272: Handle RejectedExecutionException during shutdown. (#7651)

 * Do not throw RejectedExecutionException if it is caused by cache
   closing.

GEODE-9390: Guarding membership addition code paths to omit membership duplicates (#7639)

Co-authored-by: Matthew Reddington <mreddington@pivotal.io>

GEODE-10046: Bump 3rd-party dependency versions (#7650)

Geode endeavors to update to the latest version of 3rd-party
dependencies on develop wherever possible.  Doing so increases the
shelf life of releases and increases security and reliability.
Doing so regularly makes the occasional hiccups this can cause easier
to pinpoint and address.

Dependency bumps in this batch:
* Bump classgraph from 4.8.145 to 4.8.146
* Bump micrometer from 1.8.4 to 1.8.5
* Bump netty-handler from 4.1.75 to 4.1.76
* Bump spring-boot-starter-web from 2.6.6 to 2.6.7
* Bump spring-hateoas from 1.4.1 to 1.4.2
* Bump spring-ldap-core from 2.3.6 to 2.3.7
* Bump spring-security from 5.6.2 to 5.6.3

GEODE-10275: Bump spring from 5.3.18 to 5.3.19 (#7647)

Geode endeavors to update to the latest version of 3rd-party
dependencies on develop wherever possible.

GEODE-10215: Document warning for parallel gws (#7623)

GEODE-10242: Colocated buckets share primary move lock (#7630)

 * All colocated buckets now share the same primaryMoveReadWriteLock.
   When parent bucket is being moved, no operations will be executed on
   child buckets as well. So moving primary for all colocated buckets
   shold be faster, and there is no need to hold parent locks anymore.

* Added.a dunit test to validate the fix works.

GEODE-10274: Install GCP ops-agent in CI images (#7642)

* GEODE-10274: Install GCP ops-agent in CI images

Replaces old stackdriver-agent, to enable CI utilization metrics

GEODE-10263: add additional import for VMStats50 (#7637)

GEODE-10264: remove 'connect to the server from your application' section (#7641)

Revert "GEODE-10249: Adds BufferPoolMXBean stats. (#7607)" (#7638)

This reverts commit 23e554bbe633c7fca189c788d34c85778924bafb.

GEODE-9484: Improve sending message to multy destinations (#7381)

* GEODE-9484: New solution to first try only one attempt to create all connections

GEODE-10266: update SeveralGatewayReceiversWithSamePortAndHostnameFor… (#7633)

* GEODE-10266: update SeveralGatewayReceiversWithSamePortAndHostnameForSendersTest

Bugfix/GEODE-10228 DurableClientTestCase.testDurableHAFailover is failing (#7608)

- The test was failing because it didn't wait for the
HARegionQueue to clear before shutting down the durable
client for test. Thus when it came back up, there was
an extra message in the queue.

- Reverse the order of readyforevents and registerinterest

- adding a close for the control listener

- Starting the server is not synchronous adjusted test accordingly

GEODE-10260: refactor out Filter interface to use Predicate (#7627)

GEODE-10260: make sure message is added before they are processed. (#7628)

GEODE-10263: add opens required by geode features (#7632)

added opens required by geode features and added javadoc comments
that link to the class in geode that requires an open or export

GEODE-10184: Verify the status file is accessible before handling the request. (#7582)

GEODE-10020: Introduction of option to gradually activate pinging (#7517)

* GEODE-10020: Introduction of option to gradually activate pinging toward destination

GEODE-10258: Assert at least 1 invocation (#7624)

ClearDuringNetSearchOplogRegressionTest uses an await statement to
allow the timing of a region clear to be correct. An invocation of
CacheObserver.afterSettingDiskRef() is used as the trigger for the
await to pass and allow the clear to begin.

The test was failing due to CacheObserver.afterSettingDiskRef() being
invoked more than once in the await statement, but this method is
expected to be invoked multiple times during the test, so occasionally
the await would check and find 0 invocations,then check again and find
2 or more, causing the test to fail.

Replacing the times(1) in the assertion with atLeast(1) allows the test
to pass while preserving the intended behaviour of the await.

Some incidental code clean-up was also done in the test to remove
compiler warnings.

Authored-by: Donal Evans <doevans@vmware.com>

GEODE-10248: Adding a new Suspicious Strings exception (#7612)

for Management Requests that get logged and a test

GEODE-10249: Adds BufferPoolMXBean stats. (#7607)

* Adds new class for BufferPoolMXBean.
* Adds tests for BufferPoolStats.
* Cleanup VMStats50.

GEODE-10209: Use AvailablePortHelper in InternalCacheForClientAccessDistributedTest (#7613)

Prevent BindException by using AvailablePortHelper.

GEODE-10254: Use original hostname if host is null. (#7618)

If original hostname was not resolvable then host is null. Use hostname
if host is null when marshaling.

GEODE-9466: change offheap to not use Bits.unaligned (#7611)

offheap no longer depends on java.nio.Bits.unaligned

GEODE-10243: Fail early if old client auth expires (#7603)

* change the default "waitForReAuth" time to 60 seconds

GEODE-10229: GII image should fill disk region RVV's exceptions. (#7602)

* GEODE-10229: GII image should fill disk region RVV's exceptions.
    There're 2 issues: Tombstone in GII image should save version tag into disk region RVV
    even the tombstone is the same as local one. Disk region RVV holder did not use bitset.
    Only bitset based holder can fill special exception. Should do it for non-bitset holder too.

GEODE-10245: Upgrade classgraph 4.8.143 -> 4.8.145 (#7605)

- This fixes an issue discovered with the ordering of 'manifest only'
  jars and classgraph unable to find classes.

GEODE-10251: Make DescribedExternalResource report all exceptions (#7609)

* GEODE-10251: Make DescribedExternalResource report all exceptions

Co-authored-by: Dale Emery <demery@vmware.com>
Co-authored-by: Kirk Lund <klund@apache.org>
* Fix test name

Co-authored-by: Dale Emery <demery@vmware.com>
Co-authored-by: Kirk Lund <klund@apache.org>
* Spotless

Co-authored-by: Dale Emery <demery@vmware.com>
Co-authored-by: Kirk Lund <klund@apache.org>
Co-authored-by: Kirk Lund <klund@apache.org>

GEODE-10246: fix OutOfMemoryDUnitTest for jdk17 (#7606)

* test now uses ZGC  on jdk17
* product change: added ZHeap to the list of known memoryPoolMXBeans

GEODE-10161: Remove unnecessary synchronization from RedisList (#7579)

- Now, with the addition of versioning and synchronization at the
  higher-level methods, all the synchronization can be removed from the
  small, helper methods we have.
- We still need to ensure that there is synchronization between toData
  and within methods that are mutating the list.
- Inline various helper methods in RedisList as they are not adding any
  value.
- Use AvailablePortHelper to set up the port that the crashing VM will
  use. Without this there is interference from other tests when running
  multiple tests concurrently (in the stressTest job).
- Ran at least 1000 iterations of all list-related DUnit tests without
  any failures.

GEODE-10023: do not @link to type params (#7601)

Some javadocs contained @link tags pointing to type parameters. These
should have been @code tags, as type parameters cannot be linked to

Authored-by: Donal Evans <doevans@vmware.com>

GEODE-10197: fix test for jdk17

OutOfMemoryDUnitTest now only uses CMS on jdk less than 14. (#7595)

GEODE-10023: Fix javadocs (#7594)

This commit reduces the number of errors reported in the gradle build
log due to javadocs from over 3000 to 0. It also reduces the number of
warnings reported in the gradle build log due to javadocs from over
4000 to 0.

After these changes, all published javadocs are now syntactically
correct, although no guarantee is made as to the content of the
documentation. Javadocs in classes that did not produce warnings or
errors in the gradle build log were not touched.

For classes in the geode-dunit, geode-junit, geode-assembly and
geode-concurrency-test modules, if a javadoc could be converted to
a non-javadoc comment without losing any formatting or tags, it was
converted, since those classes are not intended for use by Geode users

The changes made were:
 - Add missing @return, @param and @throws tags
 - Fix all errors due to incorrect HTML
 - Fix all errors due to improperly escaped characters
 - Fix all broken @link tags
 - Several minor spelling errors and typos fixed where spotted

In addition to this clean-up, the compiler settings were modified so
that javadocs warnings and errors will be output in the gradle build
log, and if any are present, the build will fail.

Authored-by: Donal Evans <doevans@vmware.com>
Co-authored-by: Patrick Johnson <jpatrick@vmware.com>

GEODE-10238: add open for jetty9 on java 17 (#7593)

* added open for sun.security.provider for jetty9

* test now expects stack to contain "Connection refused" instead of "Connection refused "

GEODE-10223: Fix BlockingCommandListnerTest to be less flaky. (#7592)

* GEODE-10223: Fix BlockingCommandListnerTest to be less flaky.

testTimeoutIsAdjusted is flaky on windows machines. Adding a delay will make it less flaky.

Authored-by: Bala Kaza Venkata <bkazavenkata@vmware.com>

GEODE-10185: fix MemoryOverheadIntegrationTest for jdk17 (#7591)

* If ObjectTraverser has been told to skip static fields
then it will no longer attempt to make static fields accessible.
* cleaned up ObjectTraverser class
* added unit test that verifies static fields will not be cached if they are not visited
removed statics from ObjectTraverser to aid unit testing
cleaned up warnings in ObjectGraphSizer and ObjectTraverser

GEODE-10035: fix BufferPool sys prop logic (#7587)

Now if either system property is set to true that takes precedence
and heap buffers will be used. Otherwise, direct buffers will be used.

Revert "Revert "GEODE-10126: Replace Geode for Redis properties with system properties (#7478)" (#7576)" (#7586)

This reverts commit 934f517e92854cbfeb09d46f8ed9de806ef3eccb.

GEODE-9759: replaces double-hyphens with &#8209;&#8209; (#7560)

* GEODE-9759: replaces double-hypens with &#8209;&#8209;

Co-authored-by: Max Hufnagel <mhufnagel@vmware.com>

GEODE-8228: Use assertThat without embedded comparisons.

GEODE-8228: Rename tests to current convention.

GEODE-8228: Await on stats.

GEODE-8228: Cleanup static analyzer in WANTestBase.

GEODE-8228: Refactored to AssertJ.

GEODE-10235: fix --maxHeap for java 17 (#7585)

* --maxHeap no longer forces CMS if the java version is 14 or later. It was in 14 that CMS was dropped from the JVM.

* product code now uses SystemUtils to determine java version instead of system property
tests now use junit5 EnabledForJreRange to determine java version instead of system property
fixed some other tests that were broken in jdk17 by this change

GEODE-10146: Make GradleBuildWithGeodeCoreAcceptanceTest run on JDK 17 (#7570)

PROBLEM

GradleBuildWithGeodeCoreAcceptanceTest attempts to run Gradle v5.1.1
using the test JVM. This fails when the test is run in JDK 17, because
Gradle 5.1.1 does not support JDK 17.

SOLUTION

Configure the test's "inner" build to use the value of the GRADLE_JVM
environment variable as its JVM. This makes it use the same JVM that we
use to build Geode.

GEODE-10193: Replace openhft compiler with geode-junit compiler (#7580)

PROBLEM

`QueryPdxDataDUnitTest` compiles and loads classes using the
`net.openhft.compiler` library, which attempts to access a field of
`Unsafe` that does not exist on JDK 17.

SOLUTION

Use the compiler system from `geode-junit`'s
`org.apache.geode.test.compiler` package.

GEODE-9512: close receiver connection if membership check timed out (#7409)

If a reader thread performs membership check and it fails, we should close the connection as the sender didn't pass the authentication check.

GEODE-6588: Cleanup missing @Deprecated (#7545)

Adds missing @Deprecated to all symbols with Javadoc tag @deprecated.

GEODE-10222: Handle null memberID in GII (#7569)

 - Instead of throwing from DiskVersionTag.replaceNullIDs() if memberID
 is null, check if the memberID is null after calling  replaceNullIDs() in
 InitialImageOperation.processChunk() and then return false if it is
 - Add debug log to record if this has occurred
 - Add unit test for the new behaviour

Authored-by: Donal Evans <doevans@vmware.com>

GEODE-10234: Do not generate tailKey if parallel wan not enabled. (#7583)

GEODE-10230: PDX UPDATE/DELETE in Mgmt REST API (#7578)

Add update (HTTP put) and delete (HTTP DELETE) operations support for
PDX in the cluster management REST API service.

- Add and update unit and integration tests.
- Keep old update behaviour (NotImplementedException thrown) for cache
  configurations other than PDX.

GEODE-10232: reformat list for more reliable rendering (#7581)

GEODE-10204: Add Test Coverage for SizeableByteArrayList (#7566)

GEODE-10157: Add Radish Delta Unit Tests (#7558)

* GEODE-10157: Add Radish Delta Unit Tests

Unit tests for all the Geode-for-Redis delta types.

Co-authored-by: Ray Ingles <ringles@vmware.com>

GEODE-10225: Gfsh start commands add exports on JDK 11+ (#7572)

PROBLEM

When run on JDK 17, numerous acceptance tests fail because a launched
locator or server requires access to the following packages:
- java.base/sun.nio.ch
- java.management/com.sun.jmx.remote.security

SOLUTION

- Add a `MemberJvmOptions` class that reports the JDK-dependent options
  required to launch a locator or server on the current process's JDK.
- Change `StartLocatorCommand` and `StartServerCommand` to add those
  options to the command line when launching locators and servers.

FUTURE

Future PRs will likely put these opens/exports in an argument file, then
configure these commands to use the argument file. We are deferring that
until we identify additional needs for the argument file, so we can
choose an appropriate location for the argument file.

Co-authored-by: Dale Emery <demery@vmware.com>
Co-authored-by: Kirk Lund <klund@apache.org>
Co-authored-by: Kirk Lund <klund@apache.org>

GEODE-10097: Avoid Thread.sleep for re-auth in MessageDispatcher (#7556)

Revert "GEODE-10126: Replace Geode for Redis properties with system properties (#7478)" (#7576)

This reverts commit ee1c412d8af3e6311f569948c3248eddc1fe8009.

SocketException is now also handled by messenger.handleJGroupsIOException (#7530)

GEODE-10127: Reduces verbosity of log messages.

Removes SSLConfig from toString.
Logs with original toString/marshalForClient string.

GEODE-10127: Corrects serialization backwards compatibility issues.

Serialize but ignore unused fields.
Adds annotation to identify Java Serializable compatibility fields.

GEODE-10221: Fix sizing when creating a RedisList copy (#7568)

- The size must be updated as part of cloning an existing list.

GEODE-10077: Avoid callbacks in wan-copy region (#7391)

The wan-copy region command does not always prevent
that callbacks are executed at the remote site for
the entries copied.

The value set by the wan-copy region command for
generateCallbacks to false in the events sent via
the gateway sender was not being propagated by the server
that received the events in the remote site (via the gateway
receiver) when proxying the put by means of a PutMessage or
UpdateMessage in order to send the event to the server with
the primary bucket or to replicate the put in case of more
than one replica in the region.

The flags in the PutMessage and UpdateMessage have been used
to propagate the value of generateCallbacks.

GEODE-10203: SerialVersionUID added. (#7529)

* SerialVersionUID added to GetClusterConfigurationFunction.
* Changes to this class will prevent breaking backward compatibility.

GEODE-10122: P2P Messaging Handles TLS KeyUpdate Message  (#7449)

* Key expiration works for TLSv1.3 and GCM-based ciphers
* TLS KeyUpdate messages are processed correctly

GEODE-10214: Improve speed of JvmSizeUtils.roundUpSize (#7548)

This small change improves the speed of this method by about 30% (as per
basic JMH benchmarking). benchamrk1 is the original and benchmark2 the
change.

```
Benchmark                     Mode  Cnt     Score     Error  Units
PerformanceSample.benchmark1  avgt    5  3480.815 ± 387.131  us/op
PerformanceSample.benchmark2  avgt    5  2418.937 ± 586.209  us/op
```

GEODE-10127: Corrects NullPointerException in hashCode(). (#7559)

When unmarshalling an instance with a host name that can't be resolved a
NullPointerException is thrown in hashCode(). Use Objects.hash() to
resolve hash code to avoid NullPointerException.

GEODE-10046: Bump 3rd-party dependency versions (#7557)

Geode endeavors to regularly update 3rd-party dependencies to increase
shelf life, security and reliability of releases.

Dependency bumps in this batch:
* Bump classgraph from 4.8.141 to 4.8.143
* Bump jetty from 9.4.45.v20220203 to 9.4.46.v20220331
* Bump jna from 5.10.0 to 5.11.0
* Bump junit-pioneer from 1.6.1 to 1.6.2
* Bump lettuce-core from 6.1.6.RELEASE to 6.1.8.RELEASE
* Bump maven-artifact from 3.8.1 to 3.8.5
* Bump micrometer-core from 1.8.3 to 1.8.4
* Bump nebula.lint from 17.6.1 to 17.7.0
* Bump netty from 4.1.74.Final to 4.1.75.Final
* Bump rat from 0.7.0 to 0.7.1
* Bump shiro-core from 1.8.0 to 1.9.0
* Bump spotless from 6.2.2 to 6.4.1
* Bump spring-boot-starter-web from 2.6.5 to 2.6.6
* Bump swagger-annotations from 1.6.2 to 1.6.6
* Bump tomcat from 9.0.59 to 9.0.62

GEODE-9991: Refactor for consistency and add tests. (#7533)

* Combine common configuration into method for consistency.
* Adds tests for new extracted methods.

GEODE-9704: Ensure that register interest is called before ready for events (#7442)

- RegisterInterestOps need to happen before ReadyForEventsOp is sent
  These changes make sure that happens.

- Added an InterestResultPolicyCheck

Authored-by: Barry Oglesby <boglesby@users.noreply.github.com>
Un-ignored a test that will reproduce the issue
periodically during a number of runs. It is a flaky
test without the core fix.

Authored-by: Jinmei Liao <jiliao@pivotal.io>

GEODE-10121: Fix transactional redis commands (#7513)

* GEODE-10121: Fix transactional redis commands

MSET, SMOVE commands are transactional but they have not been working as
transactional. This commit will fix these commands to behave that way.

Authored-by: Bala Kaza Venkata <bkazavenkata@vmware.com>

GEODE-10160: fixes SizeableByteArrayList sizing (#7519)

* Update SizeableByteArrayList with overrides for LinkedList's set() and add()
methods to ensure memory overhead is updated appropriately. This helps to
correct sizing issues in other RedisList methods such as LINSERT and LTRM.

Co-authored-by: Jens Deppe <jdeppe@vmware.com>

GEODE-10216: Add test for existence of VM stats (#7551)

When we started running with JDK 11, some users noticed that a few stats had gone missing. There was a one line fix in test.gradle that was added as part of a bigger effort, but the change was not tested. This adds a test for the existence of the VM stats so that none of the stats go missing in the future.

also changes all the assertions to use assertj.

GEODE-10053: Use stored DistributionConfig to create SSL configurations (#7366)

* While creating SSLConfig no new DistributionConfigImpl is created.
* Now, the DistributionConfig is taken from the InternalLocator class.

GEODE-10198: Fix LINSERT's case-sensitivity for arguments (#7552)

LINSERT was case-sensitive with its arguments BEFORE/AFTER.Redis
implementation of LINSERT is not case-sensitive.

GEODE-10126: Replace Geode for Redis properties with system properties (#7478)

* GEODE-10126: Replace Geode for Redis properties with system properties

- All previous Geode for Redis properties that were defined in
  geode-core are replaced by java system properties.
  - gemfire.geode-for-redis-port
  - gemfire.geode-for-redis-bind-address
  - gemfire.geode-for-redis-bind-username
  - gemfire.geode-for-redis-redundant-copies
  - gemfire.geode-for-redis-enabled
- Redis is enabled (using defaults) if the `enabled` property is set to
  true OR if any of the other properties are explicitly set as java
  system properties.
- Introduce `RedisConfiguration` interface to access properties.
- Concrete implementation of `SystemPropertyBasedRedisConfiguration`
  that uses system properties to define Radish properties.
- Integration Tests use `TestRedisConfiguration` to avoid having to set
  system properties.
- Services started by Geode (`CacheService`s) that throw exceptions
  during initialization will cause the Cache to fail to startup.
  Previously any such errors were only logged.
- Default bind address is explicitly 0.0.0.0 which is more expressive than just an empty string

GEODE-10106: Handle NPE in QueueManagerImpl (#7471)

CacheClientNotifierDUnitTest failed with an NPE because queuedConnection is occationally null when trying to recover the primary.

This is solved by checking for null before calling any methods on that object.

- fixes spelling mistakes.
- the primary can be null when removing a connection, and when a new
  primary can't be found when recovering a connection.
- the null check protects us from both of those circumstances when
  scheduling the redundancy satisfier.

GEODE-10211: Correctly check persistence on accessor (#7546)

 * To set possible duplicate for an event, whether region is persistent
   need to be determined. Allow an accessor to check the setting through
   profiles.

* Changed the order of condition check so that no need to send the find
version tag message to peers if persistence is true.

GEODE-10116: Enable Redis TCL tests for Lists (#7521)

* GEODE-10116: Enable Redis TCL tests for Lists

Co-authored-by: Ray Ingles <ringles@vmware.com>