[SERVER-83951] Handle exceptions more gracefully when traversing indexes during validation Created: 16/Nov/23 Updated: 08/Dec/23 |
|
| Status: | Open |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Yuhong Zhang | Assignee: | Backlog - Storage Execution Team |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Assigned Teams: |
Storage Execution
|
| Participants: |
| Description |
|
Currently we fail the validate command right after getting the first exception when traversing through indexes during validation. It will be more helpful to record them as errors and keep validating other indexes. |
| Comments |
| Comment by Chenhao Qu [ 08/Dec/23 ] |
|
I made the code change and opened a PR. However, the PR lacks testing. I tried to write a test in validation_tests.cpp by directly inserting a corrupted index. But that doesn't build because dbtest is storage engine agnostic and it doesn't build with the wiredtiger.h header. We need to write a test for the change in somewhere else. |