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

Add log messages to CMAP spec

    • 3
    • 2
    • Hide

      DRIVERS-1675:
      This ticket depends on the driver having implemented the changes from DRIVERS-1677.

      Log messages and corresponding tests for connection pool logging were added via this commit.

      To summarize:

      • New log messages have been added which are very similar to existing CMAP events.
      • Tests in the unified format have been added in a /logging directory under tests.
      • A description of a close operation for client entities has been added to the unified test format. This operation is used in one of the new tests.
      • Existing CMAP tests in the CMAP-specific format were moved into a /cmap-format subdirectory under the /tests directory.
      Show
      DRIVERS-1675 : This ticket depends on the driver having implemented the changes from DRIVERS-1677 . Log messages and corresponding tests for connection pool logging were added via this commit . To summarize: New log messages have been added which are very similar to existing CMAP events. Tests in the unified format have been added in a /logging directory under tests. A description of a close operation for client entities has been added to the unified test format. This operation is used in one of the new tests. Existing CMAP tests in the CMAP-specific format were moved into a /cmap-format subdirectory under the /tests directory.
    • 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?

      This ticket was split from DRIVERS-1675, please see that ticket for a detailed description.

      Acceptance Criteria

      Implementation Requirements

      • Implement logging of the following events according to the CMAP spec and determine the appropriate logging level by conferring with the team
        • Pool Created
        • Pool Ready
        • Pool Cleared
        • Pool Closed
        • Connection Created
        • Connection Ready
        • Connection Closed
        • Connection Checkout Started
        • Connection Checkout Failed
        • Connection Checked Out
        • Connection Checked In
      • Implement close operation for client entities in Unified Test Runner
      •  Fix duplicate CMAP logging test description (NODE-5212)

      Testing Requirements

      • Pull in and run new CMAP logging spec tests
      • Pull in and run relocated CMAP tests

            Assignee:
            warren.james@mongodb.com Warren James
            Reporter:
            dbeng-pm-bot PM Bot
            Neal Beeken
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: