Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-53112

[TC] [SQLite] rows.Err() should be checked after returning ErrQueryReturnedNoRows

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • 7.1.2
    • 7.0.0, 7.0.1, 7.0.2, 7.0.3, 7.0.4, 7.1.0, 7.1.1
    • tools
    • Untriaged
    • 1
    • No

    Description

      What's the issue?
      We could potentially return the wrong error in 'QueryRows' due to order that we're checking whether any rows were returned. Note that this would only occur for the first row, all other rows would return the correct error.

      What's the fix?
      We should check that 'rows.Err()' is <nil> before returning 'ErrQueryReturnedNoRows'.

      Attachments

        Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. MB-53226 [CBM] Handle not having enough disk space in the default temporary directory for sqlite indexes

          • Major - Major loss of function.
          • Closed
          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              james.lee James Lee
              james.lee James Lee
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty