Uploaded image for project: 'WiredTiger'
  1. WiredTiger
  2. WT-1978

WT_CURSOR.compare for index cursors does insufficient checking

        Details

        • Type: Bug
        • Status: Closed
        • Priority: Minor - P4
        • Resolution: Fixed
        • Affects Version/s: WT2.6.1
        • Fix Version/s: WT2.7.0
        • Labels:
          None
        • # Replies:
          5
        • Last comment by Customer:
          true

          Description

          Two different indices from a table can be used with the compare and equal functions and they won't necessarily give an error. It's possible that if one had a collator, then bad things would happen when the collator is applied to an index that it wasn't designed to handle.

          There are no tests for compare/equal of indices.

            Issue Links

            is depended on by

            Bug - A problem which impairs or prevents the functions of the product. SERVER-19079 WiredTiger changes for MongoDB 3.1.5

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

              Activity

              • Comments
              Ascending order - Click to sort in descending order
              Hide
              Permalink
              xgen-internal-githook Githook User added a comment -

              Author:

              {u'username': u'ddanderson', u'name': u'Don Anderson', u'email': u'dda@ddanderson.com'}

              Message: WT-1978. Add tests for WT_CURSOR::compare and equals for cases when
              the cursor is on an index.
              Branch: develop
              https://github.com/wiredtiger/wiredtiger/commit/255ebaf45e6b6d1fa0356f6ed826e762bc2c1d09

              Show
              xgen-internal-githook Githook User added a comment - Author: {u'username': u'ddanderson', u'name': u'Don Anderson', u'email': u'dda@ddanderson.com'} Message: WT-1978 . Add tests for WT_CURSOR::compare and equals for cases when the cursor is on an index. Branch: develop https://github.com/wiredtiger/wiredtiger/commit/255ebaf45e6b6d1fa0356f6ed826e762bc2c1d09
              Hide
              Permalink
              xgen-internal-githook Githook User added a comment -

              Author:

              {u'username': u'ddanderson', u'name': u'Don Anderson', u'email': u'dda@ddanderson.com'}

              Message: WT-1978. Tighten up checking when comparing indices to make sure they
              are using the same underlying URL.
              Branch: develop
              https://github.com/wiredtiger/wiredtiger/commit/5c847d8f212d04fd9ae5334d8cc8b9f8395c4e17

              Show
              xgen-internal-githook Githook User added a comment - Author: {u'username': u'ddanderson', u'name': u'Don Anderson', u'email': u'dda@ddanderson.com'} Message: WT-1978 . Tighten up checking when comparing indices to make sure they are using the same underlying URL. Branch: develop https://github.com/wiredtiger/wiredtiger/commit/5c847d8f212d04fd9ae5334d8cc8b9f8395c4e17
              Hide
              Permalink
              xgen-internal-githook Githook User added a comment -

              Author:

              {u'username': u'ddanderson', u'name': u'Don Anderson', u'email': u'dda@ddanderson.com'}

              Message: WT-1978. Fixed call to complex_populate_index_name(), which changed
              in previous commit.
              Branch: develop
              https://github.com/wiredtiger/wiredtiger/commit/99796d0b9c92e3679d1cd861c8fe5c05c947e7db

              Show
              xgen-internal-githook Githook User added a comment - Author: {u'username': u'ddanderson', u'name': u'Don Anderson', u'email': u'dda@ddanderson.com'} Message: WT-1978 . Fixed call to complex_populate_index_name(), which changed in previous commit. Branch: develop https://github.com/wiredtiger/wiredtiger/commit/99796d0b9c92e3679d1cd861c8fe5c05c947e7db
              Hide
              Permalink
              xgen-internal-githook Githook User added a comment -

              Author:

              {u'username': u'ddanderson', u'name': u'Don Anderson', u'email': u'dda@ddanderson.com'}

              Message: WT-1978. Simplify the test for compatible index cursors.
              Branch: develop
              https://github.com/wiredtiger/wiredtiger/commit/36e3c0be0a4bc2eece654388ef6da86ecb63113d

              Show
              xgen-internal-githook Githook User added a comment - Author: {u'username': u'ddanderson', u'name': u'Don Anderson', u'email': u'dda@ddanderson.com'} Message: WT-1978 . Simplify the test for compatible index cursors. Branch: develop https://github.com/wiredtiger/wiredtiger/commit/36e3c0be0a4bc2eece654388ef6da86ecb63113d
              Hide
              Permalink
              xgen-internal-githook Githook User added a comment -

              Author:

              {u'username': u'ddanderson', u'name': u'Don Anderson', u'email': u'dda@mongodb.com'}

              Message: Merge pull request #2025 from wiredtiger/index-compare

              WT-1978 Better checking and tests for index cursor comparison
              Branch: develop
              https://github.com/wiredtiger/wiredtiger/commit/5eaf63e2cf781f272afa31f4ab9b425bd15a87ec

              Show
              xgen-internal-githook Githook User added a comment - Author: {u'username': u'ddanderson', u'name': u'Don Anderson', u'email': u'dda@mongodb.com'} Message: Merge pull request #2025 from wiredtiger/index-compare WT-1978 Better checking and tests for index cursor comparison Branch: develop https://github.com/wiredtiger/wiredtiger/commit/5eaf63e2cf781f272afa31f4ab9b425bd15a87ec

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Days since reply:
                    2 years, 6 days ago
                    Date of 1st Reply: