wuchong commented on PR #3614: URL: https://github.com/apache/fluss/pull/3614#issuecomment-4946476983
Btw, Could we rename `KvLeaderReplicaCapacityManager` to `ReplicaCapacityController`? `Replica` provides a better component boundary because capacity control may later cover log replicas and KV hot replicas in addition to KV leader replicas. `Controller` is also more accurate than `Manager`: this component does not manage replica lifecycles; it makes creation admission decisions based on observed replica counts and cluster resource capacity. However, the current methods, metrics, and configuration should remain explicitly scoped to KV leaders, since that is the only replica type currently supported. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
