8000 Update proto file references to follow renaming/versioning by mzabaluev · Pull Request #1555 · cometbft/cometbft · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update proto file references to follow renaming/versioning #1555

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Nov 8, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions proto/tendermint/services/block/v1/block.proto
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@ package tendermint.services.block.v1;
import "tendermint/types/block.proto";
import "tendermint/types/types.proto";

option go_package = "github.com/cometbft/cometbft/proto/tendermint/services/block/v1";

message GetByHeightRequest {
// The height of the block requested. If set to 0, the latest height will be returned.
int64 height = 1;
Expand Down
2 changes: 0 additions & 2 deletions proto/tendermint/services/block/v1/block_service.proto
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
syntax = "proto3";
package tendermint.services.block.v1;

option go_package = "github.com/cometbft/cometbft/proto/tendermint/services/block/v1";

import "tendermint/services/block/v1/block.proto";

// BlockService provides information about blocks
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@ package tendermint.services.block_results.v1;
import "tendermint/abci/types.proto";
import "tendermint/types/params.proto";

option go_package = "github.com/cometbft/cometbft/proto/tendermint/services/block_results/v1";

message GetBlockResultsRequest {
int64 height = 1;
}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
syntax = "proto3";
package tendermint.services.block_results.v1;

option go_package = "github.com/cometbft/cometbft/proto/tendermint/services/block_results/v1";

import "tendermint/services/block_results/v1/block_results.proto";

/*
Expand Down
2 changes: 0 additions & 2 deletions proto/tendermint/services/version/v1/version.proto
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
syntax = "proto3";
package tendermint.services.version.v1;

option go_package = "github.com/cometbft/cometbft/proto/tendermint/services/version/v1";

message GetVersionRequest {}

message GetVersionResponse {
Expand Down
2 changes: 0 additions & 2 deletions proto/tendermint/services/version/v1/version_service.proto
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
syntax = "proto3";
package tendermint.services.version.v1;

option go_package = "github.com/cometbft/cometbft/proto/tendermint/services/version/v1";

import "tendermint/services/version/v1/version.proto";

// VersionService simply provides version information about the node and the
Expand Down
4 changes: 2 additions & 2 deletions spec/abci/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,10 @@ message type.

The methods are always initiated by CometBFT. The Application implements its logic
for handling all ABCI++ methods.
Thus, CometBFT always sends the `Request*` messages and receives the `Response*` messages
Thus, CometBFT always sends the `*Request` messages and receives the `*Response` messages
in return.

All ABCI++ messages and methods are defined in [protocol buffers](https://github.com/cometbft/cometbft/blob/main/proto/tendermint/abci/types.proto).
All ABCI++ messages and methods are defined in [protocol buffers](https://github.com/cometbft/cometbft/blob/main/proto/cometbft/abci/v1beta4/types.proto).
This allows CometBFT to run with applications written in many programming languages.

This specification is split as follows:
Expand Down
9 changes: 5 additions & 4 deletions spec/abci/abci++_basic_concepts.md
Original file line number Diff line number Diff line change
Expand Up @@ -278,14 +278,14 @@ Note that some methods (`Query`, `FinalizeBlock`) return non-deterministic data
of `Info` and `Log` fields. The `Log` is intended for the literal output from the Application's
logger, while the `Info` is any additional info that should be returned. These are the only fields
that are not included in block header computations, so we don't need agreement
on them. All other fields in the `Response*` must be strictly deterministic.
on them. All other fields in the `*Response` must be strictly deterministic.

## Events

[↑ Back to Outline](#outline)

Method `FinalizeBlock` includes an `events` field at the top level in its
`Response*`, and one `events` field per transaction included in the block.
`FinalizeBlockResponse`, and one `events` field per transaction included in the block.
Applications may respond to this ABCI 2.0 method with an event list for each executed
transaction, and a general event list for the block itself.
Events allow applications to associate metadata with transactions and blocks.
Expand All @@ -298,7 +298,8 @@ execution. `Event` values can be used to index transactions and blocks according
happened during their execution.

Each event has a `type` which is meant to categorize the event for a particular
`Response*` or `Tx`. A `Response*` or `Tx` may contain multiple events with duplicate
`FinalizeBlockResponse` or `Tx`. A `FinalizeBlockResponse` or `Tx` may contain
multiple events with duplicate
`type` values, where each distinct entry is meant to categorize attributes for a
particular event. Every key and value in an event's attributes must be UTF-8
encoded strings along with the event type itself.
Expand Down Expand Up @@ -387,7 +388,7 @@ enum EvidenceType {

[↑ Back to Outline](#outline)

The `Query` and `CheckTx` methods include a `Code` field in their `Response*`.
The `Query` and `CheckTx` methods include a `Code` field in their `*Response`.
Field `Code` is meant to contain an application-specific response code.
A response code of `0` indicates no error. Any other response code
indicates to CometBFT that an error occurred.
Expand Down
2 changes: 1 addition & 1 deletion spec/abci/abci++_client_server.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ You are expected to have read all previous sections of ABCI++ specification, nam
## Message Protocol and Synchrony

The message protocol consists of pairs of requests and responses defined in the
[protobuf file](https://github.com/cometbft/cometbft/blob/main/proto/tendermint/abci/types.proto).
[protobuf file](https://github.com/cometbft/cometbft/blob/main/proto/cometbft/abci/v1beta4/types.proto).

Some messages have no fields, while others may include byte-arrays, strings, integers,
or custom protobuf types.
Expand Down
4 changes: 2 additions & 2 deletions spec/abci/abci++_comet_expected_behavior.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,10 @@ what will happen during a block height _h_ in these frequent, benign conditions:
* Consensus will decide in round 0, for height _h_;
* `PrepareProposal` will be called exactly once at the proposer process of round 0, height _h_;
* `ProcessProposal` will be called exactly once at all processes, and
will return _accept_ in its `Response*`;
will return _accept_ in its `ProcessProposalResponse`;
* `ExtendVote` will be called exactly once at all processes;
* `VerifyVoteExtension` will be called exactly _n-1_ times at each validator process, where _n_ is
the number of validators, and will always return _accept_ in its `Response*`;
the number of validators, and will always return _accept_ in its `VerifyVoteExtensionResponse`;
* `FinalizeBlock` will be called exactly once at all processes, conveying the same prepared
block that all calls to `PrepareProposal` and `ProcessProposal` had previously reported for
height _h_; and
Expand Down
0