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

Allow read-only transactions to commit after failure

    Details

    • Type: Task
    • Status: Resolved
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: WT2.7.0
    • Labels:

      Description

      See SERVER-17652.

      This is arguably a case of WiredTiger being too strict. We track whether any operation has "failed" during a transaction, with a fairly broad definition of failure. If an operation fails, any attempt to call WT_SESSION::commit_transaction will cause a rollback and return an error.

      In the case of read-only transactions, this is a non-issue, because rollback and commit are equivalent. It seems like we should be able to relax the restriction and allow commit after failures in read-only transactions.

        Issue Links

          Activity

          Hide
          xgen-internal-githook Githook User added a comment -

          Author:

          {u'username': u'daveh86', u'name': u'daveh86', u'email': u'howsdav@gmail.com'}

          Message: WT-1845 - OPI Lint changes
          Branch: develop
          https://github.com/wiredtiger/wiredtiger/commit/28dd3d3797b57ea38e35d442fdf457732471085c

          Show
          xgen-internal-githook Githook User added a comment - Author: {u'username': u'daveh86', u'name': u'daveh86', u'email': u'howsdav@gmail.com'} Message: WT-1845 - OPI Lint changes Branch: develop https://github.com/wiredtiger/wiredtiger/commit/28dd3d3797b57ea38e35d442fdf457732471085c
          Hide
          xgen-internal-githook Githook User added a comment -

          Author:

          {u'username': u'daveh86', u'name': u'David Hows', u'email': u'howsdav@gmail.com'}

          Message: WT-1845 - Allow read only transactions to commit after failure
          Branch: develop
          https://github.com/wiredtiger/wiredtiger/commit/796a6e796d596b3f6b7f3d102dfdcf2e86737bc5

          Show
          xgen-internal-githook Githook User added a comment - Author: {u'username': u'daveh86', u'name': u'David Hows', u'email': u'howsdav@gmail.com'} Message: WT-1845 - Allow read only transactions to commit after failure Branch: develop https://github.com/wiredtiger/wiredtiger/commit/796a6e796d596b3f6b7f3d102dfdcf2e86737bc5
          Hide
          xgen-internal-githook Githook User added a comment -

          Author:

          {u'username': u'daveh86', u'name': u'daveh86', u'email': u'howsdav@gmail.com'}

          Message: WT-1845 - Rename test and fix review issues
          Branch: develop
          https://github.com/wiredtiger/wiredtiger/commit/a64733d8a566f36e1d16fe0fc088b7b2f3af86be

          Show
          xgen-internal-githook Githook User added a comment - Author: {u'username': u'daveh86', u'name': u'daveh86', u'email': u'howsdav@gmail.com'} Message: WT-1845 - Rename test and fix review issues Branch: develop https://github.com/wiredtiger/wiredtiger/commit/a64733d8a566f36e1d16fe0fc088b7b2f3af86be
          Hide
          xgen-internal-githook Githook User added a comment -

          Author:

          {u'username': u'daveh86', u'name': u'daveh86', u'email': u'howsdav@gmail.com'}

          Message: WT-1845 - Remove unnecessary scenarios, fix comments, re-order a conditional test.
          Branch: develop
          https://github.com/wiredtiger/wiredtiger/commit/446ddcb1f15184aa4e22f660674e5a4a729443a7

          Show
          xgen-internal-githook Githook User added a comment - Author: {u'username': u'daveh86', u'name': u'daveh86', u'email': u'howsdav@gmail.com'} Message: WT-1845 - Remove unnecessary scenarios, fix comments, re-order a conditional test. Branch: develop https://github.com/wiredtiger/wiredtiger/commit/446ddcb1f15184aa4e22f660674e5a4a729443a7
          Hide
          xgen-internal-githook Githook User added a comment -

          Author:

          {u'username': u'michaelcahill', u'name': u'Michael Cahill', u'email': u'michael.cahill@mongodb.com'}

          Message: Merge pull request #2089 from wiredtiger/WT-1845

          WT-1845 Allow read only transactions to commit after failure
          Branch: develop
          https://github.com/wiredtiger/wiredtiger/commit/55a989eaff47200d59af7531867385135e25562d

          Show
          xgen-internal-githook Githook User added a comment - Author: {u'username': u'michaelcahill', u'name': u'Michael Cahill', u'email': u'michael.cahill@mongodb.com'} Message: Merge pull request #2089 from wiredtiger/ WT-1845 WT-1845 Allow read only transactions to commit after failure Branch: develop https://github.com/wiredtiger/wiredtiger/commit/55a989eaff47200d59af7531867385135e25562d
          Hide
          xgen-internal-githook Githook User added a comment -

          Author:

          {u'username': u'michaelcahill', u'name': u'Michael Cahill', u'email': u'michael.cahill@mongodb.com'}

          Message: Merge pull request #2089 from wiredtiger/WT-1845

          WT-1845 Allow read only transactions to commit after failure
          Branch: develop
          https://github.com/wiredtiger/wiredtiger/commit/55a989eaff47200d59af7531867385135e25562d

          Show
          xgen-internal-githook Githook User added a comment - Author: {u'username': u'michaelcahill', u'name': u'Michael Cahill', u'email': u'michael.cahill@mongodb.com'} Message: Merge pull request #2089 from wiredtiger/ WT-1845 WT-1845 Allow read only transactions to commit after failure Branch: develop https://github.com/wiredtiger/wiredtiger/commit/55a989eaff47200d59af7531867385135e25562d

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Days since reply:
                1 year, 33 weeks, 3 days ago
                Date of 1st Reply: