ian.clatworthy at internode.on.net
Fri Jan 23 21:31:53 GMT 2009
Ian Clatworthy wrote:
> John Arbash Meinel wrote:
>> 1) I think we need to be thinking sooner rather than later about
>> handling partial ancestries.
> Shall do next.
>> 2) As written now, this doesn't actually win us anything, because it
>> doesn't actually cache the merge_sorted results.
> Maybe a separate patch?
The sorted graph is now cached. If we want to ditch the revid-to-revno
cache, can we do that as a separate patch?
>> 3) I would like to see us not include 'seq_num' as part of the generated
>> output. (Explicitly stripping it.) I don't think any code actually makes
>> use of it, and it is ill-defined when we are evaluating only partial
> I agree. If anything, it makes chaining of caches harder. The layer
> above can always use enumerate() if they need a sequence number.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 18441 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20090124/608d613a/attachment-0001.bin
More information about the bazaar