hyperledger-archives / fabric

THIS IS A READ-ONLY historic repository. Current development is at https://gerrit.hyperledger.org/r/#/admin/projects/fabric . pull requests not accepted
https://gerrit.hyperledger.org/
Apache License 2.0
1.17k stars 1.01k forks source link

Add field to block for consensus to store additional data #349

Closed srderson closed 8 years ago

srderson commented 8 years ago

An additional field needs to be added to the block for consensus to store additional data. For example, this is needed by PBFT consensus to store the PBFT sequence number in the block.

@kchristidis Can we just provide a string field or array of bytes?

srderson commented 8 years ago

@vukolic @corecode Please add any additional details that I may have missed.

kchristidis commented 8 years ago

@srderson: We're using b64-encoded strings.

corecode commented 8 years ago

opaque []byte or string is good.

binhn commented 8 years ago

We should also think about if any stats important to keep them readily available without much additional work. For example, the cumulative number of transactions since genesis, number of chaincodes = number of deploying transactions, etc.

srderson commented 8 years ago

Added as a []byte.