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

Plasma Stats Compact Counter not updated on CompactFullMarshal

    XMLWordPrintable

Details

    • Untriaged
    • 1
    • Unknown

    Description

      On Page Marshal, a page can get compacted if CompactFullMarshal is enabled. We are not updating the compacts counter in code paths involving LSSCleaner, Persist after a page is Marshalled.

       

      // Compact the page if going through full marshal
      func (pg *page) compactFullMarshal(maxSegments int) (bool, int, int) {
      if pg.ctx.CompactFullMarshal &&
      pg.head != nil &&
      pg.head.GetOp() != opBasePage &&
      pg.head.GetOp() != opPageRemoveDelta &&
      pg.head.GetOp() != opMetaDelta {
      // full marshal required?
      if_, fInfo := pg.findFlushInfo(); fInfo != nil {
      if fInfo.NumSegments > maxSegments {
      dataSz, hdrSz := pg.Compact()
      returntrue, dataSz, hdrSz
      }
      }
      }
      returnfalse, 0, 0
      }

       

      Attachments

        Issue Links

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

          Activity

            saptarshi.sen Saptarshi Sen created issue -
            saptarshi.sen Saptarshi Sen made changes -
            Field Original Value New Value
            Link This issue is cloned by MB-47992 [ MB-47992 ]
            saptarshi.sen Saptarshi Sen made changes -
            VERIFICATION STEPS Added a unit test to verify
            Resolution Fixed [ 1 ]
            Status Open [ 1 ] Resolved [ 5 ]
            wayne Wayne Siu made changes -
            Link This issue backports to MB-47992 [ MB-47992 ]
            wayne Wayne Siu made changes -
            Link This issue is cloned by MB-47992 [ MB-47992 ]
            ritam.sharma Ritam Sharma made changes -
            Labels plasma plasma request-dev-verify
            srinath.duvuru Srinath Duvuru made changes -
            Status Resolved [ 5 ] Closed [ 6 ]

            People

              saptarshi.sen Saptarshi Sen
              saptarshi.sen Saptarshi Sen
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty