[ccache] Using ccache with memcached

[Pierre Tardy]

Hi,

I like it very much. I think it adds great value for ccache, and to my old
memcached-only attempt.

I did not realize the use for moxi also as a connection "keep-alive"
mechanism, and a way to hide the syn-ack latency overhead. I think this is
what you mean by "avoid some of the network overhead." .This perhaps would
deserve a little bit more details in the doc.

Even if it is not ready, I think it would be worth to create a pull
request, and make it easier for everybody to review the current code.

This is what I used to do it, but its not easy to put and track review
comments.
https://github.com/jrosdahl/ccache/compare/master...itensionanders:memcached-only

Mike already put a bunch of coding style review comments on my own commits.
I would rather not fix them myself, as I know you already have an evolved
version which is more suitable for merging.

Cheers
Pierre

Le ven. 4 déc. 2015 à 22:45, Anders Björklund <anders at itension.se> a écrit :

> > The option to switch the to_cache/from_cache can be made available
> > separately, like it was in your PR. But it can use another config ?
> > Probably needs some updating and refactoring, and it would be nice
> > to try and keep the code duplication between them to a minimum...
> >
> > i.e. between the current filesystem code and the memcached code
> >
> > I can make an attempt to merge them, or if you want to do it...
> > To add a config like "memcached_only", next to "memcached_conf" ?
> > If you have a single server, then *neither* option makes any sense.
> > So it all depends on the setup, and needs to benchmarked further...
>
> Here is such an attempt, to keep *both* features available:
> https://github.com/itensionanders/ccache/tree/memcached-only
>
> If you set memcached_only to true, it will not use the fs cache.
> Currently it will store the manifests locally, as in the original.
>
> Also added a basic unit test to it.
>
> Needs some cleanup, but works OK ?
>
> /Anders
> _______________________________________________
> ccache mailing list
> ccache at lists.samba.org
> https://lists.samba.org/mailman/listinfo/ccache
>