Instrument the existing DisAgg tests with verification

    • Type: Task
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Verify
    • None
    • Storage Engines, Storage Engines - Foundations, Storage Engines - Persistence
    • None
    • 8

      During the DisAgg implementation, the team added many new tests (most of them named test_layered_*). However, as far as I know, none of these tests call verify, since for a long time the verification command simply returned ENOTSUP in DisAgg mode.

      The main goal of these tickets is to review all newly added DisAgg tests and add verify calls in places where it would be meaningful. This work shouldn’t be blocked by the MPV implementation, since we now have logic in place that reports successful verification for all DisAgg tables.

      Both DisAgg and verification testing could benefit from this, since tests would receive extra checks and verify would receive extra implicit testing.

      Please check the corresponding document section for more details. 

            Assignee:
            [DO NOT USE] Backlog - Storage Engines Team
            Reporter:
            Ivan Kochin
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: