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

[BP 6.6.1 MB-41715] Avoid overcounting NumRecordAllocs when adding bloomDelta

    XMLWordPrintable

    Details

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

      Description

      When bloomDelta is added after recovery when page is found without bloom filter, the stat NumRecordAllocs is over counted. NumRecordAllocs is only supposed to track the insert/delete deltas.

        Attachments

          Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            Activity

            akhil.mundroy Akhil Mundroy created issue -
            akhil.mundroy Akhil Mundroy made changes -
            Field Original Value New Value
            Link This issue Clones MB-41715 [ MB-41715 ]
            akhil.mundroy Akhil Mundroy made changes -
            Fix Version/s Cheshire-Cat [ 15915 ]
            Fix Version/s 6.6.1 [ 17002 ]
            akhil.mundroy Akhil Mundroy made changes -
            Link This issue Clones MB-41715 [ MB-41715 ]
            akhil.mundroy Akhil Mundroy made changes -
            Link This issue is a backport of MB-41715 [ MB-41715 ]
            wayne Wayne Siu made changes -
            Link This issue blocks MB-40528 [ MB-40528 ]
            wayne Wayne Siu made changes -
            Labels plasma approved-for-6.6.1 plasma
            Hide
            build-team Couchbase Build Team added a comment -

            Build couchbase-server-6.6.1-9101 contains plasma commit faf787a with commit message:
            MB-41717 [BP]: Avoid incrementing nrecAllocs for bloomDelta

            Show
            build-team Couchbase Build Team added a comment - Build couchbase-server-6.6.1-9101 contains plasma commit faf787a with commit message: MB-41717 [BP] : Avoid incrementing nrecAllocs for bloomDelta
            akhil.mundroy Akhil Mundroy made changes -
            Resolution Fixed [ 1 ]
            Status Open [ 1 ] Resolved [ 5 ]
            Hide
            girish.benakappa Girish Benakappa added a comment -

            Hi Akhil Mundroy

            if this needs QE Verification, Can you please update the steps to reproduce and verify this issue?

            Show
            girish.benakappa Girish Benakappa added a comment - Hi Akhil Mundroy if this needs QE Verification, Can you please update the steps to reproduce and verify this issue?
            Hide
            mihir.kamdar Mihir Kamdar added a comment -

            Akhil Mundroy Srinath Duvuru Can you pls help with pointers on testing the fix ?

            Show
            mihir.kamdar Mihir Kamdar added a comment - Akhil Mundroy Srinath Duvuru Can you pls help with pointers on testing the fix ?
            ritam.sharma Ritam Sharma made changes -
            Labels approved-for-6.6.1 plasma approved-for-6.6.1 plasma request-dev-verify
            Hide
            akhil.mundroy Akhil Mundroy added a comment -

            Verified using plasma unit test

            Show
            akhil.mundroy Akhil Mundroy added a comment - Verified using plasma unit test
            akhil.mundroy Akhil Mundroy made changes -
            Status Resolved [ 5 ] Closed [ 6 ]

              People

              Assignee:
              akhil.mundroy Akhil Mundroy
              Reporter:
              akhil.mundroy Akhil Mundroy
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Gerrit Reviews

                  There are no open Gerrit changes

                    PagerDuty