comments and renames

Author: Zoran Cvetkov

graph/src/schema/entity_type.rs

@@ -151,10 +151,9 @@ impl EntityType {
         self.schema.is_object_type(self.atom)
     }
 
-    // Changes the way the VID field is generated. It used to be autoincrement. Now its
-    // based on block number and the order of the entities in a block. The latter
-    // represents the write order across all entity types in the subgraph.
-    pub fn strict_vid_order(&self) -> bool {
+    /// Whether the table for this entity type uses a sequence for the `vid` or whether
+    /// `graph-node` sets them explicitly. See  also [`InputSchema.strict_vid_order()`]
+    pub fn has_vid_seq(&self) -> bool {
         // Currently the agregations entities don't have VIDs in insertion order
         self.schema.strict_vid_order() && self.is_object_type()
     }

graph/src/schema/input/mod.rs

@@ -1589,6 +1589,10 @@ impl InputSchema {
         Some(EntityType::new(self.cheap_clone(), obj_type.name))
     }
 
+    /// How the values for the VID field are generated.
+    /// When this is `false`, this subgraph uses the old way of autoincrementing `vid` in the database.
+    /// When it is `true`, `graph-node` sets the `vid` explicitly to a number based on block number
+    /// and the order in which entities are written, and comparing by `vid` will order entities by that order.
     pub fn strict_vid_order(&self) -> bool {
         self.inner.spec_version >= SPEC_VERSION_1_3_0
     }

store/postgres/src/relational/ddl.rs

@@ -116,7 +116,7 @@ impl Table {
             Ok(cols)
         }
 
-        let vid_type = if self.object.strict_vid_order() {
+        let vid_type = if self.object.has_vid_seq() {
             "bigint"
         } else {
             "bigserial"

store/postgres/src/relational/prune.rs

@@ -205,7 +205,7 @@ impl TablePair {
 
         // Make sure the vid sequence continues from where it was in case
         // that we use autoincrementing order of the DB
-        if !self.src.object.strict_vid_order() {
+        if !self.src.object.has_vid_seq() {
             writeln!(
                 query,
                 "select setval('{dst_nsp}.{vid_seq}', nextval('{src_nsp}.{vid_seq}'));"

store/postgres/src/relational_queries.rs

@@ -2331,7 +2331,7 @@ impl<'a> InsertQuery<'a> {
         if table.has_causality_region {
             count += 1;
         }
-        if table.object.strict_vid_order() {
+        if table.object.has_vid_seq() {
             count += 1;
         }
         for column in table.columns.iter() {
@@ -2355,7 +2355,7 @@ impl<'a> QueryFragment<Pg> for InsertQuery<'a> {
         let out = &mut out;
         out.unsafe_to_cache_prepared();
 
-        let strict_vid_order = self.table.object.strict_vid_order();
+        let strict_vid_order = self.table.object.has_vid_seq();
 
         // Construct a query
         //   insert into schema.table(column, ...)
@@ -4805,7 +4805,7 @@ impl<'a> QueryFragment<Pg> for CopyEntityBatchQuery<'a> {
     fn walk_ast<'b>(&'b self, mut out: AstPass<'_, 'b, Pg>) -> QueryResult<()> {
         out.unsafe_to_cache_prepared();
 
-        let strict_vid_order = self.src.object.strict_vid_order();
+        let strict_vid_order = self.src.object.has_vid_seq();
 
         // Construct a query
         //   insert into {dst}({columns})