Improvement `BdbGroupAdmin` class
aloyszhang opened this issue · comments
AloysZhang commented
BdbGroupAdmin
is used to remove node in replication group.
Currently, BdbGroupAdmin
does't handle the exception and missing the necessary message about the remove process. I think we should:
- print the node information before and after remove node
- catch and handler the exception separately, then print some readable information
What do you think @gosonzhang ?
Goson Zhang commented
Very good advice, can you contribute a PR?
AloysZhang commented
@gosonzhang I have open a PR here