I was waiting until you confirmed that the fix had successfully completed a 
test cycle on develop. 
I see Owen gave it the green light in that regard, so all is in order.
Thanks for this — at the beginning of the week, it was our last known blocker.
-Dave

> On Aug 13, 2020, at 5:08 PM, Mark Hanson <hans...@vmware.com> wrote:
> 
> I just cherry-picked the change in. Thanks!
> 
> On 8/13/20, 4:55 PM, "Dick Cavender" <di...@vmware.com> wrote:
> 
>    +1
> 
>    -----Original Message-----
>    From: Mark Hanson <hans...@vmware.com> 
>    Sent: Thursday, August 13, 2020 4:46 PM
>    To: dev@geode.apache.org
>    Subject: Re: [Proposal] Backport GEODE-8422 to support/1.13
> 
>    Hi All, 
> 
>    I can't remember for sure, but I think we need 3 approvals?
> 
>    Thanks,
>    Mark
> 
>    On 8/13/20, 4:22 PM, "Owen Nichols" <onich...@vmware.com> wrote:
> 
>        +1 from me.  Pipeline and all other testing I've seen suggests this 
> fix is good
> 
>        On 8/12/20, 4:52 PM, "Jianxia Chen" <jche...@apache.org> wrote:
> 
>            +1
> 
>            On Wed, Aug 12, 2020 at 4:41 PM Mark Hanson <hans...@vmware.com> 
> wrote:
> 
>> Hi All,
>> 
>> We have found a case where tombstones were being cleared when the region
>> was not initialized. This was causing some test failures and potential
>> field failures. We would like to put this into support/1.13.
>> 
>> Thanks,
>> Mark
>> 
> 
> 
> 

Reply via email to