[GODRIVER-3096] Remove "stats" package dependency Created: 12/Jan/24  Updated: 22/Jan/24

Status: Backlog
Project: Go Driver
Component/s: None
Affects Version/s: None
Fix Version/s: 2.1.0

Type: Improvement Priority: Unknown
Reporter: Matt Dale Assignee: Unassigned
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
depends on GODRIVER-2762 Use minimum RTT for CSOT maxTimeMS ca... Closed
Related
is related to GODRIVER-2762 Use minimum RTT for CSOT maxTimeMS ca... Closed
Documentation Changes Summary:

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?


 Description   

Context

PR 1507 (tentatively) removes the need to calculate 90th percentile and standard deviation for RTT samples. Those are the last two "production" use cases for the "github.com/montanaflynn/stats" package in the Go driver. We should remove all other uses of the "stats" package so we can remove it from the Go driver dependencies.

Definition of done

  • Remove the "github.com/montanaflynn/stats" dependency from the Go driver module.
  • Make the benchmark system its own submodule so it can pull in additional dependencies (similar to the awslambda/mongodb module).
  • Optional: Consider moving the internal/benchmark code into internal/cmd/benchmark since that's the only place it's used.

Pitfalls


Generated at Thu Feb 08 08:40:03 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.