etcdserver: skip range requests in txn if the result is needless

If a server isn't serving txn requests from a client, the server
doesn't need the result of range requests in the txn.

This is a succeeding commit of
#5689

etcdserver/apply.go

@@ -788,3 +788,21 @@ func isGteRange(rangeEnd []byte) bool {
 func noSideEffect(r *pb.InternalRaftRequest) bool {
 	return r.Range != nil || r.AuthUserGet != nil || r.AuthRoleGet != nil
 }
+
+func removeNeedlessRangeReqs(txn *pb.TxnRequest) {
+	f := func(ops []*pb.RequestOp) []*pb.RequestOp {
+		j := 0
+		for i := 0; i < len(ops); i++ {
+			if _, ok := ops[i].Request.(*pb.RequestOp_RequestRange); ok {
+				continue
+			}
+			ops[j] = ops[i]
+			j++
+		}
+
+		return ops[:j]
+	}
+
+	txn.Success = f(txn.Success)
+	txn.Failure = f(txn.Failure)
+}

etcdserver/apply_auth.go

@@ -94,6 +94,11 @@ func (aa *authApplierV3) DeleteRange(txnID int64, r *pb.DeleteRangeRequest) (*pb
 
 func (aa *authApplierV3) checkTxnReqsPermission(reqs []*pb.RequestOp) error {
 	for _, requ := range reqs {
+		if requ == nil {
+			// removed by removeNeedlessRangeReqs()
+			continue
+		}
+
 		switch tv := requ.Request.(type) {
 		case *pb.RequestOp_RequestRange:
 			if tv.RequestRange == nil {

etcdserver/server.go

@@ -1080,7 +1080,11 @@ func (s *EtcdServer) applyEntryNormal(e *raftpb.Entry) {
 	}
 
 	var ar *applyResult
-	if s.w.IsRegistered(id) || !noSideEffect(&raftReq) {
+	needResult := s.w.IsRegistered(id)
+	if needResult || !noSideEffect(&raftReq) {
+		if !needResult && raftReq.Txn != nil {
+			removeNeedlessRangeReqs(raftReq.Txn)
+		}
 		ar = s.applyV3.Apply(&raftReq)
 	}