Add logic to fetch persistent sample on the test-only sequential scan/strides paths

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Query Optimization
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      All existing passthrough suites that use samplingCE also enable internalQuerySamplingBySequentialScan, which is necessary to make it possible to predict the planning outcome when sampling is used. We need to support persistent samples on this path (generateSampleBySeqScanningForTesting()) as well so that the existing tests can be used in conjunction with persistent sampling to provide more test coverage.

      This will require some investigation of how `analyze` interacts with the internalQuerySamplingBySequentialScan knob (not totally sure what the _id of the generated sample doc will look like). Tests for this should be added in analyze_sample_persist.js and cbr_persistent_sample.js.

      This work should also include doing the same for the new strides sampling mode. 

            Assignee:
            Natalie Hill
            Reporter:
            Natalie Hill
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: