Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-24096

Consider making MultikeyPaths a class

    XMLWordPrintableJSON

Details

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Major - P3 Major - P3
    • None
    • None
    • Internal Code, Querying
    • None
    • Query Optimization

    Description

      Based on codereview feedback during SERVER-22726, having a class to represent the cross product of "the index is multikey" and "the index has path-level multikey information" would improve the readability of the code in plan_enumerator.cpp, index_catalog_entry.cpp, etc. It would be possible to then fold the currently separate code paths for when the index is multikey but doesn't have path-level info and when the index is multikey and has path-level info.

      https://github.com/mongodb/mongo/blob/r3.3.5/src/mongo/db/index/multikey_paths.h#L37-L43

      Attachments

        Activity

          People

            backlog-query-optimization Backlog - Query Optimization
            max.hirschhorn@mongodb.com Max Hirschhorn
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated: