On Wed, Sep 15, 2010 at 13:19, <julianf...@apache.org> wrote: >... > - - ### A comment on 'repos_id' and 'repos_relpath' says they may be null; > - is this true and wanted?
Yes. That meant "use the parent's information, and extend it to compute this node's value". > + - The Node-Rev group points to the corresponding repository node-rev. > > - - ### A comment on 'revnum' says, "this could be NULL for non-present > - nodes -- no info"; is this true and wanted? > + - If not used (as specified by the 'presence' table above), the values > + are undefined. > + ### Perhaps we should set them to null to make it clearer. Yes, we should. > > Content columns: (kind, properties, depth, target, checksum) > ---- ---------- ----- ------ -------- > @@ -165,6 +177,7 @@ Overview of BASE_NODE columns: > - Includes a copy of the corresponding date and author rev-props. > > - If not used (as specified by the 'presence' table above), all null. > + ### Not checked; in practice these columns may have undefined values. They should never be undefined. They should be null. We should never have undefined information in our schema. That leads to madness... Cheers, -g