Skip to content

Avoid unnecessary shard updates while shutting down#1342

Merged
nvollmar merged 2 commits intomainfrom
fix-unhandled-shardsupdated
Jun 4, 2024
Merged

Avoid unnecessary shard updates while shutting down#1342
nvollmar merged 2 commits intomainfrom
fix-unhandled-shardsupdated

Conversation

@nvollmar
Copy link
Contributor

Refs #1341

@pjfanning
Copy link
Member

looks sensible to me - let's see what other reviewers say

When we get a fix in, we might want to backport it to 1.0.x. I think we should make a 1.0.3 release in the next few weeks.

@nvollmar
Copy link
Contributor Author

Would be good to have that in 1.0.3

@nvollmar nvollmar requested review from He-Pin, mdedetrich and raboof May 24, 2024 12:26
@pjfanning pjfanning added this to the 1.1.0-M2 milestone May 24, 2024
Copy link
Member

@samueleresca samueleresca left a comment

Choose a reason for hiding this comment

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

looks good to me.

Copy link
Member

@He-Pin He-Pin left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@Roiocam Roiocam left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks.

ShardUpdate will only notify Shard to start detecting the passivation. This is seems like a harmless change.

@nvollmar nvollmar merged commit 92b38e7 into main Jun 4, 2024
@nvollmar nvollmar deleted the fix-unhandled-shardsupdated branch June 4, 2024 05:41
nvollmar added a commit that referenced this pull request Jun 4, 2024
* Avoid unnecessary shard updates while shutting down

* Avoid re-creating the same message for each shard
nvollmar added a commit that referenced this pull request Jun 4, 2024
* Avoid unnecessary shard updates while shutting down

* Avoid re-creating the same message for each shard
@pjfanning pjfanning added the release notes Need to release note label Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release notes Need to release note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants