Query stats fields lastExecutionMicros and totalExecMicros misleading for writes on router

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Unresolved
    • Priority: Critical - P2
    • None
    • Affects Version/s: None
    • Component/s: None
    • Query Integration
    • ALL
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      These fields in query stats can have misleading names depending on context:

      • lastExecutionMicros
      • totalExecMicros
        For queries, which are executed one at a time, these names are fine. But for writes performed on the router, we may be executing a batch of write ops, even though we collect query stats for single write ops. We don't have any way of isolating the time for any individual op on the router.

      With writes on the router, it makes more sense to use the term "latency." We are reporting the time it takes from the router receiving a batched request to the time it takes when it returns a response. However, because we subtract out execution time from the duration, it may also be misleading. We should come up with a better solution here.

      We might consider including both latency (with pauses of execution not subtracted) and execution time (which subtracts pauses in execution).

            Assignee:
            Unassigned
            Reporter:
            Chris Wolff
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: