Re: [PROPOSAL]: Include GEODE-7832, GEODE-7853 & GEODE-7863 in Geode 1.12.0

2020-03-18 Thread Donal Evans
+1 On Wed, Mar 18, 2020 at 4:53 PM Owen Nichols wrote: > +3 > > > On Mar 18, 2020, at 4:52 PM, Ju@N wrote: > > > > Hello devs, > > > > I'd like to propose including the fixes for *GEODE-7832 [1]*, *GEODE-7853 > > [2]* and *GEODE-7863 [3]* in release 1.12.0. > > All the changes are related to th

Re: [DISCUSS] Client side configuration for a SNI proxy

2020-03-18 Thread Jacob Barrett
+1 > On Mar 18, 2020, at 4:13 PM, Dan Smith wrote: > > One addendum to this proposal: > > This proposed method of configuring a SNI proxy server requires that the > client sends the SNI hostname as part of the TLS handshake. In the previous > proposal we suggested it would only be sent if a SNI

Re: [PROPOSAL]: Include GEODE-7832, GEODE-7853 & GEODE-7863 in Geode 1.12.0

2020-03-18 Thread Owen Nichols
+3 > On Mar 18, 2020, at 4:52 PM, Ju@N wrote: > > Hello devs, > > I'd like to propose including the fixes for *GEODE-7832 [1]*, *GEODE-7853 > [2]* and *GEODE-7863 [3]* in release 1.12.0. > All the changes are related to the work we have been doing in order to > bring the performance closer to t

[PROPOSAL]: Include GEODE-7832, GEODE-7853 & GEODE-7863 in Geode 1.12.0

2020-03-18 Thread Ju@N
Hello devs, I'd like to propose including the fixes for *GEODE-7832 [1]*, *GEODE-7853 [2]* and *GEODE-7863 [3]* in release 1.12.0. All the changes are related to the work we have been doing in order to bring the performance closer to the baseline (*Geode 1.10*), we are not quite there yet but it w

Re: [DISCUSS] Client side configuration for a SNI proxy

2020-03-18 Thread Dan Smith
One addendum to this proposal: This proposed method of configuring a SNI proxy server requires that the client sends the SNI hostname as part of the TLS handshake. In the previous proposal we suggested it would only be sent if a SNI Proxy is configured. Now we propose that a geode client will *alw

Re: Tips on using AsyncInvocation in DUnit Tests

2020-03-18 Thread Anilkumar Gingade
Thanks Kirk. Can you add an example here... On Wed, Mar 18, 2020 at 11:12 AM Kirk Lund wrote: > Tips on using AsyncInvocation: > > * Always use await() or get() > * Both check and throw any remote exceptions > * Both use GeodeAwaitility Timeout and will throw TimeoutException if it’s > exceeded

Re: [PROPOSAL] Include fix for GEODE-7763 into release 1.12.0

2020-03-18 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-7763: git cherry-pick -x b4c3e9413f0008635b0a0c9116c96147d1e4ceec GEODE-7763 has been marked as 'resolved in' 1.12. Regards EB On Wed, Mar 18, 2020

Re: [PROPOSAL] Include fix for GEODE-7763 into release 1.12.0

2020-03-18 Thread Bruce Schuchardt
+1 On 3/18/20, 11:41 AM, "Jason Huynh" wrote: Hello Dev list, I'd like to include a fix for GEODE-7763 in release 1.12.0. The change removes the call to exportValue, preventing a serialization, when no clients are waiting for the specific event. The reason why I think i

Re: [PROPOSAL] Include fix for GEODE-7763 into release 1.12.0

2020-03-18 Thread Donal Evans
+1 On Wed, Mar 18, 2020 at 12:32 PM John Blum wrote: > +1 > > On Wed, Mar 18, 2020 at 11:52 AM Owen Nichols wrote: > > > +1 > > > > > On Mar 18, 2020, at 11:49 AM, Dick Cavender > > wrote: > > > > > > +1 > > > > > > On Wed, Mar 18, 2020 at 11:43 AM Nabarun Nag wrote: > > > > > >> +1 > > >> >

Re: [PROPOSAL] Include fix for GEODE-7763 into release 1.12.0

2020-03-18 Thread John Blum
+1 On Wed, Mar 18, 2020 at 11:52 AM Owen Nichols wrote: > +1 > > > On Mar 18, 2020, at 11:49 AM, Dick Cavender > wrote: > > > > +1 > > > > On Wed, Mar 18, 2020 at 11:43 AM Nabarun Nag wrote: > > > >> +1 > >> > >> On Wed, Mar 18, 2020 at 11:41 AM Jason Huynh wrote: > >> > >>> Hello Dev list, >

Re: [PROPOSAL] Include fix for GEODE-7763 into release 1.12.0

2020-03-18 Thread Owen Nichols
+1 > On Mar 18, 2020, at 11:49 AM, Dick Cavender wrote: > > +1 > > On Wed, Mar 18, 2020 at 11:43 AM Nabarun Nag wrote: > >> +1 >> >> On Wed, Mar 18, 2020 at 11:41 AM Jason Huynh wrote: >> >>> Hello Dev list, >>> >>> I'd like to include a fix for GEODE-7763 in release 1.12.0. >>> The chang

Re: [PROPOSAL] Include fix for GEODE-7763 into release 1.12.0

2020-03-18 Thread Patrick Johnson
+1 On 3/18/20, 11:50 AM, "Dan Smith" wrote: +1 On Wed, Mar 18, 2020 at 11:41 AM Jason Huynh wrote: > Hello Dev list, > > I'd like to include a fix for GEODE-7763 in release 1.12.0. > The change removes the call to exportValue, preventing a serialization, >

Re: [PROPOSAL] Include fix for GEODE-7763 into release 1.12.0

2020-03-18 Thread Dan Smith
+1 On Wed, Mar 18, 2020 at 11:41 AM Jason Huynh wrote: > Hello Dev list, > > I'd like to include a fix for GEODE-7763 in release 1.12.0. > The change removes the call to exportValue, preventing a serialization, > when no clients are waiting for the specific event. > The reason why I think it sho

Re: [PROPOSAL] Include fix for GEODE-7763 into release 1.12.0

2020-03-18 Thread Dick Cavender
+1 On Wed, Mar 18, 2020 at 11:43 AM Nabarun Nag wrote: > +1 > > On Wed, Mar 18, 2020 at 11:41 AM Jason Huynh wrote: > > > Hello Dev list, > > > > I'd like to include a fix for GEODE-7763 in release 1.12.0. > > The change removes the call to exportValue, preventing a serialization, > > when no c

Re: [PROPOSAL] Include fix for GEODE-7763 into release 1.12.0

2020-03-18 Thread Nabarun Nag
+1 On Wed, Mar 18, 2020 at 11:41 AM Jason Huynh wrote: > Hello Dev list, > > I'd like to include a fix for GEODE-7763 in release 1.12.0. > The change removes the call to exportValue, preventing a serialization, > when no clients are waiting for the specific event. > The reason why I think it sho

[PROPOSAL] Include fix for GEODE-7763 into release 1.12.0

2020-03-18 Thread Jason Huynh
Hello Dev list, I'd like to include a fix for GEODE-7763 in release 1.12.0. The change removes the call to exportValue, preventing a serialization, when no clients are waiting for the specific event. The reason why I think it should be in the release is that we noticed a negative effect on perform

Tips on using AsyncInvocation in DUnit Tests

2020-03-18 Thread Kirk Lund
Tips on using AsyncInvocation: * Always use await() or get() * Both check and throw any remote exceptions * Both use GeodeAwaitility Timeout and will throw TimeoutException if it’s exceeded * Use await() for Void types and get() when expecting a non-null value Recent improvements: Timeout now ge

RE: WAN replication issue in cloud native environments

2020-03-18 Thread Alberto Bustamante Reyes
Hi all, As Bruce suggested me, I have created a wiki page describing the problem we are trying to solve: https://cwiki.apache.org/confluence/display/GEODE/Allow+same+host+and+port+for+all+gateway+receivers Please let me know if further clarifications are needed. Also, I have closed the PR I ha

Re: [PROPOSAL] eliminate file count loophole in PR StressNewTest

2020-03-18 Thread Bruce Schuchardt
-1 on changing the check I'm with Owen - let's leave things alone for now.  On 3/17/20, 7:43 PM, "Owen Nichols" wrote: At least one person on the thread (@anthony) raised concerns but has not replied since. Also since this thread was started, the bug that miscounted files has