On Wed, Dec 17, 2008 at 11:06:15AM -0600, Govind Salinas wrote:
> Yes, I was thinking that this is the natural way to do things, save that I
> would be lazy loading the trees into a cache instead of caching them
> all up front. This is one of the reasons that I think the fan out will
> help.
I was working under the assumption that you are going to do multiple
note lookups. If you are, then the fan-out isn't really going to help,
as you're going to end up pulling in all of the subtrees anyway. It
helps some if you're only doing a single lookup, but I don't know if
that is measurable.
> Yes, I completely agree that I want it to have the same scheme as what
> git will use. That is the reason I posted this here. Since no method
> has been formally accepted (checked into master) I wanted to see if
> I could nudge things along. I wasn't aware that you and Dscho had
> a (very similar) plan. Please, if you guys are decided on the format
> then I can just go off and start working on it. But it sounds like there
> isn't consensus yet.
This is probably not the answer you want, but I think the final design
depends on some C experiments. For example, whether or not there should
be fan-out depends on how it affects performance, which means we need to
do at least partial implementations to compare. So it really is just
waiting for somebody to sit down and do it.
> I like the overall plan, but I would suggest that --notes[=default] and
> --note-type=whatever would be a little friendlier and less error prone.
But keeping it as a single string means there is no ambiguity when you
specify multiple notes at once. For example:
git log \
--note-filter='test:status == "fail" && importance > 3' \
--pretty=format:%h%n%N(test:errors)
would do something like:
foreach commit $C
compare refs/notes/test:$C/status against the string "fail"
compare refs/notes/default:$C/importance against the number 3
if either don't match, skip the commit
show the hash and the contents of refs/notes/test:$C/errors
and obviously that filter language is totally made up and we may or may
not want to do something that complex. But my point is that we are
defining a namespace of notes, and we want to be able to refer to a
multiple fully qualified names.
-Peff
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
