Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-17953

Implement Driver CRUD API Spec in shell

        Details

        • Type: New Feature
        • Status: Closed
        • Priority: Major - P3
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 3.1.7
        • Component/s: Shell
        • Labels:
          None
        • Backwards Compatibility:
          Fully Compatible

          Description

          Application developers generally use two APIs: a driver and the shell. As one of my readers points out, this makes API conformance between the drivers and the shell at least as important as conformance among drivers.

          Let's implement the Driver CRUD API Spec in the shell: insertOne, insertMany, findOneAndDelete, etc. The old APIs can be deprecated, or not. The spec is written to avoid breaking backwards compatibility so existing mongo shell scripts and examples will keep working.

            Issue Links

            is documented by

            Improvement - An improvement or enhancement to an existing feature or task. DOCS-6010 Driver's CRUD API Spec implemented in the mongo shell

            • Major - P3 - Major loss of function.
            • Closed
            related to

            Improvement - An improvement or enhancement to an existing feature or task. SERVER-21273 Implement Driver CRUD API Spec for .explain()

            • Major - P3 - Major loss of function.
            • Open

            Improvement - An improvement or enhancement to an existing feature or task. SERVER-26642 JS compiled code string length limit on Windows

            • Major - P3 - Major loss of function.
            • Closed

              Activity

              • Comments
              Ascending order - Click to sort in descending order
              Hide
              Permalink
              jesse A. Jesse Jiryu Davis added a comment -

              Note the test suite for compliance:

              https://github.com/mongodb/specifications/tree/master/source/crud/tests

              This'll help validate the shell's API matches drivers'.

              Show
              jesse A. Jesse Jiryu Davis added a comment - Note the test suite for compliance: https://github.com/mongodb/specifications/tree/master/source/crud/tests This'll help validate the shell's API matches drivers'.
              Hide
              Permalink
              pasette Dan Pasette added a comment - - edited

              Assigning to Christian as discussed with @erlichson

              Show
              pasette Dan Pasette added a comment - - edited Assigning to Christian as discussed with @erlichson
              Hide
              Permalink
              xgen-internal-githook Githook User added a comment -

              Author:

              {u'username': u'christkv', u'name': u'Christian Kvalheim', u'email': u'christkv@gmail.com'}

              Message: SERVER-17953 Implement Driver CRUD API Spec in shell

              Closes #1004.

              Signed-off-by: Max Hirschhorn <max.hirschhorn@mongodb.com>

              • Moved CRUD API implementation into separate file to workaround 65K
                string length compiler limit on Windows.
              Show
              xgen-internal-githook Githook User added a comment - Author: {u'username': u'christkv', u'name': u'Christian Kvalheim', u'email': u'christkv@gmail.com'} Message: SERVER-17953 Implement Driver CRUD API Spec in shell Closes #1004. Signed-off-by: Max Hirschhorn <max.hirschhorn@mongodb.com> Moved CRUD API implementation into separate file to workaround 65K string length compiler limit on Windows. Fixed test to ignore modifiedCount property (i.e. nModified) when not using write commands. Branch: master https://github.com/mongodb/mongo/commit/8c8da71903a3325d4df19faaf2745f23bfbe7302

                People

                • Votes:
                  2 Vote for this issue
                  Watchers:
                  20 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: