Avoid PowerMockito

2021-03-09 Thread Kirk Lund
I just reviewed a PR that was adding a unit test using PowerMockito. We've had lots of problems with PowerMockito leaving the unit testing JVM corrupted for later tests. Using PowerMockito also discourages us from refactoring product code to have better design and be easier to unit test. So in prev

Re: [PROPOSAL] backport GEODE-8761 to support/1.14

2021-03-09 Thread Owen Nichols
Sounds good, added to blocker board On 3/9/21, 9:16 AM, "Bruce Schuchardt" wrote: +1 This will be very nice to have On 3/9/21, 9:08 AM, "Darrel Schneider" wrote: GEODE-8761 causes geode to detect threads in a cache server that are stuck processing a client request. This can

Re: [PROPOSAL] backport GEODE-8761 to support/1.14

2021-03-09 Thread Bruce Schuchardt
+1 This will be very nice to have On 3/9/21, 9:08 AM, "Darrel Schneider" wrote: GEODE-8761 causes geode to detect threads in a cache server that are stuck processing a client request. This can be very helpful in diagnosing hangs. The changes to make this happen are rather simple and I'd l

[PROPOSAL] backport GEODE-8761 to support/1.14

2021-03-09 Thread Darrel Schneider
GEODE-8761 causes geode to detect threads in a cache server that are stuck processing a client request. This can be very helpful in diagnosing hangs. The changes to make this happen are rather simple and I'd like to backport them to 1.14.

Question regarding VersionRequest/VersionResponse messages

2021-03-09 Thread Jakov Varenina
Hi community, I have one question regarding VersionRequest/VersionResponse messages. Before member sends actual message, it has to first determine the remote member version. This is done by exchanging /VersionRequest///VersionResponse/ messages using function /getServerVersion() /from class /