Uploaded image for project: 'Samza'
  1. Samza
  2. SAMZA-963

Add timers to help identify performance issues with KV stores and producers.

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 0.11.0
    • 0.11.0
    • None
    • None

    Description

      We have good timing metrics for many of the primary actions in the event loop:

      • Choose
        • Deserialization
        • Poll
      • Process
      • Window
      • Commit

      I've noticed a few things while analyzing job performance at LinkedIn:
      1. We can usually identify problems in Choose using the sub metrics for Deserialization and Poll. I don't think any work needs to be done here.

      2. Slowness in Process or Window is usually caused by business logic (e.g. side calls to remote DBs), but it can also be caused by slowness (e.g. "stalls" in the case of RocksDB) in the KV Store.

      3. Slowness in Commit can be caused by slowness flushing the stores or producers. It can also come from checkpointing.

      #2 would be better if we had timers around all the main KV Store operations, including get, put, delete, and the batch operations. Then we can isolate KV Store performance from business logic performance.

      #3 would be improved if we had timers around all the flushes. Specifically, I think we should add a "flush-ns" metric to the KeyValueStoreMetrics and update it from each of the stores. I noticed that KafkaSystemProducerMetrics has a "flush-ns" metric, so the kafka producer is covered.

      To summarize, this ticket is to add metrics around all KV Store operations, not just for user operations like get/put, but flush as well.

      Related work: SAMZA-449

      Attachments

        1. SAMZA-963.1.patch
          10 kB
          Fred Ji
        2. SAMZA-963.2.patch
          9 kB
          Fred Ji

        Activity

          People

            fredji Fred Ji
            jmakes Jake Maes
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: