x/stake: Speed up handleMsgEditValidator

This removes running the power update logic on handleMsgEditValidator,
as its unnecessary.

Closes #1815

PENDING.md

@@ -50,6 +50,7 @@ IMPROVEMENTS
 * [tests] Fixes ansible scripts to work with AWS too
 * [tests] \#1806 CLI tests are now behind the build flag 'cli_test', so go test works on a new repo
 * [x/gov] Initial governance parameters can now be set in the genesis file
+* [x/stake] \#1815 Sped up the processing of `EditValidator` txs. 
 
 BUG FIXES
 *  \#1666 Add intra-tx counter to the genesis validators

x/stake/handler.go

@@ -111,7 +111,9 @@ func handleMsgEditValidator(ctx sdk.Context, msg types.MsgEditValidator, k keepe
 	}
 	validator.Description = description
 
-	k.UpdateValidator(ctx, validator)
+	// We don't need to run through all the power update logic within k.UpdateValidator
+	// We just need to override the entry in state, since only the description has changed.
+	k.SetValidator(ctx, validator)
 	tags := sdk.NewTags(
 		tags.Action, tags.ActionEditValidator,
 		tags.DstValidator, []byte(msg.ValidatorAddr.String()),

x/stake/keeper/validator.go

@@ -191,9 +191,9 @@ func (k Keeper) ClearTendermintUpdates(ctx sdk.Context) {
 
 //___________________________________________________________________________
 
-// Perfom all the nessisary steps for when a validator changes its power.  This
+// Perfom all the necessary steps for when a validator changes its power. This
 // function updates all validator stores as well as tendermint update store.
-// It may kick out validators if new validator is entering the bonded validator
+// It may kick out validators if a new validator is entering the bonded validator
 // group.
 //
 // nolint: gocyclo