Improve BulkWrite organization remove BulkWriteShimOperation

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Duplicate
    • Priority: Minor - P4
    • None
    • Affects Version/s: None
    • Component/s: Operations Layer
    • Not Needed
    • None
    • 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 driver engineer
      I want a single path that operations take through the operation layer
      So that reasoning about the many features of an operation is easier.

      User Impact

      Bugginess related to bulkWrite due to it's "special" casing. It's logic doesn't follow the rest of our operation implementations.

      Dependencies

      • None

      Unknowns

      Acceptance Criteria

      • Refactor Bulk to not be an operation, the batching logic can be done to convert the many operations in to a list of normal AbstractOperation subclasses
      • Loop over the operations, standardized error handling, documented merging

      Implementation Requirements

      • Remove the BulkOperationShim class, it shouldn't be needed to execute a series of operations
      • Obtaining a shared session could be made easier

      Testing Requirements

      • Make sure to test against a sharded cluster (session)

      Documentation Requirements

      • None

      Follow Up Requirements

      • Move bulk related logic into operations folder

              Assignee:
              Unassigned
              Reporter:
              Neal Beeken
              None
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: