Uploaded image for project: 'PHP Driver: Extension'
  1. PHP Driver: Extension
  2. PHPC-2237

Add logging to command monitoring

XMLWordPrintableJSON

    • Type: Icon: New Feature New Feature
    • Resolution: Unresolved
    • Priority: Icon: Major - P3 Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Hide

      DRIVERS-1673:
      This ticket depends on the driver having implemented the changes from DRIVERS-1677. It is suggested that drivers do this ticket in tandem with DRIVERS-1677 so that they have a first set of log messages and tests to try out with their logging implementation and unified test runner changes.

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

      In summary, drivers will need to:

      • Implement the log messages defined here
      • Run the unified format tests defined here
      • Implement the prose tests defined here

      Also, note that the spec was renamed "Command Logging and Monitoring" (we'll see if the acronym CLAM catches on!) and that the spec directory/RST file were renamed to command-logging-and-monitoring, so any scripts that rely on the name to e.g. sync tests will need to be updated.

      Show
      DRIVERS-1673 : This ticket depends on the driver having implemented the changes from DRIVERS-1677 . It is suggested that drivers do this ticket in tandem with DRIVERS-1677 so that they have a first set of log messages and tests to try out with their logging implementation and unified test runner changes. Log messages and corresponding tests for command logging were added via this commit . In summary, drivers will need to: Implement the log messages defined here Run the unified format tests defined here Implement the prose tests defined here Also, note that the spec was renamed "Command Logging and Monitoring" (we'll see if the acronym CLAM catches on!) and that the spec directory/RST file were renamed to command-logging-and-monitoring, so any scripts that rely on the name to e.g. sync tests will need to be updated.

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

            Assignee:
            Unassigned Unassigned
            Reporter:
            dbeng-pm-bot PM Bot
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: