-
Type: Task
-
Resolution: Fixed
-
Priority: Minor - P4
-
Affects Version/s: None
-
Component/s: None
Use Case
As a driver engineer
I want to reduce the tech debt in executeOperation supporting callbacks
So that modifying executeOperation does not require maintaining a legacy use case
User Impact
- None, callbacks are not available to the public API, this is an internal change
Dependencies
- Parts of the driver that use the callback form:
- Bulk operations, common.ts 3 usages insert/update/delete
- endTransaction, sessions.ts 2 usages, runs command and retries if a certain error is returned
Unknowns
- None
Acceptance Criteria
Implementation Requirements
- Convert usages in "Dependencies" to .then/.catch
- remove executeOperation and maybeCallback
- rename executeOperationAsync to executeOperation and export it
Testing Requirements
- All existing tests should pass
- Remove unit tests for maybeCallback
Documentation Requirements
- None
Follow Up Requirements
- None
- related to
-
NODE-4996 maybeCallback rejects with an unhandledRejection instead of uncaught exceptions
- Closed