Mailing-List: contact dev-help@celix.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@celix.apache.org
Received-SPF: pass (athena.apache.org: domain of a.broekhuis@gmail.com
 designates 209.85.192.50 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <141e6a9c9585ba780f7cd25da0d75162@mail.sundevil.de>
References: <20140925045629.522A32388993@eris.apache.org>
	<CA+T-u8c9yWfLFh9dVDrdh73os_Lg4aZU=nD+G4N0RED3Tv5Q5A@mail.gmail.com>
	<ac7f3f83015153a2bb1df7e8378b4e9c@mail.sundevil.de>
	<CA+T-u8fG01mGcR4xsvQ_6tXtiWf_WG98yan=V=V65VcYJDFYhw@mail.gmail.com>
	<CA+T-u8cF8W1umNuUZFQZu6ShiLAVbLneo9EcT-DcYtnuWmQtnA@mail.gmail.com>
	<141e6a9c9585ba780f7cd25da0d75162@mail.sundevil.de>
Date: Tue, 7 Oct 2014 20:42:26 +0200
Message-ID: 
 <CA+T-u8cST4oHOph3_MbgZudgA-FL_3YOEGiSBnDaGwM+YHTfLw@mail.gmail.com>
Subject: Re: svn commit: r1627454 - in
 /celix/trunk/remote_services/discovery_etcd/private:
 include/etcd.h src/etcd.c src/etcd_watcher.c
From: Alexander Broekhuis <a.broekhuis@gmail.com>
To: dev@celix.apache.org
Content-Type: multipart/alternative; boundary=001a11c120d29596570504d98f06

--001a11c120d29596570504d98f06
Content-Type: text/plain; charset=UTF-8

2014-10-07 20:38 GMT+02:00 Bjoern Petri <bjoern.petri@sundevil.de>:

>
>  The Amdatu implementation does not yet support TTL. But is this behaviour
>> as expected? I don't think a TTL update should trigger a "set" on the
>> other
>> ends..
>>
>> What do you think?
>>
>
> Indeed, an "update" would be the nicer solution - this is unfortunately
> not yet supported by the Celix implementation - I will add it within the
> next days.
>
> We could also change the update interval from 10 seconds, if you'd like
> to. I just choose a value not that high, so I don't need to wait that long
> when shutting down the discovery_etcd.
>

I don't think the solution is in changing a value, because then after that
time, the other ends would still toggle all services because of the "set",
which should not happen.

Looking at the code, a TTL of 0 means no TTL. Is this correct? In that case
I can set the TTL to 0. If this is not the case, this update breaks interop
with Amdatu, which I think we don't want. Disabling the TTL would be a
reasonable fix I guess.


-- 
Met vriendelijke groet,

Alexander Broekhuis

--001a11c120d29596570504d98f06--