From b0e3afd8a3f0f2530f6b7a5020a390f9b4e29960 Mon Sep 17 00:00:00 2001
From: "Leo Zhang (zhangchiqing)" <zhangchiqing@gmail.com>
Date: Thu, 16 Nov 2023 08:58:56 -0800
Subject: [PATCH] remove changes

---
 .../storehouse/executing_block_snapshot.go    | 96 -------------------
 1 file changed, 96 deletions(-)
 delete mode 100644 engine/execution/storehouse/executing_block_snapshot.go

diff --git a/engine/execution/storehouse/executing_block_snapshot.go b/engine/execution/storehouse/executing_block_snapshot.go
deleted file mode 100644
index 7f8775912bd..00000000000
--- a/engine/execution/storehouse/executing_block_snapshot.go
+++ /dev/null
@@ -1,96 +0,0 @@
-package storehouse
-
-import (
-	"fmt"
-
-	"github.com/onflow/flow-go/engine/execution"
-	"github.com/onflow/flow-go/fvm/storage/snapshot"
-	"github.com/onflow/flow-go/ledger"
-	"github.com/onflow/flow-go/ledger/common/convert"
-	"github.com/onflow/flow-go/model/flow"
-)
-
-var _ execution.ExtendableStorageSnapshot = (*ExecutingBlockSnapshot)(nil)
-
-// ExecutingBlockSnapshot is a snapshot of the storage at an executed collection.
-// It starts with a storage snapshot at the end of previous block,
-// The register updates at the executed collection at baseHeight + 1 are cached in
-// a map, such that retrieving register values at the snapshot will first check
-// the cache, and then the storage.
-type ExecutingBlockSnapshot struct {
-	// the snapshot at the end of previous block
-	previous snapshot.StorageSnapshot
-
-	commitment      flow.StateCommitment
-	registerUpdates map[flow.RegisterID]flow.RegisterValue
-}
-
-// create a new storage snapshot for an executed collection
-// at the base block at height h - 1
-func NewExecutingBlockSnapshot(
-	previous snapshot.StorageSnapshot,
-	// the statecommitment of a block at height h
-	commitment flow.StateCommitment,
-) *ExecutingBlockSnapshot {
-	return &ExecutingBlockSnapshot{
-		previous:        previous,
-		commitment:      commitment,
-		registerUpdates: make(map[flow.RegisterID]flow.RegisterValue),
-	}
-}
-
-// Get returns the register value at the snapshot.
-func (s *ExecutingBlockSnapshot) Get(id flow.RegisterID) (flow.RegisterValue, error) {
-	// get from latest updates first
-	value, ok := s.getFromUpdates(id)
-	if ok {
-		return value, nil
-	}
-
-	// get from BlockEndStateSnapshot at previous block
-	value, err := s.previous.Get(id)
-	return value, err
-}
-
-func (s *ExecutingBlockSnapshot) getFromUpdates(id flow.RegisterID) (flow.RegisterValue, bool) {
-	value, ok := s.registerUpdates[id]
-	return value, ok
-}
-
-// Extend returns a new storage snapshot at the same block but but for a different state commitment,
-// which contains the given trieUpdate
-// Usually it's used to create a new storage snapshot at the next executed collection.
-// The trieUpdate contains the register updates at the executed collection.
-func (s *ExecutingBlockSnapshot) Extend(newCommit flow.StateCommitment, updatedRegisters flow.RegisterEntries) (execution.ExtendableStorageSnapshot, error) {
-	updates := make(map[flow.RegisterID]flow.RegisterValue)
-
-	// add the old updates
-	// overwrite with new updates
-	for _, entry := range updatedRegisters {
-		updates[entry.Key] = entry.Value
-	}
-
-	return &ExecutingBlockSnapshot{
-		previous:        s.previous,
-		commitment:      newCommit,
-		registerUpdates: updates,
-	}, nil
-}
-
-func (s *ExecutingBlockSnapshot) Commitment() flow.StateCommitment {
-	return s.commitment
-}
-
-func PayloadToRegister(payload *ledger.Payload) (flow.RegisterID, flow.RegisterValue, error) {
-	key, err := payload.Key()
-	if err != nil {
-		return flow.RegisterID{}, flow.RegisterValue{}, fmt.Errorf("could not parse register key from payload: %w", err)
-	}
-	regID, err := convert.LedgerKeyToRegisterID(key)
-	if err != nil {
-		return flow.RegisterID{}, flow.RegisterValue{}, fmt.Errorf("could not parse register key from payload: %w", err)
-	}
-
-	return regID, payload.Value(), nil
-
-}