Re: [PROPOSAL] Backport GEODE-8432 to 1.13

2020-08-21 Thread Dave Barnes
Thanks for your contribution, Gester On Thu, Aug 20, 2020 at 12:37 PM Joris Melchior wrote: > +1 > > On 2020-08-20, 12:08 PM, "Xiaojian Zhou" wrote: > > It's using region path instead of getting the region. It should be no > risk. > > On 8/19/20, 10:25 AM, "Xiaojian Zhou" wrote: > >

Re: [PROPOSAL] Backport GEODE-8432 to 1.13

2020-08-20 Thread Joris Melchior
+1 On 2020-08-20, 12:08 PM, "Xiaojian Zhou" wrote: It's using region path instead of getting the region. It should be no risk. On 8/19/20, 10:25 AM, "Xiaojian Zhou" wrote: This problem also exists in 1.13.

Re: [PROPOSAL] Backport GEODE-8432 to 1.13

2020-08-20 Thread Owen Nichols
Looks like it passes all test in the pipeline and overnight. Sounds like a good change, +1 On 8/20/20, 9:09 AM, "Xiaojian Zhou" wrote: It's using region path instead of getting the region. It should be no risk. On 8/19/20, 10:25 AM, "Xiaojian Zhou" wrote: This problem also

Re: [PROPOSAL] Backport GEODE-8432 to 1.13

2020-08-20 Thread Xiaojian Zhou
It's using region path instead of getting the region. It should be no risk. On 8/19/20, 10:25 AM, "Xiaojian Zhou" wrote: This problem also exists in 1.13.

Re: [PROPOSAL] Backport GEODE-8432 to 1.13

2020-08-19 Thread Owen Nichols
Sent: Wednesday, August 19, 2020 2:09 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Backport GEODE-8432 to 1.13 +1 On 8/19/20, 10:25 AM, "Xiaojian Zhou" wrote: This problem also exists in 1.13.

Re: [PROPOSAL] Backport GEODE-8432 to 1.13

2020-08-19 Thread Eric Shu
+1 From: Jinmei Liao Sent: Wednesday, August 19, 2020 2:09 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Backport GEODE-8432 to 1.13 +1 On 8/19/20, 10:25 AM, "Xiaojian Zhou" wrote: This problem also exists in 1.13.

Re: [PROPOSAL] Backport GEODE-8432 to 1.13

2020-08-19 Thread Jinmei Liao
+1 On 8/19/20, 10:25 AM, "Xiaojian Zhou" wrote: This problem also exists in 1.13.

[PROPOSAL] Backport GEODE-8432 to 1.13

2020-08-19 Thread Xiaojian Zhou
This problem also exists in 1.13.