Adds datapoint if rent collection changes only the rent epoch (#2922)

anza-xyz · Sep 19, 2024 · 33167a3 · 33167a3
1 parent c1b465d
commit 33167a3
Showing 1 changed file with 13 additions and 2 deletions.
diff --git a/runtime/src/bank.rs b/runtime/src/bank.rs
@@ -4322,8 +4322,8 @@ impl Bank {
             let rent_epoch_post = account.rent_epoch();
 
             // did the account change in any way due to rent collection?
-            let account_changed =
-                rent_collected_info.rent_amount != 0 || rent_epoch_post != rent_epoch_pre;
+            let rent_epoch_changed = rent_epoch_post != rent_epoch_pre;
+            let account_changed = rent_collected_info.rent_amount != 0 || rent_epoch_changed;
 
             // always store the account, regardless if it changed or not
             let always_store_accounts =
@@ -4361,6 +4361,17 @@ impl Bank {
                             );
                         }
                     }
+                } else {
+                    debug_assert_eq!(rent_collected_info.rent_amount, 0);
+                    if rent_epoch_changed {
+                        datapoint_info!(
+                            "bank-rent_collection_updated_only_rent_epoch",
+                            ("slot", self.slot(), i64),
+                            ("pubkey", pubkey.to_string(), String),
+                            ("rent_epoch_pre", rent_epoch_pre, i64),
+                            ("rent_epoch_post", rent_epoch_post, i64),
+                        );
+                    }
                 }
                 total_rent_collected_info += rent_collected_info;
                 accounts_to_store.push((pubkey, account));