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

ProgressMeter for closeAllFiles uses arguments incorrectly

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial - P5
    • Resolution: Fixed
    • Affects Version/s: 2.4.5
    • Fix Version/s: 2.5.3
    • Component/s: Internal Code
    • Labels:
    • Backwards Compatibility:
      Fully Compatible
    • Operating System:
      ALL

      Description

      The constructor for ProgressMeter takes two optional string parameters: "units" and "name". "name" is used as the prefix for logged progress reports (default is "Progress") and "units" is displayed in parentheses following the progress percentage if non-empty.

      The constructor for ProgressMeter in MongoFile::closeAllFiles() (mmap.cpp) passes "File Closing Progress" in the "units" argument instead of a more appropriate "files" and does not pass an argument for "name".

              ProgressMeter pm(mmfiles.size(), 2, 1, "File Closing Progress");

      should be

              ProgressMeter pm(mmfiles.size(), 2, 1, "files", "File Closing Progress");

        Attachments

          Activity

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: