Uploaded image for project: 'Go Driver'
  1. Go Driver
  2. GODRIVER-3096

Remove "stats" package dependency

    • Type: Icon: Improvement Improvement
    • Resolution: Fixed
    • Priority: Icon: Unknown Unknown
    • 2.0.0
    • Affects Version/s: None
    • Component/s: None
    • None
    • 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?

      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

            Assignee:
            timothy.kim@mongodb.com Timothy Kim (Inactive)
            Reporter:
            matt.dale@mongodb.com Matt Dale
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: