address the comment

Signed-off-by: yisaer <disxiaofei@163.com>

Author: Yisaer

server/schedule/filter/filters.go

@@ -389,7 +389,7 @@ func (f *StoreStateFilter) anyConditionMatch(typ int, opt *config.PersistOptions
 		funcs = []conditionFunc{f.isTombstone, f.isOffline, f.isDown, f.isDisconnected, f.isBusy,
 			f.exceedAddLimit, f.tooManySnapshots, f.tooManyPendingPeers}
 	case scatterRegionTarget:
-		funcs = []conditionFunc{f.isTombstone, f.isOffline, f.isDown, f.isDisconnected}
+		funcs = []conditionFunc{f.isTombstone, f.isOffline, f.isDown, f.isDisconnected, f.isBusy}
 	}
 	for _, cf := range funcs {
 		if cf(opt, store) {
@@ -759,32 +759,6 @@ func (f *isolationFilter) Target(opt *config.PersistOptions, store *core.StoreIn
 	return true
 }
 
-// NewStoreBusyFilter creates storeBusyFilter
-func NewStoreBusyFilter(scope string) *storeBusyFilter {
-	return &storeBusyFilter{scope: scope}
-}
-
-// StoreBusyFilter
-type storeBusyFilter struct {
-	scope string
-}
-
-func (f *storeBusyFilter) Scope() string {
-	return f.scope
-}
-
-func (f *storeBusyFilter) Type() string {
-	return "store-busy-filter"
-}
-
-func (f *storeBusyFilter) Source(opt *config.PersistOptions, store *core.StoreInfo) bool {
-	return true
-}
-
-func (f *storeBusyFilter) Target(opt *config.PersistOptions, store *core.StoreInfo) bool {
-	return !store.IsBusy()
-}
-
 // createRegionForRuleFit is used to create a clone region with RegionCreateOptions which is only used for
 // FitRegion in filter
 func createRegionForRuleFit(startKey, endKey []byte,

server/schedule/region_scatterer.go

@@ -325,7 +325,6 @@ func (r *RegionScatterer) selectCandidates(region *core.RegionInfo, sourceStoreI
 	scoreGuard := filter.NewPlacementSafeguard(r.name, r.cluster, region, sourceStore)
 	filters = append(filters, context.filters...)
 	filters = append(filters, scoreGuard)
-	filters = append(filters, filter.NewStoreBusyFilter(r.name))
 	stores := r.cluster.GetStores()
 	candidates := make([]uint64, 0)
 	for _, store := range stores {