[Dev] [Cynara] Async admin API proposal

José Bollo jose.bollo at open.eurogiciel.org
Mon Aug 25 10:54:23 GMT 2014


On lun, 2014-08-25 at 11:32 +0200, Jacek Bukarewicz wrote:
> Hi,
> 
> On 08/25/2014 08:32 AM, Aleksander Zdyb wrote:
> >
> > As Zofia listed most of advantages of v2 solution in her previous posts,
>  From the API user's point of view I see only one advantage of the 
> second version:
> callback is always invoked from the cynara_async_process function. This 
> might matter for some clients, but I doubt it's common case. 
> Furthermore, callback function in the first version is aware which 
> context it's called from so one can always defer its processing to a 
> suitable point (via eventfd for instance).
> 
> Shortcomings of the first version that have been listed also apply to 
> the second:
>   1) cynara_async_check can always fail (out of memory for example) so 
> user will have to handle error situation anyway. Success case will 
> usually be handled in the callback for both flavors.

I agree with Jacek that out of memory (even if rare) is an error case
that can't be ignored.

Best regards
josé



More information about the Dev mailing list