Darrel Schneider created GEODE-9500:
---------------------------------------

             Summary: RedisData keeps DeltaInfo references too long
                 Key: GEODE-9500
                 URL: https://issues.apache.org/jira/browse/GEODE-9500
             Project: Geode
          Issue Type: Bug
          Components: redis
            Reporter: Darrel Schneider


The current implementation of Delta for RedisData does not clear the reference 
to the DeltaInfo until toDelta is called. This works fine in the normal case 
but in the abnormal case it can case extra memory to be used by the DeltaInfo 
instances being kept alive.
What can happen is that if the extra server goes down so that geode is running 
with one server then some optimization are done by geode and it never calls 
toDelta. So in that case the DeltaInfo instances will be kept stored on the 
primary.
We actually have sizing tests that found this problem but those tests were 
changes to clear the DeltaInfo reference in the test.
Instead we should have these tests not clear it and change geode to clear it in 
AbstractRedisData.storeChanges after the put returns.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to