gsmiller commented on pull request #585:
URL: https://github.com/apache/lucene/pull/585#issuecomment-1005949811


   Maybe a very small improvement with this change, but nothing particularly 
impactful. I wonder how often we actually trigger this case in our benchmarks? 
Certainly not as often as with the "count all" cases. I think it's worth making 
this change though for consistency (and I think there's a small improvement 
there anyway).
   
   ```
                               TaskQPS baseline      StdDevQPS candidate      
StdDev                Pct diff p-value
          BrowseDayOfYearTaxoFacets       14.53     (10.8%)       14.08     
(11.4%)   -3.1% ( -22% -   21%) 0.378
               BrowseDateTaxoFacets       14.48     (10.7%)       14.07     
(11.2%)   -2.8% ( -22% -   21%) 0.416
               HighIntervalsOrdered        3.00      (5.2%)        2.94      
(7.0%)   -1.8% ( -13% -   10%) 0.359
                      OrNotHighHigh     1044.76      (4.5%)     1028.26      
(3.0%)   -1.6% (  -8% -    6%) 0.188
                MedIntervalsOrdered       35.82      (4.8%)       35.28      
(6.1%)   -1.5% ( -11% -    9%) 0.384
                       OrNotHighMed      969.48      (2.7%)      957.92      
(2.3%)   -1.2% (  -5% -    3%) 0.129
                          OrHighLow     1228.88      (2.8%)     1215.30      
(2.5%)   -1.1% (  -6% -    4%) 0.188
              BrowseMonthTaxoFacets       15.15     (10.3%)       14.99     
(10.0%)   -1.1% ( -19% -   21%) 0.740
                       OrNotHighLow      864.18      (2.4%)      856.27      
(2.0%)   -0.9% (  -5% -    3%) 0.187
                LowIntervalsOrdered       14.43      (2.5%)       14.32      
(3.3%)   -0.7% (  -6% -    5%) 0.422
        BrowseRandomLabelTaxoFacets       12.34      (9.2%)       12.26      
(9.9%)   -0.6% ( -18% -   20%) 0.834
                      OrHighNotHigh      877.19      (3.9%)      872.54      
(3.8%)   -0.5% (  -7% -    7%) 0.662
                         HighPhrase      106.42      (1.7%)      106.06      
(1.4%)   -0.3% (  -3% -    2%) 0.498
                            MedTerm     1930.85      (4.6%)     1924.69      
(3.5%)   -0.3% (  -8% -    8%) 0.806
                           HighTerm     1383.88      (4.6%)     1380.07      
(4.1%)   -0.3% (  -8% -    8%) 0.843
                          MedPhrase      744.32      (2.1%)      742.71      
(2.4%)   -0.2% (  -4% -    4%) 0.761
                            LowTerm     3065.99      (3.3%)     3060.87      
(3.8%)   -0.2% (  -7% -    7%) 0.882
                          LowPhrase       82.62      (1.6%)       82.51      
(1.4%)   -0.1% (  -3% -    2%) 0.768
                       OrHighNotLow     1094.83      (4.7%)     1094.10      
(3.1%)   -0.1% (  -7% -    8%) 0.958
                    MedSloppyPhrase      104.44      (2.5%)      104.41      
(3.3%)   -0.0% (  -5% -    5%) 0.980
                          OrHighMed      193.98      (4.4%)      193.96      
(4.6%)   -0.0% (  -8% -    9%) 0.994
                            Respell       65.50      (1.1%)       65.52      
(1.2%)    0.0% (  -2% -    2%) 0.913
                             Fuzzy2       84.12      (1.1%)       84.16      
(1.0%)    0.0% (  -2% -    2%) 0.901
                         AndHighLow     1163.35      (3.5%)     1163.95      
(3.1%)    0.1% (  -6% -    6%) 0.961
        BrowseRandomLabelSSDVFacets        9.34      (1.7%)        9.35      
(2.9%)    0.1% (  -4% -    4%) 0.940
              BrowseMonthSSDVFacets       12.91     (13.1%)       12.92     
(13.1%)    0.1% ( -23% -   30%) 0.978
                       OrHighNotMed     1094.06      (3.9%)     1095.46      
(3.1%)    0.1% (  -6% -    7%) 0.908
                           PKLookup      171.75      (3.0%)      172.12      
(3.3%)    0.2% (  -5% -    6%) 0.829
                            Prefix3      365.59      (9.9%)      366.48      
(9.1%)    0.2% ( -17% -   21%) 0.936
                             Fuzzy1      113.46      (1.2%)      113.82      
(1.2%)    0.3% (  -2% -    2%) 0.390
                           Wildcard       38.65      (8.2%)       38.77      
(7.8%)    0.3% ( -14% -   17%) 0.898
                   HighSloppyPhrase       11.81      (3.3%)       11.85      
(3.9%)    0.3% (  -6% -    7%) 0.773
                         OrHighHigh       12.04      (4.0%)       12.10      
(4.0%)    0.5% (  -7% -    8%) 0.713
                    LowSloppyPhrase        7.75      (3.6%)        7.79      
(4.3%)    0.5% (  -7% -    8%) 0.703
                        LowSpanNear       29.78      (2.5%)       29.96      
(2.4%)    0.6% (  -4% -    5%) 0.420
                  HighTermMonthSort      126.96     (16.0%)      127.81     
(14.3%)    0.7% ( -25% -   36%) 0.889
                       HighSpanNear       12.43      (3.3%)       12.52      
(3.2%)    0.7% (  -5% -    7%) 0.488
                        MedSpanNear       12.64      (2.9%)       12.73      
(2.9%)    0.8% (  -4% -    6%) 0.402
                         AndHighMed       55.03      (5.2%)       55.48      
(5.6%)    0.8% (  -9% -   12%) 0.631
                         TermDTSort      100.53     (15.0%)      101.68     
(14.3%)    1.1% ( -24% -   35%) 0.805
                        AndHighHigh       38.00      (5.1%)       38.49      
(6.3%)    1.3% (  -9% -   13%) 0.481
           AndHighHighDayTaxoFacets       32.44      (2.4%)       32.89      
(1.8%)    1.4% (  -2% -    5%) 0.039
          BrowseDayOfYearSSDVFacets       12.17     (11.0%)       12.36     
(13.2%)    1.5% ( -20% -   28%) 0.696
               HighTermTitleBDVSort      115.13     (16.1%)      117.00     
(15.2%)    1.6% ( -25% -   39%) 0.744
            AndHighMedDayTaxoFacets       29.45      (2.5%)       29.96      
(2.2%)    1.7% (  -2% -    6%) 0.024
               MedTermDayTaxoFacets       39.09      (4.8%)       39.85      
(6.7%)    1.9% (  -9% -   14%) 0.294
             OrHighMedDayTaxoFacets       13.61      (5.6%)       13.91      
(5.3%)    2.2% (  -8% -   13%) 0.199
              HighTermDayOfYearSort      145.34     (15.1%)      149.53     
(14.4%)    2.9% ( -23% -   38%) 0.536
                             IntNRQ      112.30     (12.5%)      116.40      
(1.7%)    3.7% (  -9% -   20%) 0.197
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to