[ccache] Using ccache with memcached
anders at itension.se
Sun Dec 6 20:14:13 UTC 2015
Pierre Tardy wrote:
>> Here is such an attempt, to keep *both* features available:
> I like it very much. I think it adds great value for ccache, and to my
> old memcached-only attempt.
Yeah, if it doesn't bloat the code base too much it makes sense to
leave the final decision up to the user with runtime configuration.
> 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.
Right, that is what I meant. Think it depends a bit on the number
of servers involved, but I don't think it can hurt much either way.
Suppose another paragraph or two couldn't hurt, but the more advanced
config can remain with memcached and moxi documentation - I think ?
However, it does make a lot of sense to offer the "complete package"
and is something that we are looking into. Software and configuration.
For us that would entail ccache*, moxi, distcc, memcached and distccd.
So it spans at least three or four different open source projects.
* including zlib and libmemcached
> 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
Yes, that works for testing. You can append a .diff or a .patch to it,
and use "diff" or "git am". But that's more read-only, and not social.
I wanted to do some more squashing and rebasing to "master" - but I
suppose there is no reason why all that couldn't be done as a PR...
> 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.
I think some of these might also have been fixed by "uncrustify" ?
More information about the ccache