Changed DB to store node positions as Ordinals.

sync/syncable/syncable_columns.h

Index: sync/syncable/syncable_columns.h
diff --git a/sync/syncable/syncable_columns.h b/sync/syncable/syncable_columns.h
index a6152c42de7d3f337bac8fa3819a01ee13784a2d..2dc7c7f4b7a1c6b73ff9426774fa53664fedfe5e 100644
--- a/sync/syncable/syncable_columns.h
+++ b/sync/syncable/syncable_columns.h
@@ -16,14 +16,13 @@ struct ColumnSpec {
   const char* spec;
 };
 
-// Must be in exact same order as fields in syncable.
+// Must be in exact same order as fields in entry_kernel.h.
 static const ColumnSpec g_metas_columns[] = {
   //////////////////////////////////////
   // int64s
   {"metahandle", "bigint primary key ON CONFLICT FAIL"},
   {"base_version", "bigint default " CHANGES_VERSION_STRING},
   {"server_version", "bigint default 0"},
-  {"server_position_in_parent", "bigint default 0"},
   // This is the item ID that we store for the embedding application.
   {"local_external_id", "bigint default 0"},
   // These timestamps are kept in the same format as that of the
@@ -54,10 +53,13 @@ static const ColumnSpec g_metas_columns[] = {
   {"unique_server_tag", "varchar"},
   {"unique_client_tag", "varchar"},
   //////////////////////////////////////
-  // Blobs.
+  // Blobs (serialized protos).
   {"specifics", "blob"},
   {"server_specifics", "blob"},
-  {"base_server_specifics", "blob"}
+  {"base_server_specifics", "blob"},
+  //////////////////////////////////////
+  // Blobs (ordinals)

[akalin, 2012/10/05 22:40:27]

period after (ordinals)

[vishwath, 2012/10/08 20:17:49]

Done.

+  {"server_ordinal_in_parent", "blob"},
 };
 
 // At least enforce that there are equal number of column names and fields.