Skip to content

Commit

Permalink
[FAB-18262] Fix wrong comments on OrdererType options (#37)
Browse files Browse the repository at this point in the history
This patch replaces wrong comments on OrdererType options with
the correct constants ConsensusTypeSolo, ConsensusTypeKafka
and ConsensusTypeEtcdRaft.

Also, this patch includes a minor fix to pass lint checks.

Signed-off-by: Tatsuya Sato <[email protected]>
  • Loading branch information
satota2 authored Oct 8, 2020
1 parent dfa925e commit ac4b35d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion configtx/internal/policydsl/policyparser.go
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ func secondPass(args ...interface{}) (interface{}, error) {
}

/* get the n in the t out of n */
var n int = len(args) - 2
var n = len(args) - 2

/* sanity check - t should be positive, permit equal to n+1, but disallow over n+1 */
if t < 0 || t > n+1 {
Expand Down
2 changes: 1 addition & 1 deletion configtx/orderer.go
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ const (
// Orderer configures the ordering service behavior for a channel.
type Orderer struct {
// OrdererType is the type of orderer
// Options: `Solo`, `Kafka` or `Raft`
// Options: `ConsensusTypeSolo`, `ConsensusTypeKafka` or `ConsensusTypeEtcdRaft`
OrdererType string
// BatchTimeout is the wait time between transactions.
BatchTimeout time.Duration
Expand Down

0 comments on commit ac4b35d

Please sign in to comment.