Re: [Proposal] Backport GEODE-8422 to support/1.13

2020-08-13 Thread Mark Hanson
I just cherry-picked the change in. Thanks! On 8/13/20, 4:55 PM, "Dick Cavender" wrote: +1 -Original Message- From: Mark Hanson Sent: Thursday, August 13, 2020 4:46 PM To: dev@geode.apache.org Subject: Re: [Proposal] Backport GEODE-8422 to support/1.13 Hi

RE: [Proposal] Backport GEODE-8422 to support/1.13

2020-08-13 Thread Dick Cavender
+1 -Original Message- From: Mark Hanson 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"

Re: [Proposal] Backport GEODE-8422 to support/1.13

2020-08-13 Thread Mark Hanson
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" 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" wrote: +1 On Wed, Aug 1

Re: [Proposal] Backport GEODE-8422 to support/1.13

2020-08-13 Thread Owen Nichols
+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" wrote: +1 On Wed, Aug 12, 2020 at 4:41 PM Mark Hanson wrote: > Hi All, > > We have found a case where tombstones were being cleared when the region > wa

Re: TimeIntegrationTest is flaky

2020-08-13 Thread Kirk Lund
Thanks guys! On Thu, Aug 13, 2020 at 10:47 AM Raymond Ingles wrote: > Yes, we're on it. Thanks! > > On 8/12/20, 7:25 PM, "Kirk Lund" wrote: > > Since this is a new test, can we please prioritize fixing it? > > > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fissues.apach

Re: TimeIntegrationTest is flaky

2020-08-13 Thread Raymond Ingles
Yes, we're on it. Thanks! On 8/12/20, 7:25 PM, "Kirk Lund" wrote: Since this is a new test, can we please prioritize fixing it? https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fissues.apache.org%2Fjira%2Fbrowse%2FGEODE-8379&data=02%7C01%7Cringles%40vmware.com%7C9c67a4

Re: TimeIntegrationTest is flaky

2020-08-13 Thread Darrel Schneider
Will do. Looks like an easy fix From: Kirk Lund Sent: Wednesday, August 12, 2020 4:25 PM To: dev@geode.apache.org Subject: TimeIntegrationTest is flaky Since this is a new test, can we please prioritize fixing it? https://nam04.safelinks.protection.outlook.com/?

Re: [Proposal] Backport GEODE-8422 to support/1.13

2020-08-13 Thread Mark Hanson
Actually what I should say is that this does not entirely fix GEODE-8422 it reduces the frequency to 1/3 of what it was and fixes one issue of GEODE-8422. It is my understanding we didn't want to hold this release for the other part of this issue. I will file that under a separate bug. Thanks,

Re: [Proposal] Backport GEODE-8422 to support/1.13

2020-08-13 Thread Mark Hanson
Yes. On 8/13/20, 7:27 AM, "Dave Barnes" wrote: Do I understand correctly that this fix, if validated, addresses the last remaining release-blocker for 1.13? On Wed, Aug 12, 2020 at 6:02 PM Mark Hanson wrote: > Not yet, we will need to wait for that to finish cleanly relative

Re: [Proposal] Backport GEODE-8422 to support/1.13

2020-08-13 Thread Dave Barnes
Do I understand correctly that this fix, if validated, addresses the last remaining release-blocker for 1.13? On Wed, Aug 12, 2020 at 6:02 PM Mark Hanson wrote: > Not yet, we will need to wait for that to finish cleanly relative to this > change. I guess I was just seeking to gain approval for p