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

Storage engine's updateWithDamages function should include the old record

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor - P4
    • Resolution: Fixed
    • Affects Version/s: 2.7.7
    • Fix Version/s: 2.7.8
    • Component/s: Storage
    • Labels:

      Description

      The current API for updateWithDamages does not include the record to be modified, so the engine has to re-read it in order to apply the damages locally and write the new value. The read is going to be in memory because the caller had to read it first, but it's still a non-trivial amount of CPU work.

      We should just pass down a RecordData constructed from the base pointer and objsize of the target BSONObj. Unit tests can be modified to fit this API without a lot of churn.

        Attachments

          Issue Links

            Activity

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: