Remove MultiIndexBlockImpl::insertAllDocumentsInCollection function parameter that is always a nullptr

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Fixed
    • Priority: Major - P3
    • 4.1.2
    • Affects Version/s: None
    • Component/s: Storage
    • Fully Compatible
    • Storage NYC 2018-08-13
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      No callers of MultiIndexBlockImpl::insertAllDocumentsInCollection(std::set<RecordId>* dupsOut = nullptr), except for an index unit test, pass in a function parameter. They all let the function parameter default to nullptr.

      I suspect that this field was outdated with the introduction of the 'unique' index build setting. A build will handle the issue of duplicates internally, succeeding or failing depending on settings that were passed into the index build request.

            Assignee:
            Audrey Fang (Inactive)
            Reporter:
            Dianna Hohensee (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: