tree b44490ce8ece8dbe01eae322c421a889b3fd9268
parent 4e206f09df87d26fd22a3238596ac4bc12cf4846
author Darrel Schneider <darrel@vmware.com> 1607046380 -0800
committer GitHub <noreply@github.com> 1607046380 -0800
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJfyZTsCRBK7hj4Ov3rIwAAdHIIAKPN7o0LEbK0RYT8SlWHoGW+
 nKRgZk+yn6W0L9mJzYgs3aqAd+OWtiWvROPgrWH526+2n8GxJ1s+jEjY+zc91CKi
 LH2/sHByAb9bQs9NA1EHyuJ8tjNXVKRk0XPSnUNuQQRMONRJdAh8a91Fin3yEziE
 0tqOf38OYRmraad3RUXEIbc+BH2+x05vjbpTfa7nRTEj9p2su8NNTRcgg6poIdxz
 f8JRAHUbfOU+I1mFlhfafPZbd+QbQn7vS2s3Kp5T5v4CTp08iwMivoBq4Fyujfg1
 HT7yo8VxbMFYWcINfNAhl2bclC4gWSQYr2MgSVj454zLHYGiAEK5rREUCW+plUg=
 =5ooR
 -----END PGP SIGNATURE-----
 

GEODE-8521: detect if a p2p reader thread is hung (#5763)

* AbstractExecutor can now be suspended and resumed.
This does nothing on the base class but on P2PReaderExecutorGroup
it turns monitoring on and off.
Also the monitor thread will now detect a zero startTime
and in that case set the startTime allowing threads being
monitored to not keep setting it themselves (which happens more often).

* the timeInterval and timeLimit that come from gemfire properties
will now always be the values configured on the DistributedSystem.
Previously the thread monitoring classes had their own instance of
DistributionConfigImpl which at least in some cases could be inconsistent
the the config on the DistributedSystem. Now these two config values
are passed in to the constructor.

* If an instance of ResourceManagerStats can not be found then
it will now only prevent the monitor from updating the "numThreadsStuck"
stat. Before it prevented it from doing any detection and logging.
The lazy initialization of the ResourceManagerStats is now cleaner and does
not require the integration test to explicitly call run().

Co-authored-by: Darrel Schneider <darrel@vmware.com>