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

Improve perf of large document logging

    • Type: Icon: Task Task
    • Resolution: Unresolved
    • Priority: Icon: Unknown Unknown
    • None
    • Affects Version/s: None
    • Component/s: 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?

      Spreadsheet: Large BSON Serialization causes degradation

      Use Case

      As a.. Node Driver user
      I want... Logging to not slow down my application by 100x in certain cases
      So that... My application is efficient even when logging is turned on.

      Unknowns

      If the problem is converting BSON to JSON that will end up being truncated, is this something that On-Demand BSON could help with? E.g. implementing a reviver that only generates a string up to the point it would have been truncated.

      Acceptance Criteria

      Implementation Requirements

      Testing Requirements

      • Run perf on logging worst case and ensure all test cases are <1.2x as slow as base
      • Cases:
        • Worst case: trace on everything
        • Support case: debug (same as trace since we don't have trace logs)
        • Production use case: error or warn level logs (we don't have any of these logs right now, but we can run the setting just as a sanity check)

            Assignee:
            Unassigned Unassigned
            Reporter:
            aditi.khare@mongodb.com Aditi Khare
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: