chaosi-zju
chaosi-zju
CC @RainbowMango
CC @RainbowMango #4564 and #4570 are all PRs about code modification, and ready to review, thanks.
> Sorry, let this sit. @chaosi-zju can you help to take a look? @zhzhuang-zju is more familiar with `karmadactl`, I have asked him for help
this PR's reviewing should wait util #4510 and #4553 merged and do a rebase. /hold
I have tested and confirmed as a problem. besides, you gave a good scenario for #3318
> will this modification require compatibility I don't think it matters. What do you think? > Will it force users to upgrade the Estimator component I think it's only about...
CC @RainbowMango may be ready to merge, any further comments?
@whitewindmills sorry for delayed a while, I have updated the code, do you have time checking once more?