Skip to content

Commit 4bb6697

Browse files
author
Hui Kang
committed
[FAB-9607] doc fix: peers.md, ledger.md, tutorials.rst
- Fix some types and broken links - Clarify the difference about endorsement validation between phase 1 and 3 - write_first_app and build_network are reordered to be consistent with the intro text Change-Id: I16987b082021a110f86ed26d66bd2a48cf46784e Signed-off-by: Hui Kang <kangh@us.ibm.com>
1 parent 5d23c48 commit 4bb6697

File tree

3 files changed

+9
-4
lines changed

3 files changed

+9
-4
lines changed

docs/source/ledger/ledger.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,7 @@ as JSON documents because CouchDB supports the rich queries and update of richer
279279
data types often found in business transactions. Implementation-wise, CouchDB
280280
runs in a separate operating system process, but there is still a 1:1 relation
281281
between a network node and a CouchDB instance. All of this is invisible to
282-
chaincode. See [CouchDB as the StateDatabase](./couchdb_as_state_database.html)
282+
chaincode. See [CouchDB as the StateDatabase](../couchdb_as_state_database.html)
283283
for more information on CouchDB.
284284

285285
In LevelDB and CouchDB, we see an important aspect of Hyperledger Fabric -- it is

docs/source/peers/peers.md

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -375,7 +375,7 @@ been provided by Org1's peer P1!".
375375
Phase 1 ends when the application receives signed proposal responses from
376376
sufficient peers. We note that different peers can return different and
377377
therefore inconsistent transaction responses to the application *for the same
378-
transaction proposal*. It might simply be that the result was generated a
378+
transaction proposal*. It might simply be that the result was generated at
379379
different time on different peers with ledgers at different states -- in which
380380
case an application can simply request a more up-to-date proposal response. Less
381381
likely, but much more seriously, results might be different because the chaincode is *non-deterministic*. Non-determinism is the enemy of chaincodes
@@ -485,7 +485,12 @@ the transaction has been endorsed by the required organizations according to the
485485
example, some transactions may only need to be endorsed by a single
486486
organization, whereas others may require multiple endorsements before they are
487487
considered valid. This process of validation verifies that all relevant
488-
organizations have generated the same outcome or result.
488+
organizations have generated the same outcome or result. Also note that this
489+
validation is different than the endorsement check in phase 1, where it is the
490+
application that receives the response from endorsing peers and makes the
491+
decision to send the proposal transactions. In case the application violates
492+
the endorsement policy by sending wrong transactions, the peer is still able to
493+
reject the transaction in the validation process of phase 3.
489494

490495
If a transaction has been endorsed correctly, the peer will attempt to apply it
491496
to the ledger. To do this, a peer must perform a ledger consistency check to

docs/source/tutorials.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ Finally, we offer two chaincode tutorials. One oriented to developers,
2727
:maxdepth: 1
2828
:caption: Tutorials
2929

30-
build_network
3130
write_first_app
31+
build_network
3232
channel_update_tutorial
3333
upgrading_your_network_tutorial
3434
chaincode

0 commit comments

Comments
 (0)