redo of hyperledger#5039 without the saveTrieLog change

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
macfarla · Feb 9, 2023 · 94e8742 · 94e8742
1 parent b2d378e
commit 94e8742
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 1 deletion.
diff --git a/...reum/core/src/main/java/org/hyperledger/besu/ethereum/bonsai/BonsaiLayeredWorldState.java b/...reum/core/src/main/java/org/hyperledger/besu/ethereum/bonsai/BonsaiLayeredWorldState.java
@@ -76,13 +76,33 @@ public Optional<BonsaiWorldView> getNextWorldView() {
   }
 
   public void setNextWorldView(final Optional<BonsaiWorldView> nextWorldView) {
+    maybeUnSubscribe();
     this.nextWorldView = nextWorldView;
   }
 
+  private void maybeUnSubscribe() {
+    nextWorldView
+        .filter(WorldState.class::isInstance)
+        .map(WorldState.class::cast)
+        .ifPresent(
+            ws -> {
+              try {
+                ws.close();
+              } catch (final Exception e) {
+                // no-op
+              }
+            });
+  }
+
   public TrieLogLayer getTrieLog() {
     return trieLog;
   }
 
+  @Override
+  public void close() throws Exception {
+    maybeUnSubscribe();
+  }
+
   public long getHeight() {
     return height;
   }

diff --git a/.../hyperledger/besu/ethereum/eth/transactions/sorter/AbstractPendingTransactionsSorter.java b/.../hyperledger/besu/ethereum/eth/transactions/sorter/AbstractPendingTransactionsSorter.java
@@ -428,7 +428,11 @@ private TransactionAddedStatus addTransaction(
             LOG,
             "Transaction {} not added because nonce too far in the future for sender {}",
             transaction::toTraceLog,
-            maybeSenderAccount::toString);
+            () ->
+                maybeSenderAccount
+                    .map(Account::getAddress)
+                    .map(Address::toString)
+                    .orElse("unknown"));
         return NONCE_TOO_FAR_IN_FUTURE_FOR_SENDER;
       }