Code Review suggestions and bug fixes.

Made Balance a class instead of a record as no benefit in using record here.

Added Id attribute to Balance.Value so that it is serialized and to allow state to be persisted after a Withdrawal.

Removed Reentrant attribute from AccountGrain as this could introduce race conditions when using Transactions.

Author: Rossimac

orleans/BankAccount/AccountTransfer.Grains/AccountGrain.cs

@@ -1,16 +1,15 @@
 using AccountTransfer.Interfaces;
-using Orleans.Concurrency;
 using Orleans.Transactions.Abstractions;
 
 namespace AccountTransfer.Grains;
 
 [GenerateSerializer]
-public class class Balance
+public class Balance
 {
+    [Id(0)] 
     public int Value { get; set; } = 1_000;
 }
 
-[Reentrant]
 public sealed class AccountGrain : Grain, IAccountGrain
 {
     private readonly ITransactionalState<Balance> _balance;
@@ -34,7 +33,7 @@ public Task Withdraw(int amount) =>
                     $" This account has {balance.Value} credits.");
             }
 
-            balance = balance.Value -= amount;
+            balance.Value -= amount;
         });
 
     public Task<int> GetBalance() =>