Skip to content

fix joinGroup and update CI#547

Merged
cipepser merged 10 commits intomasterfrom
fix/joinGroup
Mar 29, 2021
Merged

fix joinGroup and update CI#547
cipepser merged 10 commits intomasterfrom
fix/joinGroup

Conversation

@cipepser
Copy link
Copy Markdown
Collaborator

@cipepser cipepser commented Mar 28, 2021

  • joinGroup、handshake時に全state-runtimeの_groupKeyCounterを更新するよう修正
  • state-runtime1/2両方からの更新/状態取得のテストをCIに追加

@cipepser cipepser marked this pull request as ready for review March 29, 2021 01:27
handleReport(_report, _reportSig);
// It is assumed that the nodes participate in the order of roster index,
// and all the nodes finish participating before the state transition.
for (int i = 0; i < _rosterIdx; i++) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uint32 i = 0 で回せば 毎回 uint32(i) で丸める必要なさそうですね

);

_groupKeyCounter[_rosterIdx] = GroupKeyCounter(_generation, _epoch);
for (int i = 0; i < _rosterIdx + 1; i++) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここもっかな

@cipepser cipepser merged commit 8b7be07 into master Mar 29, 2021
@cipepser cipepser deleted the fix/joinGroup branch March 29, 2021 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants