+1 drop it if we don't want people to use it. On Tue, Oct 11, 2016 at 8:15 PM, Kirk Lund <kl...@apache.org> wrote:
> +1 drop it as a bug with no need for replacement (or in favor of using a > function) > > On Tuesday, October 11, 2016, William Markito <wmark...@pivotal.io> wrote: > > > +1 - We always discouraged people using dynamic regions > > > > On Tue, Oct 11, 2016 at 4:33 PM, Dan Smith <dsm...@pivotal.io > > <javascript:;>> wrote: > > > > > I'd like to reopen the discussion about deprecating > > > DynamicRegionFactory. This is some old crufty code I don't think we > > > want anyone to use. I see we had a ticket to deprecated it > > > (GEODE-388), but we chose not to do it because we don't have an > > > alternative API to create regions from the client (GEODE-215). > > > > > > The thing is, there is a pretty easy workaround to use functions to > > > create regions from the client. Personally I'd recommend that over > > > DynamicRegionFactory. Can we just deprecate DynamicRegionFactory now > > > and implement GEODE-215 later? > > > > > > -Dan > > > > > > > > > > > -- > > > > ~/William > > > -- Cheers Jinmei
