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

Add log messages to SDAM spec

    • Type: Icon: New Feature New Feature
    • Resolution: Fixed
    • Priority: Icon: Major - P3 Major - P3
    • 6.4.0
    • Affects Version/s: None
    • Component/s: Logging
    • 2
    • 0
    • Hide

      DRIVERS-1670:
      Log messages and corresponding tests for server selection logging were added via this commit.

      In summary, drivers will need to:

      • Implement the log messages defined here
      • Run the logging-* tests defined here
      Show
      DRIVERS-1670 : Log messages and corresponding tests for server selection logging were added via this  commit . In summary, drivers will need to: Implement the log messages defined  here Run the logging-* tests defined  here
    • Not Needed
    • 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?

      Description

      Implement SDAM logging per spec guidelines (except those affected by DRIVERS-2711.

      Use Case

      As a... Node user
      I want... to log SDAM events when logging >= debug level
      So that... I can see which SDAM events are taking place in my code paths 

      User Impact

      • None - Logging will not be released until NODE-5672

      Unknowns

      • None

      Acceptance Criteria

      Implementation Requirements

      Testing Requirements

      • Ensure driver passes relevant unified spec tests
        • Except tests affected by DRIVERS-2711, will be run separately, (since the Node driver's current behavior is no TopologyDescriptionChangeEvent publishing on Topology.CLOSE, which contradicts certain spec tests)
      • Unit Tests TBD - will see if needed as implemented

      Documentation Requirements

      • N/A

            Assignee:
            aditi.khare@mongodb.com Aditi Khare
            Reporter:
            dbeng-pm-bot PM Bot
            Neal Beeken
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: