Don't expose "InternalCache" on RegionFactory. You probably want it to be "Cache".
On Wed, Dec 11, 2019 at 3:35 PM Mark Hanson <mhan...@pivotal.io> wrote: > > In Cache.java > > + <K, V> RegionFactory<K, V> createRegionFactory(RegionFactory<K, V> > regionFactory); > > In RegionFactory.java > + public RegionFactory(InternalCache cache, RegionFactory<K, V> > regionFactory) > and > + public RegionFactory(RegionFactory<K, V> regionFactory) > > Lastly in GemfireCacheImpl.java > + public <K, V> RegionFactory<K, V> createRegionFactory(RegionFactory<K, > V> regionFactory) > > Thanks, > Mark > > > > On Dec 11, 2019, at 3:25 PM, Dan Smith <dsm...@pivotal.io> wrote: > > > > I see a lot of PR comments on those PRs. What is the new API you added? > > > > -Dan > > > > On Wed, Dec 11, 2019 at 2:57 PM Mark Hanson <mhan...@pivotal.io> wrote: > > > >> Hi All, > >> > >> There was a suggestion that since I am making a couple user visible API > >> changes that I might want to ask the dev list. > >> > >> Basically I was migrating code from AttributesFactory to RegionFactory > and > >> hit a few snags along the way. > >> > >> Please see https://github.com/apache/geode/pull/4409 < > >> https://github.com/apache/geode/pull/4409> specifically Cache.java, > >> RegionFactory.java, and for extra credit GemfireCacheImpl.java > >> > >> I have commented at the relevant changes. > >> > >> Thanks, > >> Mark > >