Re: [PROPOSAL]: Include GEODE-7728 in Release 1.12.0

2020-02-07 Thread Ernest Burghardt
There appears to be consensus that this is a critical fix. The following commit has been brought into release/1.12 as the critical fix for GEODE-7728: git cherry-pick -x 6e35c201ea605075433203d4e64ca887bafd8fcb GEODE-7728 has been marked as 'resolved in' 1.12. Regards EB On Wed, Feb 5, 2020 a

Re: [PROPOSAL]: Include GEODE-7728 in Release 1.12.0

2020-02-05 Thread Ernest Burghardt
+1 I'll plan to cherry-pick in the morning if there are no objections... On Wed, Feb 5, 2020 at 11:20 AM Ivan Godwin wrote: > +1 > > On Wed, Feb 5, 2020 at 10:16 AM Nabarun Nag wrote: > > > ++ > > > > On Wed, Feb 5, 2020 at 9:54 AM Alexander Murmann > > wrote: > > > > > +1 > > > > > > On Wed

Re: [PROPOSAL]: Include GEODE-7728 in Release 1.12.0

2020-02-05 Thread Ivan Godwin
+1 On Wed, Feb 5, 2020 at 10:16 AM Nabarun Nag wrote: > ++ > > On Wed, Feb 5, 2020 at 9:54 AM Alexander Murmann > wrote: > > > +1 > > > > On Wed, Feb 5, 2020 at 9:29 AM Udo Kohlmeyer wrote: > > > > > +1 - for inclusion > > > > > > On 2/5/20 4:22 AM, Ju@N wrote: > > > > Hello devs, > > > > > >

Re: [PROPOSAL]: Include GEODE-7728 in Release 1.12.0

2020-02-05 Thread Nabarun Nag
++ On Wed, Feb 5, 2020 at 9:54 AM Alexander Murmann wrote: > +1 > > On Wed, Feb 5, 2020 at 9:29 AM Udo Kohlmeyer wrote: > > > +1 - for inclusion > > > > On 2/5/20 4:22 AM, Ju@N wrote: > > > Hello devs, > > > > > > I'd like to include the fix for GEODE-7728 [1] in release 1.12.0. > > > The chang

Re: [PROPOSAL]: Include GEODE-7728 in Release 1.12.0

2020-02-05 Thread Alexander Murmann
+1 On Wed, Feb 5, 2020 at 9:29 AM Udo Kohlmeyer wrote: > +1 - for inclusion > > On 2/5/20 4:22 AM, Ju@N wrote: > > Hello devs, > > > > I'd like to include the fix for GEODE-7728 [1] in release 1.12.0. > > The change is basically to avoid throwing an exception and return the > > correct result wh

Re: [PROPOSAL]: Include GEODE-7728 in Release 1.12.0

2020-02-05 Thread Udo Kohlmeyer
+1 - for inclusion On 2/5/20 4:22 AM, Ju@N wrote: Hello devs, I'd like to include the fix for GEODE-7728 [1] in release 1.12.0. The change is basically to avoid throwing an exception and return the correct result whenever a query has more than one condition and one of them compares two indexed

[PROPOSAL]: Include GEODE-7728 in Release 1.12.0

2020-02-05 Thread Ju@N
Hello devs, I'd like to include the fix for GEODE-7728 [1] in release 1.12.0. The change is basically to avoid throwing an exception and return the correct result whenever a query has more than one condition and one of them compares two indexed fields. This is not a new issue but it can be hit by