Skip to content

Commit 907689c

Browse files
committed
Remove dup Transaction timestamp and version fields
The Transaction timestamp and version fields are duplicates of fields that are already on the transaction's ChainHeader, and nobody is using them. The header is the appropriate place for these fields, not in the Transaction itself. Note, there is a 1-to-1 relationship between the ChainHeader and the Transaction, no reason to include the fields twice. Change-Id: I039d4ca07d51d27d0e9c4b1a00f9b39db2ba08c8 Signed-off-by: denyeart <[email protected]>
1 parent eab249d commit 907689c

File tree

2 files changed

+30
-52
lines changed

2 files changed

+30
-52
lines changed

protos/peer/transaction.pb.go

+29-44
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

protos/peer/transaction.proto

+1-8
Original file line numberDiff line numberDiff line change
@@ -67,16 +67,9 @@ message ProcessedTransaction {
6767
// to ensure that they are treated atomically.
6868
message Transaction {
6969

70-
// Version indicates message protocol version.
71-
int32 version = 1;
72-
73-
// Timestamp is the local time that the
74-
// message was created by the sender
75-
google.protobuf.Timestamp timestamp = 2;
76-
7770
// The payload is an array of TransactionAction. An array is necessary to
7871
// accommodate multiple actions per transaction
79-
repeated TransactionAction actions = 3;
72+
repeated TransactionAction actions = 1;
8073
}
8174

8275
// TransactionAction binds a proposal to its action. The type field in the

0 commit comments

Comments
 (0)