Re: [PATCH 04/19] index-helper: new daemon for caching index and related stuff
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Mar 15, 2016, 15:56 UTC
- Message-ID
- <xmqqk2l391m2.fsf@gitster.mtv.corp.google.com>
- In-Reply-To
- <CACsJy8Cgz0zTryCCXoPMh40Rq4yOCy1L8Nih-YbKaZLyV=Gi-w@mail.gmail.com>
Duy Nguyen <pclouds@gmail.com> writes:
Show 5 quoted lines
> Another aspect that's not mentioned is, we keep this daemon's logic as > thin as possible. The "brain" stays in git. So the daemon can read and > validate stuff, but that's about all it's allowed to do. It's not > supposed to add/create new contents. It's not even allowed to accept > direct updates from git.
That explanation does make the intent clear. It is a kind of design decision that needs to be made early and that is hard to change later (I am not saying I see the need of changing, though), so it is worth stating explicitly to guide future readers and updaters of the code.
Thanks.