Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-12-09 Thread Donal Evans
https://github.com/apache/geode/pull/5832 From: Dan Smith Sent: Wednesday, December 9, 2020 11:22 AM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false I will go ahead and withdraw my objection to this change.

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-12-09 Thread John Blum
e of false) where the conserve-sockets setting is concerned. Let's just make sure our old and new users are aware, too. -j From: Dan Smith Sent: Wednesday, December 9, 2020 11:22 AM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default va

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-12-09 Thread Dan Smith
I will go ahead and withdraw my objection to this change. Based on some side conversations, at least at VMWare it sounds like we don't have customers that are not setting this flag. So the scenario I'm worried about where a customer upgrades their production cluster and has it crash due to this

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-12-04 Thread Anilkumar Gingade
__ From: Anthony Baker Sent: Monday, November 23, 2020 2:16 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false Udo, you’re correct that individual servers can set the pro

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-12-03 Thread Barrett Oglesby
06932400296; message=ReplyMessage processorId=42 from 192.168.1.8(server1:63224):41001; recipients=[null] From: Anthony Baker Sent: Monday, November 23, 2020 2:16 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-12-03 Thread Bruce Schuchardt
server1:63224):41001; recipients=[null] From: Anthony Baker Sent: Monday, November 23, 2020 2:16 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false Udo, you

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-12-03 Thread Anilkumar Gingade
From: Anthony Baker Sent: Monday, November 23, 2020 2:16 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false Udo, you’re correct that individual servers can set the property independently. I was assuming this is more like

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-12-02 Thread Xiaojian Zhou
OK, I double checked, my memory is wrong. It was true as early as 6.0. From: Xiaojian Zhou Date: Wednesday, December 2, 2020 at 3:29 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false +1 I think it’s good to change back the default to be

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-12-02 Thread Xiaojian Zhou
+1 I think it’s good to change back the default to be false. It was false before. From: Barrett Oglesby Date: Wednesday, December 2, 2020 at 3:14 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false I ran a bunch of tests using the long

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-12-02 Thread Barrett Oglesby
: TestDistributionMessageObserver operation=afterProcessMessage; time=1606932400296; message=ReplyMessage processorId=42 from 192.168.1.8(server1:63224):41001; recipients=[null] From: Anthony Baker Sent: Monday, November 23, 2020 2:16 PM To: dev@geode.apache.org S

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-23 Thread Anthony Baker
Udo, you’re correct that individual servers can set the property independently. I was assuming this is more like the ’security-manager` property and others that require all cluster members to be in agreement. I’m not sure I understand the use case to allow this setting to be per-member. That ma

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-23 Thread Xiaojian Zhou
showed any problems. From: Anthony Baker Sent: Friday, November 20, 2020 8:52 AM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false Question: how would this work with a rolling upgrade? If the user did no

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-20 Thread Donal Evans
have showed any problems. From: Anthony Baker Sent: Friday, November 20, 2020 8:52 AM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false Question: how would this work with a rolling upgrade? If the user did no

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-20 Thread Udo Kohlmeyer
our “shared nothing value” here.. From: Xiaojian Zhou Date: Saturday, November 21, 2020 at 5:34 AM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false 1) Conserve-socket will only impact p2p connection. If set to false, that mean the p2p

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-20 Thread Xiaojian Zhou
From: Anthony Baker > Sent: Thursday, November 19, 2020 5:57:33 PM > To: dev@geode.apache.org > Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false > > I think there are many good reasons to flip the default value for this property. I

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-20 Thread Anthony Baker
LCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=V65jEFWBflK8CWzFgxuFwQBD%2BV2BDlOlPa%2FtLR2N3eY%3D&reserved=0> > > > From: Anthony Baker > Sent: Thursday, November 19, 2020 5:57:33 PM > To: dev@geode.apache.org > Subject: Re: [PROPOSAL] Chan

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-20 Thread Donal Evans
Sent: Thursday, November 19, 2020 5:57:33 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false I think there are many good reasons to flip the default value for this property. I do question whether requiring a user to allocate new hardware to s

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-19 Thread Anthony Baker
I think there are many good reasons to flip the default value for this property. I do question whether requiring a user to allocate new hardware to support the changed resource requirements is appropriate for a minor version bump. In most cases I think that would come as an unwelcome surprise du

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-19 Thread Dan Smith
Personally, this has caused enough grief in the past (both ways, actually!) that I 'd say this is a major version change. I agree with John. Either value of conserve-sockets can crash or hang your system depending on your use case. If this was just a matter of slowing down or speeding up perform

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-19 Thread Donal Evans
gt; actually!) that I 'd say this is a major version change. >> >> -j >> >> >> >> From: Nabarun Nag >> Sent: Wednesday, November 18, 2020 6:09 PM >> To: dev@geode.apache.org >> Subject: Re: [PROPOSAL] Ch

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-19 Thread Jacob Barrett
or version change. >> >> -j >> >> >> ____ >> From: Nabarun Nag >> Sent: Wednesday, November 18, 2020 6:09 PM >> To: dev@geode.apache.org >> Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to >

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-19 Thread Ju@N
> > Personally, this has caused enough grief in the past (both ways, > actually!) that I 'd say this is a major version change. > > -j > > > > From: Nabarun Nag > Sent: Wednesday, November 18, 2020 6:09 PM > To: dev@geode.apache.

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-18 Thread John Blum
grief in the past (both ways, actually!) that I 'd say this is a major version change. -j From: Nabarun Nag Sent: Wednesday, November 18, 2020 6:09 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-18 Thread Nabarun Nag
s, Nabarun From: Udo Kohlmeyer Sent: Wednesday, November 18, 2020 6:00 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Change the default value of conserve-sockets to false Hi there Donal, Thank you for raising this. It is not an uncommon request to change the default value of this field.

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-18 Thread Owen Nichols
value of conserve-sockets to false Hi Geode dev, First, from the docs[1], a brief explanation of the purpose of the conserve-sockets property: "The conserve-sockets setting indicates whether application threads share sockets with other threads or use their own sockets for member communic

Re: [PROPOSAL] Change the default value of conserve-sockets to false

2020-11-18 Thread Udo Kohlmeyer
Evans Date: Thursday, November 19, 2020 at 12:04 PM To: dev@geode.apache.org Subject: [PROPOSAL] Change the default value of conserve-sockets to false Hi Geode dev, First, from the docs[1], a brief explanation of the purpose of the conserve-sockets property: "The conserve-sockets se

[PROPOSAL] Change the default value of conserve-sockets to false

2020-11-18 Thread Donal Evans
Hi Geode dev, First, from the docs[1], a brief explanation of the purpose of the conserve-sockets property: "The conserve-sockets setting indicates whether application threads share sockets with other threads or use their own sockets for member communication. This setting has no effect on comm