Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.3
    • Fix Version/s: 3.4
    • Component/s: search
    • Labels:
      None

      Description

      When the queryResultCache is not defined in the configuration, the start parameter increments the rows parameter. Start + rows is returned and start is always 0.

      1. SOLR-2662.patch
        1 kB
        Yonik Seeley

        Activity

        Hide
        Mark Miller added a comment -

        Nice find Markus.

        Show
        Mark Miller added a comment - Nice find Markus.
        Hide
        Markus Jelsma added a comment -

        The following may be related. Below the config, just an attempt to disable the query cache.

        <queryResultCache class="solr.FastLRUCache" size="0" initialSize="0" autowarmCount="0"/>
        

        Here's what happens when looking at the stats:

        Concurrent LRU Cache(maxSize=2, initialSize=0, minSize=1, acceptableSize=1, cleanupThread=false)
        

        Strange, it's actually in use and it even works!

        lookups : 41
        hits : 0
        hitratio : 0.00
        inserts : 0
        evictions : 0
        size : 0
        warmupTime : 0
        cumulative_lookups : 145
        cumulative_hits : 2
        cumulative_hitratio : 0.01
        cumulative_inserts : 2
        cumulative_evictions : 0 
        
        Show
        Markus Jelsma added a comment - The following may be related. Below the config, just an attempt to disable the query cache. <queryResultCache class= "solr.FastLRUCache" size= "0" initialSize= "0" autowarmCount= "0" /> Here's what happens when looking at the stats: Concurrent LRU Cache(maxSize=2, initialSize=0, minSize=1, acceptableSize=1, cleanupThread= false ) Strange, it's actually in use and it even works! lookups : 41 hits : 0 hitratio : 0.00 inserts : 0 evictions : 0 size : 0 warmupTime : 0 cumulative_lookups : 145 cumulative_hits : 2 cumulative_hitratio : 0.01 cumulative_inserts : 2 cumulative_evictions : 0
        Hide
        Yonik Seeley added a comment -

        Here's a patch for trunk.

        Show
        Yonik Seeley added a comment - Here's a patch for trunk.
        Hide
        Robert Muir added a comment -

        bulk close for 3.4

        Show
        Robert Muir added a comment - bulk close for 3.4

          People

          • Assignee:
            Unassigned
            Reporter:
            Markus Jelsma
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development