-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58315/
-----------------------------------------------------------
Review request for geode, anilkumar gingade, Eric Shu, and Lynn Gallinat.
Bugs: GEODE-2485
https://issues.apache.org/jira/browse/GEODE-2485
Repository: geode
Description
-------
The CCPTimer is now purged for every 1000 cancels done. So we will now no
longer have more than 1000
cancelled tasks eating up memory.
Now uses internalSuspend in two places the previously used suspend.
Since internalSuspend does not schedule a timer task these places will have no
more issues with leaking memory
and these code paths will perform better.
renamed resume(TxStateProxy) to internalResume for clarity.
internalResume no longer checks for TimerTask to cancel since internalSuspend
does not add one.
Instead the only code that checks for TimerTask is "resume".
Diffs
-----
geode-core/src/main/java/org/apache/geode/cache/query/internal/DefaultQuery.java
a7210916ed77e7e628a9d858fdad70a8276dbbe2
geode-core/src/main/java/org/apache/geode/cache/query/internal/index/IndexManager.java
6cf9a3f1068c5c69bd755b82f120ca719a7cb7c8
geode-core/src/main/java/org/apache/geode/internal/SystemTimer.java
bddd06288161c2e6ba30b97c1c20915e0514438d
geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRegion.java
fa0257432aa149e70142e35d7432a5727ba035fd
geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java
15755175fc16609e0341d8c9e97039b5b83ce5d2
geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java
957038c8a173c7f0d1f5d3c7301673fd44add965
geode-core/src/main/java/org/apache/geode/internal/cache/PartitionedRegionDataView.java
263df0f86b7c18504704506a1162cfebff394468
geode-core/src/main/java/org/apache/geode/internal/cache/TXManagerImpl.java
14fdf991b13661e5702b48480624ad1142a129b7
geode-core/src/main/java/org/apache/geode/internal/cache/TXState.java
d577f3925c2087c204a9f43b46b26d421ed51be4
geode-core/src/main/java/org/apache/geode/internal/cache/TXStateProxy.java
c5e7cf6aef6134008f49716ff995a7e3772dd07e
geode-core/src/main/java/org/apache/geode/internal/cache/TXStateProxyImpl.java
a89cdc4a59c656cc3a2081c8aaa46d36b359c161
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/CacheClientProxy.java
d7b923c55bfca019956a2bffb613e310310ebedd
geode-core/src/main/java/org/apache/geode/pdx/internal/PeerTypeRegistration.java
653cafbdd4c8c93beafaa3dd9c97ff60750ce7ab
geode-core/src/test/java/org/apache/geode/TXExpiryJUnitTest.java
f55926ad7006375010fa37af2772d190a31a77aa
geode-core/src/test/java/org/apache/geode/TXJUnitTest.java
5166a160ad71bc21d61c0c6199722abf113d7878
geode-core/src/test/java/org/apache/geode/cache30/MultiVMRegionTestCase.java
c51c9cc6da130b13480c68d93a6294f1c7bd4e3b
geode-core/src/test/java/org/apache/geode/internal/cache/ClientServerTransactionDUnitTest.java
3771fa87fa81bd752000f4291a9472f2b898e496
geode-core/src/test/java/org/apache/geode/internal/cache/GemFireCacheImplTest.java
288a3e1b9a5a17d1e4802626e17c7d74f308f468
geode-core/src/test/java/org/apache/geode/internal/cache/RemoteTransactionDUnitTest.java
751bcd11fa7357f2d51be9d0bee732e3f9af395e
geode-core/src/test/java/org/apache/geode/internal/cache/TransactionsWithDeltaDUnitTest.java
c6aa376f868995b9335552abcded3fe62dd19bcc
geode-core/src/test/java/org/apache/geode/internal/cache/execute/MyTransactionFunction.java
9a0f3e66d2d05c889bd3e9020551fa4bad3f17f9
Diff: https://reviews.apache.org/r/58315/diff/1/
Testing
-------
precheckin
Thanks,
Darrel Schneider
