You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As far as I can see, this dependency on com.hedera.node.hapi is for three reasons:
SemanticVersion class
Printing detailed information about keys in StateLogger
NodeInfo.getAccountId() method
SemanticVersion can probably moved to swirlds-state-api or even swirlds-common. StateLogger can be simplified to utilize keys' toString() method or alike. The only real usage is in NodeInfo, which is in turn used from NetworkInfo. These classes are used in Hedera app code only, they aren't needed in platform code.
This ticket is to check if this dependency is really needed or can be removed.
The text was updated successfully, but these errors were encountered:
Current
module-info.java
inplatform-sdk/swirlds-state-api
:As far as I can see, this dependency on
com.hedera.node.hapi
is for three reasons:SemanticVersion
classStateLogger
NodeInfo.getAccountId()
methodSemanticVersion
can probably moved toswirlds-state-api
or evenswirlds-common
.StateLogger
can be simplified to utilize keys'toString()
method or alike. The only real usage is inNodeInfo
, which is in turn used fromNetworkInfo
. These classes are used in Hedera app code only, they aren't needed in platform code.This ticket is to check if this dependency is really needed or can be removed.
The text was updated successfully, but these errors were encountered: