Merge pull request #4792 from systeminit/jkeiser/correct-snapshot-add…

…ress Report new snapshot address (not old) in billing event
systeminit · Oct 14, 2024 · 8d05e31 · 8d05e31
2 parents 49cd674 + a1f2ffb
commit 8d05e31
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/dal/src/change_set.rs b/lib/dal/src/change_set.rs
@@ -297,10 +297,6 @@ impl ChangeSet {
         ctx: &DalContext,
         workspace_snapshot_address: WorkspaceSnapshotAddress,
     ) -> ChangeSetResult<()> {
-        billing_publish::for_head_change_set_pointer_update(ctx, self)
-            .await
-            .map_err(Box::new)?;
-
         ctx.txns()
             .await?
             .pg()
@@ -312,6 +308,10 @@ impl ChangeSet {
 
         self.workspace_snapshot_address = workspace_snapshot_address;
 
+        billing_publish::for_head_change_set_pointer_update(ctx, self)
+            .await
+            .map_err(Box::new)?;
+
         Ok(())
     }