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

Storage engine's updateWithDamages function should include the old record

    • Type: Icon: Improvement Improvement
    • Resolution: Done
    • Priority: Icon: Minor - P4 Minor - P4
    • 2.7.8
    • Affects Version/s: 2.7.7
    • Component/s: Storage

      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.

            Assignee:
            Unassigned Unassigned
            Reporter:
            john.esmet@gmail.com John Esmet
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: