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

XMLWordPrintableJSON

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

      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:
              [DO NOT USE] Backlog - Storage Execution Team
              Reporter:
              Vishnu Kaushik
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated: