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

Deprecate "bsonrw.Copier" type and all associated functions

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

      The bsonrw.Copier type is an empty struct that exposes a collection of functions related to copying BSON or Extended JSON documents. Most of the functions are only used in the BSON library internally. Only the CopyValueToBytes function has known uses outside of the bson package (in the mgocompat package and in ADL code). It's possible that function can be replaced by other functionality already in the bson package. If that's not the case, add a package function called CopyValueToBytes that replaces Copier.CopyValueToBytes.

      Definition of done:

      • Deprecate the bsonrw.Copier type and all functions on that type.
      • Deprecate the bsonrw.CopyDocument function.
      • Investigate whether CopyValueToBytes can be replaced by bson.MarshalValue. If so, suggest using that function. If not, add a package function called CopyValueToBytes and suggest using that function instead.
      • Investigate deprecating the bsonrw.BytesWriter and bsonrw.BytesReader interfaces, which may not be necessary if bsonrw.Copier is fully deprecated.

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

              Created:
              Updated:
              Resolved: