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

          Activity

          Hide
          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
          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
          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
          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
          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:
                1 year, 39 weeks ago
                Date of 1st Reply: