-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53745/
-----------------------------------------------------------
(Updated Nov. 15, 2016, 12:25 a.m.)
Review request for geode, anilkumar gingade and Dan Smith.
Bugs: geode-2107
https://issues.apache.org/jira/browse/geode-2107
Repository: geode
Description
-------
This is a thread leak we found in GEM-933.
Diffs
-----
geode-core/src/main/java/org/apache/geode/internal/cache/wan/AbstractGatewaySenderEventProcessor.java
e953c0c
geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderAdvisor.java
ff36e06
Diff: https://reviews.apache.org/r/53745/diff/
Testing (updated)
-------
Tests run so far. One of these tests is failing with a compatibility failure
I'm investigating.
include $JTESTS/hct/backwardCompatibility/hctBackwardCompatibility.bt
include $JTESTS/newWan/newWanBackwardCompatibility.bt
//I'm seeing what might be a compatiblity failure with this test
cq/serialCQBridgeBackwardCompatibilityWithClientCache.conf edgeHosts=8
edgeVMsPerHost=1 edgeThreadsPerVM=2 bridgeHosts=3 bridgeVMsPerHost=1
bridgeThreadsPerVM=2
parReg/bridge/concParRegHABridgeCompat.conf edgeHosts=8 edgeVMsPerHost=1
edgeThreadsPerVM=5 bridgeHosts=5 bridgeVMsPerHost=1 bridgeThreadsPerVM=5
redundantCopies=2 numVMsToStop=2 numAccessors=1 numEmptyClients=1
numThinClients=1 version1=default version2=822 version3=822 version4=822
version5=822 version6=822 version7=822 version8=822
delta/feederIsCurrentVersionClient.conf
A = bridge bridgeHosts = 2 bridgeVMsPerHost = 1 bridgeThreadsPerVM = 1
B = feed feedHosts = 1 feedVMsPerHost = 1 feedThreadsPerVM = 1
C = edge edgeHosts = 2 edgeVMsPerHost = 1 edgeThreadsPerVM = 1
D = edgeOld edgeOldHosts = 2 edgeOldVMsPerHost = 1
edgeOldThreadsPerVM = 1
nPutThreads=10 nPutKeyRange=10 version1=default version2=822
// validation for old and current client where feeder is old client
delta/feederIsOldVersionClient.conf
A = bridge bridgeHosts = 2 bridgeVMsPerHost = 1 bridgeThreadsPerVM = 1
B = feed feedHosts = 1 feedVMsPerHost = 1 feedThreadsPerVM = 1
C = edge edgeHosts = 2 edgeVMsPerHost = 1 edgeThreadsPerVM = 1
D = edgeOld edgeOldHosts = 2 edgeOldVMsPerHost = 1
edgeOldThreadsPerVM = 1
nPutThreads=10 nPutKeyRange=10 version1=default version2=822
Thanks,
xiaojian zhou
