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

Merge the "gridfs" package into the "mongo" package

XMLWordPrintableJSON

    • Type: Icon: Improvement Improvement
    • Resolution: Fixed
    • Priority: Icon: Unknown Unknown
    • 2.0.0
    • Affects Version/s: None
    • Component/s: GridFS
    • Labels:
      None
    • Major Change
    • Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      There is no longer a gridfs package in v2, the gridfs methods were moved to the mongo package. So all examples for gridfs will need to be updated to be sourced from mongo.
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      NA
      3. Which versions of the driver/connector does this apply to?
      v2

      Show
      1. What would you like to communicate to the user about this feature? There is no longer a gridfs package in v2, the gridfs methods were moved to the mongo package. So all examples for gridfs will need to be updated to be sourced from mongo. 2. Would you like the user to see examples of the syntax and/or executable code and its output? NA 3. Which versions of the driver/connector does this apply to? v2

      Keeping the GridFS API in a separate package deviates from the existing pattern of having all MongoDB API operations in the mongo package and creates visibility issues for code in the gridfs package that needs to access configurations from a mongo.Client. Merge the gridfs package into the mongo package to resolve those issues.

      Definition of done:

      • Merge all code from the gridfs package into the mongo package.
      • Rename type gridfs.Bucket to mongo.GridFSBucket
      • Add a function to *mongo.Database that creates a *mongo.GridFSBucket for that database (e.g. .Bucket() or .GridFS()).
      • (Optional) Consider removing the Client.Timeout function, which was added to allow accessing the timeoutMS value from the gridfs package.

            Assignee:
            preston.vasquez@mongodb.com Preston Vasquez
            Reporter:
            matt.dale@mongodb.com Matt Dale
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: