You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "jdeppe-pivotal (GitHub)" <gi...@apache.org> on 2019/02/05 22:38:30 UTC

[GitHub] [geode] jdeppe-pivotal commented on pull request #3156: GEODE-6322: refactor createRegionCommand to use RegionConfig only

I think this would be better as a method on `RegionConfig` - maybe `mergeEntryIdleTime`. I realize that it would actually result in 4 methods: `merge[Entry|Region][IdleTime|TimeToLive]`. That way it would also be more 'discoverable' by someone scanning the API for `RegionConfig`.

[ Full content available at: https://github.com/apache/geode/pull/3156 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org