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
More detail in message "wrong page type" [CORE3588] #3942
Comments
Commented by: @aafemt If page type has unexpected value, who knows what else is damaged. If the page is completely filled with garbage, attempt to resolve value like 123 into page type will cause crash in absence of additional check. |
Commented by: Ann Harrison (awharrison) Yes, it would be wise to have a choice for "unknown page type." sorry I wasn't more specific. |
Modified by: @AlexPeshkoffassignee: Alexander Peshkov [ alexpeshkoff ] |
Modified by: @AlexPeshkoffstatus: Open [ 1 ] => Open [ 1 ] Target: 3.0 Beta 1 [ 10332 ] Implementation: [Trivial] |
Modified by: @AlexPeshkoffstatus: Open [ 1 ] => Resolved [ 5 ] resolution: Fixed [ 1 ] Fix Version: 3.0 Alpha 1 [ 10331 ] |
Modified by: @pcisarstatus: Resolved [ 5 ] => Closed [ 6 ] |
Modified by: @pavel-zotovQA Status: No test |
Submitted by: Ann Harrison (awharrison)
Twenty years ago, concise error messages made some sense, but diagnosing the wrong page type errors would be much easier if Firebird said "expected Index Page, encountered Data Page" rather than "expected <n> encountered <m>" - at least it would save me looking up the page types each time. Probably not worth much since those errors are mostly found in older versions, but still, if one realized that something that should have been an index page was something else, then there would be a clue that the workaround would be to rebuild the index. Hey, maybe even include the name of the index and table...
Commits: 64a89b7
The text was updated successfully, but these errors were encountered: