Merge branch 'release/1.8.0'

GEODE-6065: Continue event processing when hostname lookup fails (#2883)

Co-authored-by: Ryan McMahon <rmcmahon@pivotal.io>
Co-authored-by: Bill Burcham <bburcham@pivotal.io>

GEODE-6063 remove PublishArtifacts from Geode release pipelines (#2865)

GEODE-6053: Parameterized Queries fixed (#2859)

* Parameterized queries were throwing UnsupportedOperationExceptions.
* This was because of trying to pre-compute values for join optimization.
* When the query was parameterized it is not possible to compute these values and hence these exceptions are thrown
* The fix was to prevent this computation when it is not needed or not possible.

GEODE-5884: Adding to function exception list if cause is FunctionInvocationTargetException (#2809)

GEODE-5884: Added new command and restored pre 1.8 region function behavior (#2829)

* Due to the refactor that caused us to lose wrapping of exceptions,
  the fix to GEODE-5884 cannot be applied to GEODE 1.0-1.7 clients
* Adding new command class will allow clients to identify which function class
  it is expecting to execute and what behavior to expect

GEODE-5993: Eliminate race in monitorQueryThread() (#2818)

A race existed between setting low memory in the heap monitor thread,
and checking the low memory state in query monitoring thread.

The cancelation executor was shut down and no longer accepting new
tasks when this race occurred, causing a RejectedExecutionException.

This commit solves that problem by encapsulating the scheduling
behavior using the state design pattern.

Co-authored-by: Ryan McMahon <rmcmahon@pivotal.io>
Co-authored-by: Bill Burcham <bburcham@pivotal.io>

Revert "GEODE-5729: when DistributedCacheOperation needs 2 messages, should let (#2458)" (#2801)

This reverts commit 49eb1c5fd13aefff0995d76ec7864c82d5730dd8.

GEODE-6048 build should work on release branches (#2847)

before this fix, ./gradlew build install will fail on release branch

with this fix, now it will succeed (signing will only be attempted if
signing properties are set)

Cherry-picked from 249fa0ed6078901ffe08991ee39193efd4ecdede

GEODE-6018 Remove reflections from LICENSE

Remove reflections from LICENSE.  It was previously a transitive
dependency of springfox, which no longer pulls it in.  WTFPL is
no longer needed.

(cherry picked from commit 379ee68ce61d95e569a042c2fecdf5a588223dea)

Update version numbers for 1.8.0 release

GEODE-5947: Make dependencies explicit in connectors and experimental-driver modules (#2738)

GEODE-5918 Geode function security should be dynamically determined by function arguments (#2727)

GEODE-5978: Expand GeodeAwaitility and add test coverage

Add getTimeout to allow the default timeout to be used for more than
just Awaitility (for example, with Mockito.timeout).

Add toTimeDuration to facilitate conversions between java.time.Duration
and org.awaitility.Duration.

Make timeout non-static for testing and write some unit tests.

GEODE-5985 Possible integer overflow

We had a number of calculations that were taking place with ints instead
of longs.  I've modified them to, in most places, just add an L to the
constant multiplier.  In other places I had to change a variable type
from int to long.

There were a few places that were also assigning a long to an int
variable or an int to a byte variable.  In these I added bit AND
operators to reduce the chances of overflow.

IndexElemArray needed a lot of work due to its "size" instvar being
a byte.

This closes #2783

GEODE-5984 Address LGTM recommendations

Addressing minor issues in javadocs and code reported by LooksGoodToMe.

One change that I made wasn't reported by that tool: I removed
two static variables from AcceptorImpl that duplicated variables in
Handshake.

This closes #2780

GEODE-5884: Restores behavior of exceptions/wrapping of exceptions (#2786)

* Previous refactor in a5daa924 was causing behavioral changes
   with function exceptions when combining with GEODE-5884

GEODE-5992 Fix bugs identified by static analysis

GEODE-5983: use equals() instead of == (#2777)

* For boxed primitive types equals() is used instead of ==

GEODE-5980: Removed unused collections. (#2774)

* Removed collections that are populated but are never used.

GEODE-5976:Prevent conversion of string to bytes (#2772)

* prevents the conversion of string to bytes everytime equals is called on the immutable strings

GEODE-5937: Made dependencies explicit in geode-protobuf modules. (#2722)

GEODE-5938: Make dependencies explicit in cq and rebalancer modules. (#2729)

GEODE-5975: Fixed container access. (#2771)

* Ensured that container access is done with the proper object type as a parameter.

GEODE-5981: No operations on null values (#2775)

* no method invocation on de-referenced objects.

GEODE-5974: Removal of dead code. (#2770)

* Due to contradictory dynamic type checks, dead codes were create.
* This commit deletes those dead code.

GEODE-5963: Change destroy no entry found log statement to debug (#2756)

Also fixed new Object[] mistake in log statement in Destroy65
that was introduced by changes for GEODE-5686.

GEODE-5971: Refactor gateway receiver/gateway sender commands to extend SingleGfshCommand base type (#2769)

* GEODE-5971 Refactor gateway receiver/gateway sender commands to extend SingleGfshCommand base type

Signed-off-by: Aditya Anchuri <aanchuri@pivotal.io>

GEODE-5970: remove field-mapping from jdbc:mapping (#2766)

GEODE-5969: remove value-contains-primary-key from jdbc-mapping (#2765)

GEODE-5686 Remove LocalizedStrings

Fixing two tests to expect exception text that has been slightly
changed.

GEODE-5968: remove gfsh alter jdbc-mapping command (#2763)

GEODE-5967: change pdx-class-name to pdx-name (#2761)

Renamed the gfsh "--pdx-class-name" to "--pdx-name".
Renamed the xsd "pdx-class" to "pdx-name".

GEODE-5951: add a create data-source gfsh command (#2758)

The new "create data-source" command is a replacement for "create jndi-binding". It has fewer options so will be easier for users to understand. Once it is done parsing its options it ends up using the same internal code as "create jndi-binding" to create the actual data-source.

Co-authored-by: @dschneider-pivotal
Co-authored-by: @jchen21
Co-authored-by: @monkeyherder

Add public key for Alexander Murmann to KEYS file

GEODE-5965 make JDK11 tests gating

GEODE-5966: mark PooledDataSourceFactory experimental (#2760)

GEODE-5338: Document 'ssl-use-default-context' property (#2762)

* GEODE-5338: Document 'ssl-use-default-context' property

GEODE-5686: Remove LocalizedStrings

Fixing mixed parameter markers (\{.*%s and %s.*\{)

GEODE-5686: Remove LocalizedStrings

Fixing problems found by lgtm.com

There were a lot of logging calls that were improperly converted from
(MessageFormatString, Argument, Exception) to a form that treated the
Exception as part of the String.format() call.

There were a number of errors in format strings or number of arguments.
MessageFormat was more forgiving about this than String.format().

There were a few dangling MessageFormat specs that needed to be
converted to String.format() specs.

This closes #2757

GEODE-5960: Add test to verify CommandMarker file for the JDBC connector (#2751)

Added a new test to the JDBC connector project to verify that the list
of commands loaded via the scanner and the command marker match the list
of commands found via these two sources. This is to avoid
errors/warnings due to commands which appear in Command Marker files but
don't have available source code.

Co-authored-by: Ben Ross <bross@pivotal.io>
Co-authored-by: Darrel Schneider <dschneider@pivotal.io>

GEODE-5803: Remove dependencies resulting from bad merge. (#2720)

Co-authored-by: Robert Houghton <rhoughton@pivotal.io>
Co-authored-by: Patrick Rhomberg <prhomberg@pivotal.io>

Revert "GEODE-5908: DiskStoreID.compare should compare mostSig, then leastSig (#2676)"

This reverts commit ab5fafbe188de4ffb30c0ddff926a42a713da914.

This fix conceptually is correct, but it somehow caused more data mismatch in
concurrency conflicts handling. Since there's no found bug related with this
fix, revert it for now.

GEODE-5881: simplify DefaultQuery.TestHook (#2755)

Consolidate three interface methods into one that takes an enum
to identify the "spot" in the query processing logic.

Co-authored-by: Bill Burcham <bburcham@pivotal.io>
Co-authored-by: Dale Emery <demery@pivotal.io>
Co-authored-by: Galen O'Sullivan <gosullivan@pivotal.io>

GEODE-5948: Updated the cookie name (#2740)

Co-Authored-By: Patrick Rhomberg <prhomberg@pivotal.io>

GEODE-5943: rework calculation for expected eviction to fix this test in jdk11 (#2754)

Co-authored-by: Patrick Rhomberg <prhomberg@pivotal.io>

GEODE-5953: set target member when transaction hosted locally (#2746)

  * to be able to detect primary bucket has moved due to rebalance.
  * add debug level logging for transaction.

GEODE-5952: Provide full cache XML entity for root region when a subregion is created with gfsh (#2747)

Signed-off-by: Jens Deppe <jdeppe@pivotal.io>

GEODE-5961: Add commons-validator jar to docs for tomcat dependencies (#2749)

GEODE-5935: move jdbc xml mapping to be in a region  (#2739)

The jdbc region-mapping xml element is now jdbc:mapping.
It no longer has a "region" attribute since it is nested in a region.
Creating a jdbc-mapping requires that the region already exists.
The jdbc:mapping element is now nested in a region element instead of the cache element.
The old connector-service element has been completely removed.
Some of the jdbc-mapping gfsh commands now require at least one server to be running.

Co-authored-by: Darrel Schneider <dschneider@pivotal.io>
Co-authored-by: Jianxia Chen <jchen21@apache.org>
Co-authored-by: Ben Ross @BenjaminPerryRoss

GEODE-5943: Refactor EvictionDUnitTest and OffHeapEvictionDUnitTest (#2734)

* GEODE-5943: Refactor EvictionDUnitTest and OffHeapEvictionDUnitTest

* break the test into dunit and integration tests
* use the rules to better see what's the setup and what the test is doing

GEODE-5884: Fix spotless errors

GEODE-5884: Fix typo and renamed test exception class

GEODE-5884: Refactored methods and added tests

GEODE-5884: Non HA Function waits for all results if exception occurs

  * Non HA functions will attempt to return as many results as possible when an exception occurs
  * This behavior will be more in-line with single-hop execution

GEODE-5568: Rewrite QueryMonitor to use ScheduledThreadPoolExecutor (#2744)

* GEODE-5568: Rewrite QueryMonitor to use ScheduledThreadPoolExecutor

Eliminate notify/wait bugs and improve hot-path performance.

Co-authored-by: Bill Burcham <bburcham@pivotal.io>
Co-authored-by: Ryan McMahon <rmcmahon@pivotal.io>

GEODE-5864: Put GLOBAL_GRADLE_ARGS in the right place. (#2753)

Authored-by: Sean Goller <sgoller@pivotal.io>

GEODE-5864: Add GLOBAL_GRADLE_ARGS to PublishArtifacts. (#2752)

Authored-by: Sean Goller <sgoller@pivotal.io>

GEODE-5945: Remove ref to JDBC connection commands (#2737)

Removed the references to the JDBC connection commands, which no longer exist,  from the gfsh CommandMarker file.

Co-authored-by: Ben Ross <bross@pivotal.io>
Co-authored-by: Jianxia Chen <jchen@pivotal.io>

deploy_meta enhancements: (#2748)

* currently deploy_meta infers the branch name with no way to override.  print this with the other default variables and allow to be overridden if necessary like the other defaults (for example if your local branch name is tracking a remote branch that is named differently)
* when local overrides are not set up, print a kinder message about how to set them up
* handle 'pending' status for concourse jobs

GEODE-5917: Modified instanceof to check for CachedDeserializable

GEODE-5819: update the keystore used in ssl test to make it jdk11 com… (#2743)

* GEODE-5819: update the keystore used in ssl test to make it jdk11 compatible.

Co-authored-by: Patrick Rhomberg <prhomberg@pivotal.io>

GEODE-3 Update LICENSE to include jaxb reference

Merge pull request #2742 from onichols-pivotal/GEODE-5950-automate-jdk11-updates

GEODE-5950 ensure images pick up new Java11 versions

GEODE-5950 ensure images pick up new Java11 versions

GEODE-5797: Move assertion to async invoke (#2635)

GEODE-5797: throw CacheClosed in ColocationHelper

The ColocationHelper runs in the background and wrote a suspect string
to the logs during this test when the cache was closed at the wrong
time, causing the test to fail.

The ColocationHelper would check a static variable to get colocated
regions, and if the cache was closed at the wrong point in this process,
the static region would have already been reset by closing the cache,
and the ColocationHelper would write a suspect string.

To fix this test, and the behavior of the ColocationHelper, we first
check if the cache is in the process of being cancelled. If it is, a
CacheClosedException is thrown instead of an IllegalStateException.
CacheClosedExceptions are ignored.

* add unit test for changes to ColocationHelper. Tests that a
CacheClosedException is thrown from getColocatedRegion when the cache is
closed.
* remove wait for regions to be ready as it was never the correct fix
for this bug.
* create the senders before the regions, since that is the correct order

Signed-off-by: Dan Smith <dsmith@pivotal.io>

GEODE-1603: wait for receiver MBean federation (#2726)

The check of receiver MBeans was failing with an exception of invalid
member name or id intermittently, when the check occured before the
member was added to DistributedSystemBridge's list of members. This adds
an awaitility around the method that calls this check, since the member
will eventually be put to that list.

GEODE-5925 Server shutdown delays election of new primary bucket owners

Undoing removal of null check when sending profile updates.  The method
that sends profile update messages may return null instead of a reply
processor object so this check is still needed.

GEODE-5941: Remove comments containing developer name (#2731)

This removes all mention of KIRK. There are still other developer
names present in comments.

GEODE-5946 spelling error in PartitionedRegion.java

Reworded to "Done waiting for index removal"

Revert "GEM-2250 Misspelling in info level log message "Done wating for remove index...""

This reverts commit 93e27d1ded29f3d20a7a42d9c4b04ae30fbba8a4.

GEM-2250 Misspelling in info level log message "Done wating for remove index..."

Reworded to "Done waiting for index removal"

GEODE-5925 Server shutdown delays election of new primary bucket owners

Addressing nitpick with variable names in old test code

GEODE-5819: Reduce 16h40m wait on shutdown to 120 seconds. (#2736)

* While this is unlikely to correct the Java11 SSL handshake failures, it should make tests currently hanging properly fail out so that a full failure-set can be analyzed.

GEODE-5925 Server shutdown delays election of new primary bucket owners

This PR addresses the problem of AcceptorImpl sending out expensive
profile update messages when it's being stopped during Cache.close().
There is no reason to send these messages if the affected regions are
going to be destroyed since that also sends profile updates
to other members of the cluster.

This closes #2717

Restore update passing ref (#2733)

* add x permissions

Co-authored-by: Sean Goller <sgoller@pivotal.io>
Co-authored-by: Dick Cavender <dcavender@pivotal.io>

GEODE-5925 locks are not released in shutdown hook

If the cache is closed because the JVM is exiting we aren't releasing
locks until the DistributedSystem disconnects.  This alters the behavior
of shutdown and, in particular, causes long delays in designating new
primary bucket owners and reestablishing redundancy.

This commit enables unlocking during shutdown.

This closes #2725

Restore update passing ref (#2732)

* Restore UpdatePassingRef for now.

Co-authored-by: Sean Goller <sgoller@pivotal.io>
Co-authored-by: Dick Cavender <dcavender@pivotal.io>
* Restore update-passing-ref.sh

Co-authored-by: Sean Goller <sgoller@pivotal.io>
Co-authored-by: Dick Cavender <dcavender@pivotal.io>

GEODE-5927: enhance create jndi-binding to configure pool (#2711)

The create jndi-binding --datasource-config-properties will now check each
name for a "pool." prefix. If found then that property, minus the prefix,
will be added to the properties that the PooledDataSourceFactory gets to
configure the pool. The other datasource config properties are still used
to configure the database.
The help strings on create jndi-binding have been updated.

Co-authored-by: @BenjaminPerryRoss

Restore UpdatePassingRef for now. (#2728)

Co-authored-by: Sean Goller <sgoller@pivotal.io>
Co-authored-by: Dick Cavender <dcavender@pivotal.io>

GEODE-5922 SerialGatewaySenderQueue concurrency is poorly implemented

Replaced simple synchronization with a fair write-lock.  The use of
sync to enforce serial queueing wasn't fair and caused inconsistent
latency.  I observed one thread that was blocked trying to get the
sync for over 30 seconds while other subsequent threads were able to
get the sync and queue their events.

This closes #2705

GEODE-5936 Document OpenJDK support for Geode (#2719)

Fix typo. (#2724)

Co-authored-by: Sean Goller <sgoller@pivotal.io>
Co-authored-by: Dick Cavender <dcavender@pivotal.io>

Echo out pre and post hashing name for instances. (#2721)

Co-authored-by: Sean Goller <sgoller@pivotal.io>
Co-authored-by: Dick Cavender <dcavender@pivotal.io>

Merge pull request #2718 from onichols-pivotal/GEODE-5930-revert-concourse-metadata

Revert "GEODE-5930 clean up dangling outputs in concourse (#2713)"

Revert "GEODE-5930 clean up dangling outputs in concourse (#2713)"

This reverts commit 5503d5cd1e48f9810c85c2a0a05e730157d13211.

GEODE-5764: Use a thread-safe set to record keys (#2709)

AbstractMovingAsyncEventListener.processEvents() recorded keys using
unsynchronized modification of a HashSet, though some tests could call
the method concurrently on as many as 3 threads.

AbstractMovingAsyncEventListener now uses a synchronized set to record
keys.

GEODE-5923: Add test for local alerts and cleanup classes (#2710)

* Create DistributedSystemMXBeanIntegrationTest
* Add ignored test that confirms bug GEODE-5923
* Cleanup alert related classes and add lots of comments about how they work

GEODE-5827: Use standard pattern converters in suspect appender (#2704)

The suspect appender is added programmatically without a log4j2.xml
so there is apparently no way to specify additional plugin packages.

Change hexTid back to tid to prevent error messages about not finding
the HexThreadIdPatternConverter.

Merge pull request #2685 from onichols-pivotal/GEODE-5913

GEODE-5913 resolve jstack using JAVA_HOME, if set

Merge pull request #2716 from onichols-pivotal/GEODE-5934-fix-examples-pipeline

GEODE-5934 change start/stop instance to create/delete instance

GEODE-5783: have a single source for ThreadFactory

Use the keepalive parameter when initializing the thread factory.

I also made some logging changes while debugging the problems this
missing setting was causing.

GEODE-5819: fix ssl test failures in jdk11 (#2702)

* do not call getSession after startHandshake on a socket to make it TLS1.3 compliant
* when using TLS1.3 protocol in the tests, make sure to user a certificate that are TLS1.3 compliant.

GEODE-5934 change start/stop instance to create/delete instance

GEODE-5929 concourse workers no longer need java (#2712)

GEODE-5930 clean up dangling outputs in concourse (#2713)

GEODE-5933 make it easier to use deploy_meta with forks (#2715)

GEODE-5909: Remove unknown tag causing compile warning. (#2678)

* Also cleanup various pieces of dead code or now-inapplicable comments.

GEODE-5899: Make dependencies explicit in old-client and lucene* modules. (#2675)

Co-authored-by: Robert Houghton <rhoughton@pivotal.io>
Co-authored-by: Patrick Rhomberg <prhomberg@pivotal.io>

GEODE-5897: change create jndi-binding to fail on an exception (#2680)

Create jndi-binding will now fail if an exception occurs during the create.
Also the data-source is now validated by asking it for a connection when the jndi-binding is created.
If a connection can not be obtained, the create fails and the data-source is closed.
Previously if an exception occurred when creating a jndi-binding an info message was logged.
Now, as the docs state, a warning is logged.

The problem was caused by an exception being caught, logged, and not rethrown.
Now the gfsh function will get the thrown exception and pass a failure back
to gfsh. For backwards compatibility, the XML parser still logs a warning if
the jndi-binding can not be created, instead of having it cause the cache not
to start.