Uploaded image for project: 'Node.js Driver'
  1. Node.js Driver
  2. NODE-4283 Performance regression of parsing Array type fields
  3. NODE-5403

Create regression test to track bson string deserialization performance

    • Type: Icon: Sub-task Sub-task
    • Resolution: Fixed
    • Priority: Icon: Unknown Unknown
    • bson-6.0.0
    • Affects Version/s: None
    • Component/s: BSON
    • Labels:
    • 0
    • 2
    • 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?

      Use Case

      As a... driver engineer
      I want... to add a string deserialization performance test that runs in CI
      So that... the team will be alerted if the string deserialization code path's performance is negatively impacted by a future change.

      User Impact

      N/A

      Dependencies

      • N/A

      Unknowns

      • Would it be better to add these tests in the bson repo or in the driver repo?
        • We will add the new test to the bson repo for simplicity of this ticket

      Acceptance Criteria

      Implementation Requirements

      • Create a benchmark in js-bson repo that runs the existing string deserialization benchmark and report
      • Create an evergreen task that only runs the new test on an appropriate evergreen host and node version that reports results using perf.send
      • Do NOT design new benchmarks for this ticket

      Testing Requirements

      • Manually ensure that the tests are actually running in CI and that the test reports results successfully

      Documentation Requirements

      • N/A

      Follow Up Requirements

      • N/A

            Assignee:
            warren.james@mongodb.com Warren James
            Reporter:
            warren.james@mongodb.com Warren James
            Neal Beeken
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: