Skip to content

Commit

Permalink
Add comment explaining GetValidatorSet implementation
Browse files Browse the repository at this point in the history
  • Loading branch information
michaelkaplan13 committed Oct 31, 2023
1 parent 5b83f0f commit e398734
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions relayer/canonical_validator_client.go
Original file line number Diff line number Diff line change
Expand Up @@ -36,10 +36,11 @@ func (v *CanonicalValidatorClient) GetSubnetID(ctx context.Context, chainID ids.

// Gets the current validator set of the given subnet ID, include the validators' BLS public keys.
// This implementation of GetValidatorSet currently makes two RPC requests, one to get the
// subnet validators, and another to get their BLS public keys. This is necessary because BLS keys
// are currently only associated with primary network validation periods. If ACP-13 is implementated
// in the future (https://github.com/avalanche-foundation/ACPs/blob/main/ACPs/13-subnet-only-validators.md),
// it may become possible to reduce this to a single RPC request that returns both the subnet validators
// subnet validators, and another to get their BLS public keys. This is necessary in order to enable
// the use of the public APIs (which don't support "GetValidatorsAt") because BLS keys are currently
// only associated with primary network validation periods. If ACP-13 is implementated in the future
// (https://github.com/avalanche-foundation/ACPs/blob/main/ACPs/13-subnet-only-validators.md), it may
// become possible to reduce this to a single RPC request that returns both the subnet validators
// as well as their BLS public keys.
func (v *CanonicalValidatorClient) GetValidatorSet(
ctx context.Context,
Expand Down

0 comments on commit e398734

Please sign in to comment.