Skip to content

fix: FBS::releaseRef w/o deletion on replica #747

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dorjesinpo
Copy link
Collaborator

When purging App, replica must decrement refCount but should not attempt to write deletion record.
Calls leading to releaseRef now have asPrimary as an argument.
An alternative would be to use FileStore::d_isPrimary but it is interesting to trace all calls explicitly as primary or not.

@dorjesinpo dorjesinpo requested a review from a team as a code owner May 14, 2025 21:46
@dorjesinpo dorjesinpo added the bug Something isn't working label May 14, 2025
@dorjesinpo dorjesinpo changed the title FBS::releaseRef w/o deletion on replica [fix] FBS::releaseRef w/o deletion on replica May 14, 2025
@dorjesinpo dorjesinpo force-pushed the fix/releaseRef-asReplica branch from af5813f to 5c96845 Compare May 14, 2025 21:49
@dorjesinpo dorjesinpo changed the title [fix] FBS::releaseRef w/o deletion on replica fix: FBS::releaseRef w/o deletion on replica May 14, 2025
@kaikulimu kaikulimu self-assigned this May 15, 2025
@kaikulimu kaikulimu self-requested a review May 15, 2025 15:28
Copy link

@bmq-oss-ci bmq-oss-ci bot left a comment

Choose a reason for hiding this comment

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

Build 2662 of commit 5c96845 has completed with SUCCESS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants