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

Database monitoring may crash the server or badly affect its page locking logic [CORE1881] #2313

Closed
firebird-automations opened this issue May 5, 2008 · 7 comments

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: @dyemanov

The monitoring code unconditionally fetches the header page to report the actual database state. But, if being done at the AST level, this may cause two concurrent pending requests being linked to a single owner. The lock manager highly dislikes this situation and terminates the process via a bugcheck.

It can be reproduced under high load with the SuperClassic architecture. So far I was unable to see this issue on v2.1 Classic, maybe it could be explained by different synchronization techniques used. But anyway, I mark v2.1.0 as possibly affected and the fix will be committed into both branches.

Commits: ce7baa7 d55e7eb

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

assignee: Dmitry Yemanov [ dimitr ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

status: Open [ 1 ] => Open [ 1 ]

Target: 2.5 Alpha 1, 2.1.1 [ 10224, 10223 ]

Planning Status: Unspecified => Considered for inclusion

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

Fix Version: 2.5 Alpha 1 [ 10224 ]

Fix Version: 2.1.1 [ 10223 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

status: Open [ 1 ] => Open [ 1 ]

Planning Status: Considered for inclusion => Included to release

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pcisar

status: Resolved [ 5 ] => Closed [ 6 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

QA Status: No test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment