Apache9 commented on PR #4979:
URL: https://github.com/apache/hbase/pull/4979#issuecomment-1498851807

   > 
   
   Checked the API description
   
   
https://github.com/kubernetes-client/java/blob/master/kubernetes/docs/CoreV1Api.md#deleteNamespacedPod
   
   The deleteNamespacedPod method has a gracePeriodSeconds, 0 means delete 
immediately, so I think it could archive what we want.
   
   But what I concern more is about how to correctly support stop and kill, as 
in K8s, if you do not shift the replica count, the framework will launch a new 
pod right after you delete a pod...
   
   I think this is exactly what we want, but seems still not fully implemented 
yet...
   https://github.com/kubernetes/kubernetes/issues/45509
   
   And we also need to change some semantics for the cluster manager. For 
example, on K8s, it is useless to specify a hostname when starting a new region 
server, so maybe we could change the API to "startNewRegionServer", as even for 
non k8s environment, I do not think we must start a region server on a given 
host, we just need to start a new one, right?
   
   And for stop, kill, restart, maybe we could also change the semantice so it 
would fit both k8s and non k8s environment. For example, we just remove stop 
and kill, only leave restart there, but we provide a flag to indicate how to 
stop the region server, i.e, a graceful shutdown, or a force kill. And we 
provide another api called reduceRegionServerNumber. For K8s environment, it is 
just a API call, and for non k8s environment, we can randomly select a region 
server to stop. This is not perfect but I think it could fit most of our test 
scenarios.
   
   What do you guys think?
   
   Thanks.


-- 
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]

Reply via email to