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

parsed.Index() might return wrong value

    XMLWordPrintable

    Details

    • Triage:
      Untriaged
    • Story Points:
      1
    • Is this a Regression?:
      Yes

      Description

      parsed.Index() caches the wrong element.
      Should the same element be requested twice, and caching be on, second time round it may return the wrong value.
      Users accessing the same element of an an array twice might receive wrong results, which is unacceptable.

        Attachments

        For Gerrit Dashboard: MB-41308
        # Subject Branch Project Status CR V

          Activity

          Hide
          build-team Couchbase Build Team added a comment -

          Build couchbase-server-6.5.1-6311 contains query commit ab512c2 with commit message:
          MB-41308 parsed.Index() caches wrong value

          Show
          build-team Couchbase Build Team added a comment - Build couchbase-server-6.5.1-6311 contains query commit ab512c2 with commit message: MB-41308 parsed.Index() caches wrong value
          Hide
          mihir.kamdar Mihir Kamdar added a comment -

          Validated on 6.5.1-6311 using the tests listed in https://docs.google.com/spreadsheets/d/1e7H8T6VXAtIn9Ix_-DhdwDHqpB8wfxXks3Olv2VZBZA/edit#gid=0. Works fine.

          Show
          mihir.kamdar Mihir Kamdar added a comment - Validated on 6.5.1-6311 using the tests listed in https://docs.google.com/spreadsheets/d/1e7H8T6VXAtIn9Ix_-DhdwDHqpB8wfxXks3Olv2VZBZA/edit#gid=0 . Works fine.
          Hide
          ajay.bhullar Ajay Bhullar added a comment -
          Show
          ajay.bhullar Ajay Bhullar added a comment - verified in 7.0.0-3445 using tests listed in https://docs.google.com/spreadsheets/d/1e7H8T6VXAtIn9Ix_-DhdwDHqpB8wfxXks3Olv2VZBZA/edit#gid=0
          Hide
          ajay.bhullar Ajay Bhullar added a comment -
          Show
          ajay.bhullar Ajay Bhullar added a comment - re-verified in 6.6.1-9132 using the tests listed in https://docs.google.com/spreadsheets/d/1e7H8T6VXAtIn9Ix_-DhdwDHqpB8wfxXks3Olv2VZBZA/edit#gid=0
          Hide
          ajay.bhullar Ajay Bhullar added a comment -

          closing as this issue was verified in 6.5.1,6.6.1, and 7.0.0 using the list of test cases here https://docs.google.com/spreadsheets/d/1e7H8T6VXAtIn9Ix_-DhdwDHqpB8wfxXks3Olv2VZBZA/edit#gid=0

          Show
          ajay.bhullar Ajay Bhullar added a comment - closing as this issue was verified in 6.5.1,6.6.1, and 7.0.0 using the list of test cases here https://docs.google.com/spreadsheets/d/1e7H8T6VXAtIn9Ix_-DhdwDHqpB8wfxXks3Olv2VZBZA/edit#gid=0

            People

            Assignee:
            mihir.kamdar Mihir Kamdar
            Reporter:
            marco.greco Marco Greco
            Votes:
            0 Vote for this issue
            Watchers:
            7 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Gerrit Reviews

                There are no open Gerrit changes

                  PagerDuty