Skip to content
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

Testing RATIS-2162 #1154 over release-3.1.1. #1157

Open
wants to merge 1 commit into
base: release-3.1.1
Choose a base branch
from

Conversation

szetszwo
Copy link
Contributor

For testing #1154 over release-3.1.1.

@szetszwo szetszwo changed the title RATIS-2162. When closing leaderState, if the logAppender thread sends a snapshot, a deadlock may occur. () For testing RATIS-2162 #1154 over release-3.1.1. Sep 29, 2024
@szetszwo szetszwo changed the title For testing RATIS-2162 #1154 over release-3.1.1. Testing RATIS-2162 #1154 over release-3.1.1. Sep 29, 2024
@szetszwo
Copy link
Contributor Author

@133tosakarin , misc unit test also failed here. Please take a look. Thanks.

@133tosakarin
Copy link
Contributor

@133tosakarin , misc unit test also failed here. Please take a look. Thanks.

@133tosakarin , misc unit test also failed here. Please take a look. Thanks.

Sze, thank you for helping to test, I will continue to investigate RATIS-2162.

@133tosakarin
Copy link
Contributor

133tosakarin commented Sep 30, 2024

@szetszwo

image

I found that the failure of mics occurred in simulateRpc because logAppenderDaemon s9 was blocked waiting for rpc reply.

This problem seems to be unstable recurrence, and I have tried many times locally without triggering it.

@szetszwo
Copy link
Contributor Author

Let's rerun it here to see if it can be reproduced.

@133tosakarin
Copy link
Contributor

Let's rerun it here to see if it can be reproduced.

Sze, thank you for continuing to follow this PR, misc has passed.

In addition, I found that the reason why logAppender s9 continues to block is that this FileChannel cannot be created. Other logAppenders that are working normally have already created FileChannels.

image

@szetszwo
Copy link
Contributor Author

Let's merge #1154 then.

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