Jeremy Hylton wrote:
On 12/9/05, Tim Peters <[EMAIL PROTECTED]> wrote:

Log message for revision 40664:
 Nail the type of oids, as agreed ;-) to in

 U   ZODB/trunk/src/persistent/

Modified: ZODB/trunk/src/persistent/
--- ZODB/trunk/src/persistent/     2005-12-09 16:04:25 UTC (rev 
+++ ZODB/trunk/src/persistent/     2005-12-09 16:19:03 UTC (rev 
@@ -167,7 +167,9 @@

        It is up to the data manager to assign this.
        The special value None is reserved to indicate that an object
-        id has not been assigned.  Non-None object ids must be strings.
+        id has not been assigned.  Non-None object ids must be non-empty
+        strings.  The 8-byte string '\0'*8 (8 NUL bytes) is reserved to
+        identify the database root object.

A minor point, but it seems like it would make more sense for the
storage to choose the id of the root object.  The database doesn't
actually care what the root is, so long as the storage can return it. You could add a ROOT_OID class attribute to a storage to distinguis
the value.

As I said, a minor point.  If the database doesn't care about the
details of oids, it seems to make more sense to have the storage be
responsible for all aspects of oids.


This is a design change that would need to be implemented.
It's obviously not just a documentation change.


Jim Fulton           mailto:[EMAIL PROTECTED]       Python Powered!
CTO                  (540) 361-1714  
Zope Corporation
For more information about ZODB, see the ZODB Wiki:

ZODB-Dev mailing list  -

Reply via email to