Skip to content

Commit

Permalink
Addressed review comments - removed SoftwareVersion message and rep…
Browse files Browse the repository at this point in the history
…laced it with `SemanticVersion`

Signed-off-by: Ivan Malygin <ivan@swirldslabs.com>
  • Loading branch information
imalygin committed Jun 13, 2024
1 parent f1f8e31 commit d854ee2
Showing 1 changed file with 3 additions and 30 deletions.
33 changes: 3 additions & 30 deletions platform/state/platform_state.proto
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ message PlatformState {
* <p>
* This SHALL be the software version that created this state.
*/
SoftwareVersion creation_software_version = 3;
proto.SemanticVersion creation_software_version = 3;

/**
* A number of non-ancient rounds.
Expand Down Expand Up @@ -100,14 +100,14 @@ message PlatformState {
proto.Timestamp last_frozen_time = 7;

/**
* A consensus node software version.<br/>
* A consensus node semantic version.<br/>
* The software version that enabled birth round mode.
* <p>
* This SHALL be unset if birth round migration has not yet happened.<br/>
* If birth round migration is complete, this SHALL be the _first_ software
* version that enabled birth round mode.
*/
SoftwareVersion first_version_in_birth_round_mode = 8;
proto.SemanticVersion first_version_in_birth_round_mode = 8;

/**
* A consensus round.<br/>
Expand Down Expand Up @@ -138,33 +138,6 @@ message PlatformState {
}


/**
* A consensus node software version.
*
* This message records version information for configuration, the Hedera
* API, and the "Services" subsystems.
*/
message SoftwareVersion {
/**
* A single numeric version.<br/>
* This is the current configuration version read on node startup.
*/
uint32 config_version = 1;

/**
* A semantic version entry.<br/>
* This is the version of the HAPI module (Hedera API).
*/
proto.SemanticVersion hapi_version = 2;

/**
* A semantic version entry.<br/>
* This is the version of the services module.
*/
proto.SemanticVersion services_version = 3;
}


/**
* A consensus snapshot.<br/>
* This is a snapshot of the consensus state for a particular round.
Expand Down

0 comments on commit d854ee2

Please sign in to comment.