Thanks! The current stuff has worked when it was just Hyrum and I, but you're right: as we scale up the number of people, they become more important to do "now" rather than "later".
Most of the wc_db "operation" functions are unknown since we haven't tried to implement around them. Even the functions for maintaining the BASE nodes aren't really used. One particular pattern missing across the entire API is passing in an svn_skel_t for one or more work items to be transactionally inserted into WORK_QUEUE at the same time as the other database operations are performed. The work item(s) describe filesystem changes that need to occur to bring the on-disk world inline with the database view of the world. (compared to today, where the work items kind of do a mix of filesystem and database changes; they should only be doing filesystem in the future) Cheers, -g On Mon, Mar 1, 2010 at 08:02, Julian Foad <[email protected]> wrote: > We need doc strings. We always do, but especially at this point we need > them in order to assist other people in joining the effort to implement > WC-NG. I think one of the most useful things I can do at this point is > help write them. > > Doc strings go hand in hand with API design, as I'm sure we all know > (subconsciously at least), so as I try to write them I am necessarily > examining and trying to guess and then sometimes suggesting changes to > the API design. Sometimes that will be a separate step, sometimes part > of the same patch. > > Most of the APIs I'm looking at are private and sparsely documented, and > 'wc_db.h' has a comment ending, '... do not want to doc until it feels > like it is "Right"'. It seems the time is now Right to consolidate, > document, and change where better options are now apparent. I don't > want to step on anyone's toes or make gratuitous changes, so let me know > if I seem to be doing so. > > And I'm not going to do the whole API on my own. I'll do some parts. > Please step in and fix up another part of it if you can. > > - Julian > > >
