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

InsertStatement's recordId might be used only for testing - name it as such

XMLWordPrintableJSON

    • Type: Icon: Task Task
    • Resolution: Unresolved
    • Priority: Icon: Major - P3 Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Storage Execution

      This comment suggests that InsertStatement's recordId field is only used in a testing context. However the ticket that introduced it, SERVER-66283, uses it a lot so it's not really clear if the comment is true or not.

      If true, we should rename the field. If not, we should get rid of the comment.

      Finally, consider de-duplicating that field and replRid.

            Assignee:
            backlog-server-execution [DO NOT USE] Backlog - Storage Execution Team
            Reporter:
            vishnu.kaushik@mongodb.com Vishnu Kaushik
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: