Hello, I just push the changes to remove the @author tags and updated the RegionProviderTest.
On Fri, Feb 24, 2017 at 7:36 PM, Jason Huynh <jhu...@pivotal.io> wrote: > It looks like travis ci failed on that pr? Also there are some @author > tags that should probably be scrubbed out > > On Fri, Feb 24, 2017 at 4:33 PM Michael Stolz <mst...@pivotal.io> wrote: > > > +1 experimental means changing. Go for it. > > > > -- > > Mike Stolz > > Principal Engineer - Gemfire Product Manager > > Mobile: 631-835-4771 <(631)%20835-4771> > > > > On Feb 24, 2017 7:30 PM, "Kirk Lund" <kl...@apache.org> wrote: > > > > > +1 for merging in these changes even though they break rolling upgrade > > for > > > redis storage format -- it should be ok to break API or data format if > it > > > was "experimental" in all releases so far > > > > > > On Fri, Feb 24, 2017 at 3:25 PM, Bruce Schuchardt < > > bschucha...@pivotal.io> > > > wrote: > > > > > > > Gregory Green has posted a pull request that warrants discussion. It > > > > improves performance for Sets and Hashes by altering the storage > format > > > for > > > > these collections. As such it will not permit a rolling upgrade, > > though > > > > the Redis adapter is labelled "experimental" so maybe that's okay. > > > > > > > > https://github.com/apache/geode/pull/404 > > > > > > > > The PR also fixes GEODE-2469, inability to handle hash keys having > > > colons. > > > > > > > > There was some discussion about altering the storage format that was > > > > initiated by Hitesh. Personally I think Gregory's changes are better > > > than > > > > the current implementation and we should accept them, though I > haven't > > > gone > > > > through the code changes extensively. > > > > > > > > > > > > > > -- *Gregory Green* (Senior Data Engineer) ggr...@pivotal.io 201.736.1016