alluxio icon indicating copy to clipboard operation
alluxio copied to clipboard

Published 20 hours ago • verified publisher icon Alluxio

Remove addquorumrequest

Open jenoudet opened this issue 2 years ago • 5 comments

jenoudet avatar Aug 15 '22 19:08 jenoudet

Hi @jenoudet, Can you provide details on why the add quorum request needs to be deleted.

qian0817 avatar Aug 16 '22 08:08 qian0817

Hi @jenoudet, Can you provide details on why the add quorum request needs to be deleted.

HI @qian0817, after doing some research I've concluded that calling RaftClient#admin().setConfiguration(...) should be sufficient to adding a new member to an existing cluster. addQuorumRequest sends an rpc to all members of the quorum who all individually call RaftServer#setConfiguration(...). This seems redundant and I was hoping to refactor it away.

jenoudet avatar Aug 16 '22 19:08 jenoudet

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Feb 01 '23 15:02 github-actions[bot]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Jun 16 '23 15:06 github-actions[bot]