-
Type: Sub-task
-
Resolution: Fixed
-
Priority: Major - P3
-
Affects Version/s: None
-
Component/s: None
Use Case
As a Node.js engineer
I want to improve the performance of the socketWrite function on our Connection class
So that the driver continues to operate with the same performance prior to the async connection layer change.
User Impact
- None
Dependencies
- None
Unknowns
- Should we remove signal use from the connection layer?
- No, I think that can be something we consider post-CSOT, this change will avoid the issues with accessing signal repeatedly
Acceptance Criteria
Implementation Requirements
- Store the signal on the Connection class and update relevant properties to readonly
- Consolidate all signal access to a throwIfAborted helper
- Create and store aborted promise for socketWrite on the Connection class (avoids making a new signal handling promise every write)
- Remove unused abortable helper
Testing Requirements
- assert addEventListener is only called once per command (in onData)
Documentation Requirements
- None
Follow Up Requirements
- None