Remove IndexBuilder in favor of IndexBuildsCoordinator

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Fixed
    • Priority: Major - P3
    • 4.3.1
    • Affects Version/s: None
    • Component/s: Storage
    • None
    • Fully Compatible
    • Execution Team 2019-12-02
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      We still use the IndexBuilder to build indexes in the foreground thread in applyOps. This is currently only used when creating indexes with applyOps on a primary or when rebuilding indexes during replication recovery.

      We have too many duplicated implementations of index building and removing this class entirely would help to prevent diverging logic.

      The functionality can be entirely replaced by using the IndexBuildsCoordinator and waiting for the future, similar to what we do in create_indexes.cpp.

              Assignee:
              Benety Goh
              Reporter:
              Louis Williams
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: