Solr
  1. Solr
  2. SOLR-666

Expose warmup time in statistics for SolrIndexSearcher and LRUCache

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.3
    • Fix Version/s: 1.3
    • Component/s: search, web gui
    • Labels:
      None

      Description

      Expose time taken for auto warming in getStatistics method for both SolrIndexSearcher (total warmup time) as well as in LRUCache (individual warmup time) to aid monitoring.

      1. SOLR-666.patch
        2 kB
        Shalin Shekhar Mangar

        Activity

        Hide
        Shalin Shekhar Mangar added a comment -

        Adds warmupTime instance variables to both SolrIndexSearcher and LRUCache. The time is calculated in the respective warm methods. The getStatistics methods add this value to the statistics NamedList.

        I will commit this shortly.

        Show
        Shalin Shekhar Mangar added a comment - Adds warmupTime instance variables to both SolrIndexSearcher and LRUCache. The time is calculated in the respective warm methods. The getStatistics methods add this value to the statistics NamedList. I will commit this shortly.
        Hide
        Yonik Seeley added a comment -

        Isn't this redundant? We already have the time it was created, and the time it was registered. Everything in between is warming time.

        Show
        Yonik Seeley added a comment - Isn't this redundant? We already have the time it was created, and the time it was registered. Everything in between is warming time.
        Hide
        Shalin Shekhar Mangar added a comment -

        Yes, that is true for SolrIndexSearcher but we don't have it individually for the two caches. Secondly, JMX monitoring and graphing tools usually can't do arithmetic out of the box.

        Show
        Shalin Shekhar Mangar added a comment - Yes, that is true for SolrIndexSearcher but we don't have it individually for the two caches. Secondly, JMX monitoring and graphing tools usually can't do arithmetic out of the box.
        Hide
        Shalin Shekhar Mangar added a comment -

        Committed revision 680916.

        Show
        Shalin Shekhar Mangar added a comment - Committed revision 680916.

          People

          • Assignee:
            Shalin Shekhar Mangar
            Reporter:
            Shalin Shekhar Mangar
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development