-
Notifications
You must be signed in to change notification settings - Fork 878
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
New ACCOUNT_FREEZER_STATE DB segment. New BonsaiArchiveFreezer to lis…
…ten for blocks and move account state to new DB segment Signed-off-by: Matthew Whitehead <[email protected]>
- Loading branch information
1 parent
67fa46f
commit 7d4a524
Showing
7 changed files
with
299 additions
and
8 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
161 changes: 161 additions & 0 deletions
161
...a/org/hyperledger/besu/ethereum/trie/diffbased/bonsai/worldview/BonsaiArchiveFreezer.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,161 @@ | ||
| /* | ||
| * Copyright contributors to Hyperledger Besu. | ||
| * | ||
| * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with | ||
| * the License. You may obtain a copy of the License at | ||
| * | ||
| * http://www.apache.org/licenses/LICENSE-2.0 | ||
| * | ||
| * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on | ||
| * an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the | ||
| * specific language governing permissions and limitations under the License. | ||
| * | ||
| * SPDX-License-Identifier: Apache-2.0 | ||
| */ | ||
| package org.hyperledger.besu.ethereum.trie.diffbased.bonsai.worldview; | ||
|
|
||
| import org.hyperledger.besu.datatypes.Hash; | ||
| import org.hyperledger.besu.ethereum.chain.BlockAddedEvent; | ||
| import org.hyperledger.besu.ethereum.chain.BlockAddedObserver; | ||
| import org.hyperledger.besu.ethereum.chain.Blockchain; | ||
| import org.hyperledger.besu.ethereum.trie.diffbased.common.storage.DiffBasedWorldStateKeyValueStorage; | ||
| import org.hyperledger.besu.ethereum.trie.diffbased.common.trielog.TrieLogManager; | ||
| import org.hyperledger.besu.plugin.services.trielogs.TrieLog; | ||
|
|
||
| import java.util.Comparator; | ||
| import java.util.Optional; | ||
| import java.util.concurrent.atomic.AtomicInteger; | ||
| import java.util.function.Consumer; | ||
|
|
||
| import com.google.common.collect.ArrayListMultimap; | ||
| import com.google.common.collect.Multimap; | ||
| import com.google.common.collect.TreeMultimap; | ||
| import org.apache.tuweni.bytes.Bytes; | ||
| import org.slf4j.Logger; | ||
| import org.slf4j.LoggerFactory; | ||
|
|
||
| /** | ||
| * This class manages the "freezing" of historic state that is still needed to satisfy queries but | ||
| * doesn't need to be in the main DB segment for. Doing so would degrade block-import performance | ||
| * over time so we move state beyond a certain age (in blocks) to other DB segments, assuming there | ||
| * is a more recent (i.e. changed) version of the state. If state is created once and never changed | ||
| * it will remain in the primary DB segment(s). | ||
| */ | ||
| public class BonsaiArchiveFreezer implements BlockAddedObserver { | ||
|
|
||
| private static final Logger LOG = LoggerFactory.getLogger(BonsaiArchiveFreezer.class); | ||
|
|
||
| private final DiffBasedWorldStateKeyValueStorage rootWorldStateStorage; | ||
| private final Blockchain blockchain; | ||
| private final Consumer<Runnable> executeAsync; | ||
| private final long numberOfBlocksToKeepInWarmStorage; | ||
| private final TrieLogManager trieLogManager; | ||
|
|
||
| private final Multimap<Long, Hash> blocksToMoveToFreezer = | ||
| TreeMultimap.create(Comparator.reverseOrder(), Comparator.naturalOrder()); | ||
|
|
||
| public BonsaiArchiveFreezer( | ||
| final DiffBasedWorldStateKeyValueStorage rootWorldStateStorage, | ||
| final Blockchain blockchain, | ||
| final Consumer<Runnable> executeAsync, | ||
| final long numberOfBlocksToKeepInWarmStorage, | ||
| final TrieLogManager trieLogManager) { | ||
| this.rootWorldStateStorage = rootWorldStateStorage; | ||
| this.blockchain = blockchain; | ||
| this.executeAsync = executeAsync; | ||
| this.numberOfBlocksToKeepInWarmStorage = numberOfBlocksToKeepInWarmStorage; | ||
| this.trieLogManager = trieLogManager; | ||
| } | ||
|
|
||
| public int initialize() { | ||
| // TODO Probably need to freeze old blocks that haven't been frozen already? | ||
| return 0; | ||
| } | ||
|
|
||
| public synchronized void addToFreezerQueue(final long blockNumber, final Hash blockHash) { | ||
| LOG.atDebug() | ||
| .setMessage( | ||
| "adding block to archive freezer queue for moving to cold storage, blockNumber {}; blockHash {}") | ||
| .addArgument(blockNumber) | ||
| .addArgument(blockHash) | ||
| .log(); | ||
| blocksToMoveToFreezer.put(blockNumber, blockHash); | ||
| } | ||
|
|
||
| public synchronized int moveBlockStateToFreezer() { | ||
| final long retainAboveThisBlock = | ||
| blockchain.getChainHeadBlockNumber() - numberOfBlocksToKeepInWarmStorage; | ||
| if (rootWorldStateStorage.getFlatDbMode().getVersion() == Bytes.EMPTY) { | ||
| throw new IllegalStateException("DB mode version not set"); | ||
| } | ||
|
|
||
| AtomicInteger frozenStateCount = new AtomicInteger(); | ||
|
|
||
| LOG.atDebug() | ||
| .setMessage( | ||
| "Moving cold state to freezer storage (chainHeadNumber: {} - numberOfBlocksToKeepInWarmStorage: {}) = {}") | ||
| .addArgument(blockchain::getChainHeadBlockNumber) | ||
| .addArgument(numberOfBlocksToKeepInWarmStorage) | ||
| .addArgument(retainAboveThisBlock) | ||
| .log(); | ||
|
|
||
| final var blocksToMove = | ||
| blocksToMoveToFreezer.asMap().entrySet().stream() | ||
| .dropWhile((e) -> e.getKey() > retainAboveThisBlock); | ||
| // TODO - limit to a configurable number of blocks to move per loop | ||
|
|
||
| final Multimap<Long, Hash> movedToFreezer = ArrayListMultimap.create(); | ||
|
|
||
| // Determine which world state keys have changed in the last N blocks by looking at the | ||
| // trie logs for the blocks. Then move the old keys to the freezer segment (if and only if they | ||
| // have changed) | ||
| blocksToMove.forEach( | ||
| (block) -> { | ||
| for (Hash blockHash : block.getValue()) { | ||
| Optional<TrieLog> trieLog = trieLogManager.getTrieLogLayer(blockHash); | ||
| if (trieLog.isPresent()) { | ||
| trieLog | ||
| .get() | ||
| .getAccountChanges() | ||
| .forEach( | ||
| (address, change) -> { | ||
| // Move any previous state for this account | ||
| frozenStateCount.addAndGet( | ||
| rootWorldStateStorage.freezePreviousAccountState( | ||
| blockchain.getBlockHeader(blockHash), | ||
| blockchain.getBlockHeader(block.getKey() - 1), | ||
| address.addressHash())); | ||
| // TODO - block number - 1 is a hack until getNearestBefore() is pulled in | ||
| }); | ||
| } | ||
| movedToFreezer.put(block.getKey(), blockHash); | ||
| } | ||
| }); | ||
|
|
||
| movedToFreezer.keySet().forEach(blocksToMoveToFreezer::removeAll); | ||
|
|
||
| if (frozenStateCount.get() > 0) { | ||
| LOG.atInfo() | ||
| .setMessage("froze {} state entries for {} blocks") | ||
| .addArgument(frozenStateCount.get()) | ||
| .addArgument(movedToFreezer::size) | ||
| .log(); | ||
| } | ||
|
|
||
| return movedToFreezer.size(); | ||
| } | ||
|
|
||
| @Override | ||
| public void onBlockAdded(final BlockAddedEvent addedBlockContext) { | ||
| final Hash blockHash = addedBlockContext.getBlock().getHeader().getBlockHash(); | ||
| final Optional<Long> blockNumber = | ||
| Optional.of(addedBlockContext.getBlock().getHeader().getNumber()); | ||
| blockNumber.ifPresent( | ||
| blockNum -> | ||
| executeAsync.accept( | ||
| () -> { | ||
| addToFreezerQueue(blockNum, blockHash); | ||
| moveBlockStateToFreezer(); | ||
| })); | ||
| } | ||
| } |
Oops, something went wrong.