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

Measure node driver TS compilation time

    • 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... user of the Node.js driver
      I want... the node driver to proactively monitor its TS compilation time
      So that... I don't experience excessive compilation times when upgrading to a new version of the driver

      User Impact

      • TS loading times affect how quickly users are able to build their applications for testing and deployment

      Dependencies

      • N/A

      Unknowns

      • What is the best way to get a consistent metric in CI? When measuring the TS compilation time, should we measure the absolute quantity or relative? Which version of TS should we build with?

      Acceptance Criteria

      Implementation Requirements

      • Create a performance test to measure TS compilation time of the driver inside a test application
      • Add a new task to the evergreen performance test variant
      • Target the dedicated perf test distro
      • Output the results into a json format and push it to the analytics backend using `perf.send`

      Testing Requirements

      • Confirm that the trend chart tab is present in the new CI task

      Documentation Requirements

      • N/A

      Follow Up Requirements

      • Consider whether it's worthwhile to independently test js-bson

            Assignee:
            Unassigned Unassigned
            Reporter:
            daria.pardue@mongodb.com Daria Pardue
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: