Refactor to fix levelization:

* Remove Application & Database dependency in PerfLog. Replace it with
  a callback passed into the constructor.
* Fixes the circular dependency between ripple/nodestore and ripple/basics
This commit is contained in:
Edward Hennis
2022-01-12 10:53:34 -05:00
committed by manojsdoshi
parent a529b218f3
commit 0623a40f02
7 changed files with 23 additions and 16 deletions

View File

@@ -442,7 +442,6 @@ target_sources (rippled PRIVATE
#]===============================]
src/ripple/basics/impl/Archive.cpp
src/ripple/basics/impl/BasicConfig.cpp
src/ripple/basics/impl/PerfLogImp.cpp
src/ripple/basics/impl/ResolverAsio.cpp
src/ripple/basics/impl/UptimeClock.cpp
src/ripple/basics/impl/make_SSLContext.cpp
@@ -640,6 +639,11 @@ target_sources (rippled PRIVATE
src/ripple/rpc/impl/ShardVerificationScheduler.cpp
src/ripple/rpc/impl/Status.cpp
src/ripple/rpc/impl/TransactionSign.cpp
#[===============================[
main sources:
subdir: perflog
#]===============================]
src/ripple/perflog/impl/PerfLogImp.cpp
#[===============================[
main sources:

View File

@@ -36,12 +36,13 @@ listed later.
| 07 | ripple/shamap ripple/overlay
| 08 | ripple/app
| 09 | ripple/rpc
| 10 | test/jtx test/beast test/csf
| 11 | test/unit_test
| 12 | test/crypto test/conditions test/json test/resource test/shamap test/peerfinder test/basics test/overlay
| 13 | test
| 14 | test/net test/protocol test/ledger test/consensus test/core test/server test/nodestore
| 15 | test/rpc test/app
| 10 | ripple/perflog
| 11 | test/jtx test/beast test/csf
| 12 | test/unit_test
| 13 | test/crypto test/conditions test/json test/resource test/shamap test/peerfinder test/basics test/overlay
| 14 | test
| 15 | test/net test/protocol test/ledger test/consensus test/core test/server test/nodestore
| 16 | test/rpc test/app
(Note that `test` levelization is *much* less important and *much* less
strictly enforced than `ripple` levelization, other than the requirement

View File

@@ -28,15 +28,9 @@ Loop: ripple.basics ripple.core
Loop: ripple.basics ripple.json
ripple.json ~= ripple.basics
Loop: ripple.basics ripple.nodestore
ripple.nodestore > ripple.basics
Loop: ripple.basics ripple.protocol
ripple.protocol > ripple.basics
Loop: ripple.basics ripple.rpc
ripple.rpc > ripple.basics
Loop: ripple.core ripple.net
ripple.net > ripple.core

View File

@@ -29,6 +29,7 @@ ripple.net > ripple.beast
ripple.net > ripple.json
ripple.net > ripple.protocol
ripple.net > ripple.resource
ripple.nodestore > ripple.basics
ripple.nodestore > ripple.beast
ripple.nodestore > ripple.core
ripple.nodestore > ripple.json
@@ -46,6 +47,13 @@ ripple.peerfinder > ripple.basics
ripple.peerfinder > ripple.beast
ripple.peerfinder > ripple.core
ripple.peerfinder > ripple.protocol
ripple.perflog > ripple.basics
ripple.perflog > ripple.beast
ripple.perflog > ripple.core
ripple.perflog > ripple.json
ripple.perflog > ripple.nodestore
ripple.perflog > ripple.protocol
ripple.perflog > ripple.rpc
ripple.protocol > ripple.beast
ripple.protocol > ripple.crypto
ripple.protocol > ripple.json
@@ -53,6 +61,7 @@ ripple.resource > ripple.basics
ripple.resource > ripple.beast
ripple.resource > ripple.json
ripple.resource > ripple.protocol
ripple.rpc > ripple.basics
ripple.rpc > ripple.beast
ripple.rpc > ripple.core
ripple.rpc > ripple.crypto

View File

@@ -1339,8 +1339,6 @@ ApplicationImp::setup()
{
// Fall back to syncing from the network, such as
// when there's no existing data.
if (startUp == Config::NETWORK && !config_->standalone())
m_networkOPs->setNeedNetworkLedger();
startGenesisLedger();
}
else

View File

@@ -17,8 +17,9 @@
*/
//==============================================================================
#include <ripple/perflog/impl/PerfLogImp.h>
#include <ripple/basics/BasicConfig.h>
#include <ripple/basics/impl/PerfLogImp.h>
#include <ripple/beast/core/CurrentThreadName.h>
#include <ripple/beast/utility/Journal.h>
#include <ripple/core/JobTypes.h>