BSON performance benchmarking should test the current PR

XMLWordPrintableJSON

    • 2
    • 1
    • Not Needed
    • None
    • Hide

      Create a copy of the Kickoff Template with the issue key (NODE-XXX) in the filename and share a link to the new doc via this field.

      Show
      Create a copy of the Kickoff Template with the issue key (NODE-XXX) in the filename and share a link to the new doc via this field.
    • 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?
    • None
    • None
    • None
    • None
    • None
    • None

      Use Case

      As a Node.js engineer
      I want confirm if my changes will cause a performance change
      So that I can adapt and modify code before merging

      User Impact

      • None

      Dependencies

      • BSON CI

      Unknowns

      • How do we fix the fact that the test names include the branch "main" the name needs to remain the same in order to get history.

      Acceptance Criteria

      Implementation Requirements

      • Update the dbx-js-tools to crash the parent process when the child process crashes (i.e., if the tests throw an error, we should exit(1) )
      • Make the default bson perf test logic uses the current branch

      Testing Requirements

      • For each of the 3 perf test tasks, confirm that the local run and the evergreen patch run use the correct branch (the current branch)

      Documentation Requirements

      • Update the test readme instructions

      Follow Up Requirements

      • None

            Assignee:
            Durran Jordan
            Reporter:
            Neal Beeken
            None
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: