Uploaded image for project: 'Node.js Driver'
  1. Node.js Driver
  2. NODE-6245

Restore keepAliveInitialDelay configurability

    • Type: Icon: Improvement Improvement
    • Resolution: Unresolved
    • Priority: Icon: Critical - P2 Critical - P2
    • None
    • Affects Version/s: None
    • Component/s: None
    • 2
    • 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

      These were removed in 6.0 with NODE-5190, however it's unclear what the value of this removal was.

      Let's keep the defaults that have been set but restore the ability to configure keepAliveInitialDelay (note that keepAlive should still not be toggleable)

      Use Case

      As a... driver user
      I want... to be able to configure keepAliveInitialDelay
      So that... I can choose a setting that works for my application

      User Experience

      • Use is able to set keepAliveInitialDelay to a value of their choice.

      Dependencies

      • N/A

      Risks/Unknowns

      • What could go wrong while implementing this change? (e.g., performance, inadvertent behavioral changes in adjacent functionality, existing tech debt, etc)
      • Is there an opportunity for better cross-driver alignment or testing in this area?
      • Is there an opportunity to improve existing documentation on this subject?

      Acceptance Criteria

      Implementation Requirements

      • Ensure users can configure the keepAliveInitialDelay option via client (connection string and constructor), allow it to be a pass-through (no validation)

      Testing Requirements

      • Test the pass-through with valid and invalid values (negative numbers, non-numeric types)

      Documentation Requirements

      • API docs
      • Add manual docs for the new option

      Follow Up Requirements

      • File ticket in V7 to change default keepAliveInitialDelay to 120s

            Assignee:
            Unassigned Unassigned
            Reporter:
            alex.bevilacqua@mongodb.com Alex Bevilacqua
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              None
              None
              None
              None