operatror_controller:remove redundant logic introduced by #6427

[AndreMouche]

Enhancement Task

the following logic was redundant since #6427
we could remove L469~L470

pd/pkg/schedule/operator/operator_controller.go

Lines 469 to 470 in 982fa22

	if op.SchedulerKind() == OpAdmin || op.IsLeaveJointStateOperator() {
	continue

from the follow part:

pd/pkg/schedule/operator/operator_controller.go

Lines 427 to 472 in 982fa22

	for _, op := range ops {
	region := oc.cluster.GetRegion(op.RegionID())
	if region == nil {
	log.Debug("region not found, cancel add operator",
	zap.Uint64("region-id", op.RegionID()))
	operatorCounter.WithLabelValues(op.Desc(), "not-found").Inc()
	return false, RegionNotFound
	}
	if region.GetRegionEpoch().GetVersion() != op.RegionEpoch().GetVersion() ||
	region.GetRegionEpoch().GetConfVer() != op.RegionEpoch().GetConfVer() {
	log.Debug("region epoch not match, cancel add operator",
	zap.Uint64("region-id", op.RegionID()),
	zap.Reflect("old", region.GetRegionEpoch()),
	zap.Reflect("new", op.RegionEpoch()))
	operatorCounter.WithLabelValues(op.Desc(), "epoch-not-match").Inc()
	return false, EpochNotMatch
	}
	if oldi, ok := oc.operators.Load(op.RegionID()); ok && oldi.(*Operator) != nil && !isHigherPriorityOperator(op, oldi.(*Operator)) {
	old := oldi.(*Operator)
	log.Debug("already have operator, cancel add operator",
	zap.Uint64("region-id", op.RegionID()),
	zap.Reflect("old", old))
	operatorCounter.WithLabelValues(op.Desc(), "already-have").Inc()
	return false, AlreadyExist
	}
	if op.Status() != CREATED {
	log.Error("trying to add operator with unexpected status",
	zap.Uint64("region-id", op.RegionID()),
	zap.String("status", OpStatusToString(op.Status())),
	zap.Reflect("operator", op), errs.ZapError(errs.ErrUnexpectedOperatorStatus))
	failpoint.Inject("unexpectedOperator", func() {
	panic(op)
	})
	operatorCounter.WithLabelValues(op.Desc(), "unexpected-status").Inc()
	return false, NotInCreateStatus
	}
	if !isPromoting && oc.wopStatus.getCount(op.Desc()) >= oc.config.GetSchedulerMaxWaitingOperator() {
	log.Debug("exceed max return false", zap.Uint64("waiting", oc.wopStatus.getCount(op.Desc())), zap.String("desc", op.Desc()), zap.Uint64("max", oc.config.GetSchedulerMaxWaitingOperator()))
	operatorCounter.WithLabelValues(op.Desc(), "exceed-max-waiting").Inc()
	return false, ExceedWaitLimit
	}
	if op.SchedulerKind() == OpAdmin || op.IsLeaveJointStateOperator() {
	continue
	}
	}

[lhy1024]

why is it redundant? Would you like to say more details?