Remove BulkWriteShimOperation and BulkWriteOperation

XMLWordPrintableJSON

    • Hide

      Create a copy of the Kickoff Template with the issue key (NODE-XXX) in the filename and share a link to the new doc via this field.

      Show
      Create a copy of the Kickoff Template with the issue key (NODE-XXX) in the filename and share a link to the new doc via this field.
    • Not Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?
    • None
    • None
    • None
    • None
    • None
    • None

      Use Case

      As a Node engineer,
      I want to remove the BulkWrite and BulkWriteShimOperation classes in favor of executing bulk operation batches directly,
      So that we can remove unnecessary server selections from bulk API execution path.

      User Experience

      • n/a - internal refactor.

      Dependencies

      • n/a

      Risks/Unknowns

      • n/a

      Acceptance Criteria

      Implementation Requirements

      • Instead of encapsulating bulk API execution logic inside of BulkWrite.execute() and BulkWriteShimOperation.execute() methods, move the bulk execution logic out of operation classes.

      Testing Requirements

      • All existing tests pass.
      • Bulk APIs are a critical performance path - ensure no performance degradation occurs after the refactor.

      Documentation Requirements

      • n/a

      Follow Up Requirements

      • n/a

          1.
          		 Validation NODE-7072 Sub-task Closed Unassigned 6.19.0

              Assignee:
              Bailey Pearson
              Reporter:
              Bailey Pearson
              Durran Jordan
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: