Changed DB to store node positions as Ordinals.

sync/syncable/syncable_columns.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef SYNC_SYNCABLE_SYNCABLE_COLUMNS_H_
 #define SYNC_SYNCABLE_SYNCABLE_COLUMNS_H_
 
 #include "sync/syncable/entry_kernel.h"
 #include "sync/syncable/syncable_changes_version.h"
 
 namespace syncer {
 namespace syncable {
 
 struct ColumnSpec {
   const char* name;
   const char* spec;
 };
 
 // Must be in exact same order as fields in syncable.
 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"},
+  {"server_version", "blob"},

[akalin, 2012/10/01 23:13:35]

why is server_version a blob also?

[vishwath, 2012/10/02 01:06:33]

Not sure how that crept in, fixed.

On 2012/10/01 23:13:35, akalin wrote:

+  {"server_position_in_parent", "blob"},
   // 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
   // protocol (ms since Unix epoch).
   {"mtime", "bigint default 0"},
   {"server_mtime", "bigint default 0"},
   {"ctime", "bigint default 0"},
   {"server_ctime", "bigint default 0"},
   //////////////////////////////////////
   // Ids
@@ skipping 29 matching lines @@
 
 static inline const char* ColumnName(int field) {
   DCHECK(field < BEGIN_TEMPS);
   return g_metas_columns[field].name;
 }
 
 }  // namespace syncable
 }  // namespace syncer
 
 #endif  // SYNC_SYNCABLE_SYNCABLE_COLUMNS_H_