Skip to content

Conversation

@nbbeeken
Copy link
Contributor

Description

What is changing?

  • Currently we only have the cumulative parallel benchmarks reporting, This adds the individual tests to the report
Is there new documentation needed for these changes?

What is the motivation for this change?

  • I want to see read vs write parallel performance

Double check the following

  • Ran npm run check:lint script
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: type(NODE-xxxx)[!]: description
    • Example: feat(NODE-1234)!: rewriting everything in coffeescript
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@W-A-James W-A-James changed the title chore: add reporting for individual parallel benchmarks chore(NOED-5794): add reporting for individual parallel benchmarks Apr 19, 2024
@W-A-James W-A-James changed the title chore(NOED-5794): add reporting for individual parallel benchmarks chore(NODE-5794): add reporting for individual parallel benchmarks Apr 19, 2024
@W-A-James W-A-James self-assigned this Apr 19, 2024
@W-A-James W-A-James added the Primary Review In Review with primary reviewer, not yet ready for team's eyes label Apr 19, 2024
@W-A-James W-A-James self-requested a review April 19, 2024 18:00
@W-A-James
Copy link
Contributor

Verified that the following benchmarks are now reported individually

  • LDJSON multi-file import
  • LDJSON multi-file export
  • GridFS multi-file upload
  • GridFS multi-file download

@W-A-James W-A-James merged commit eece8c1 into main Apr 19, 2024
@W-A-James W-A-James deleted the chore-perf-parallel branch April 19, 2024 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Primary Review In Review with primary reviewer, not yet ready for team's eyes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants