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

    • Untriaged
    • 1
    • 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 6.6.1 [ 17002 ]
            Fix Version/s Cheshire-Cat [ 15915 ]
            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

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

            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 ]

            Hi Akhil Mundroy

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

            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?

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

            mihir.kamdar Mihir Kamdar (Inactive) 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

            Verified using plasma unit test

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

            People

              akhil.mundroy Akhil Mundroy
              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