tikv / pd

Placement driver for TiKV
Apache License 2.0
1.05k stars 721 forks source link

operatror_controller:remove redundant logic introduced by #6427 #8696

Open AndreMouche opened 6 days ago

AndreMouche commented 6 days ago

Enhancement Task

the following logic was redundant since #6427 we could remove L469~L470 https://github.com/tikv/pd/blob/982fa22ffad01ebe640e5c23f3c6d0c67bde6742/pkg/schedule/operator/operator_controller.go#L469-L470

from the follow part:

https://github.com/tikv/pd/blob/982fa22ffad01ebe640e5c23f3c6d0c67bde6742/pkg/schedule/operator/operator_controller.go#L427-L472

lhy1024 commented 2 days ago

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