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

View read per sec. should not add per-node counters but max

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-beta
    • Fix Version/s: 3.0
    • Component/s: ns_server
    • Security Level: Public
    • Labels:

      Description

      Each view query hits all nodes. Thus adding view hits from all nodes multiplies query rate by times number of nodes

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

        Activity

        Hide
        perry Perry Krug added a comment -

        I'm of the opinion that only counting requests on the query node makes a bit more sense here to the end user. I totally accept that all of the nodes are participating in these requests, but it will match closer to what the application is actually doing to measure only at the query node. Additionally, it will allow us to diagnose any imbalance of requests (we've seen that previously) whereas measuring all index access would mask this.

        My two cents of course

        Show
        perry Perry Krug added a comment - I'm of the opinion that only counting requests on the query node makes a bit more sense here to the end user. I totally accept that all of the nodes are participating in these requests, but it will match closer to what the application is actually doing to measure only at the query node. Additionally, it will allow us to diagnose any imbalance of requests (we've seen that previously) whereas measuring all index access would mask this. My two cents of course
        Hide
        dipti Dipti Borkar added a comment -

        I agree. I think that was the original goal. so let's try to fix next release.

        Show
        dipti Dipti Borkar added a comment - I agree. I think that was the original goal. so let's try to fix next release.
        Hide
        dipti Dipti Borkar added a comment -

        btw, do we have another query per sec metric as well or is this it?

        Show
        dipti Dipti Borkar added a comment - btw, do we have another query per sec metric as well or is this it?
        Hide
        alkondratenko Aleksey Kondratenko (Inactive) added a comment -

        Created separate ticket: MB-9314.

        Show
        alkondratenko Aleksey Kondratenko (Inactive) added a comment - Created separate ticket: MB-9314 .
        Hide
        alkondratenko Aleksey Kondratenko (Inactive) added a comment -

        That's the only metric we expose

        Show
        alkondratenko Aleksey Kondratenko (Inactive) added a comment - That's the only metric we expose

          People

          • Assignee:
            artem Artem Stemkovski
            Reporter:
            pavelpaulau Pavel Paulau
          • Votes:
            0 Vote for this issue
            Watchers:
            7 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes