gf2121 removed a comment on pull request #578:
URL: https://github.com/apache/lucene/pull/578#issuecomment-1004760012


   Indeed, I am very suprised to see such a big QPS improvement just by 
removing some null check and method call. It is also interesting that 
BrowseMonthTaxoFacets can be much faster then BrowseMonthSSDVFacets now but I 
do think they should have similar QPS... I noticed that the stddev is also 
becoming much higher, so I wonder if there is a amazing optimization in JVM 
that is triggered from time to time after this change? It seems delicious to 
find this out and try to make it stable.
   
   To double check, I rerun a wikimediumall benchmark, getting a similar result 
(higher QPS & higher stddev)
   
   ```
                               TaskQPS baseline      StdDevQPS 
my_modified_version      StdDev                Pct diff p-value
              BrowseMonthSSDVFacets        4.67      (9.8%)        4.50      
(6.0%)   -3.6% ( -17% -   13%) 0.161
        BrowseRandomLabelSSDVFacets        3.09      (5.6%)        3.03      
(4.6%)   -2.0% ( -11% -    8%) 0.211
                         HighPhrase       15.15      (3.3%)       14.84      
(3.8%)   -2.0% (  -8% -    5%) 0.074
                          LowPhrase       37.67      (3.2%)       36.95      
(3.6%)   -1.9% (  -8% -    5%) 0.080
                            LowTerm     1671.58      (4.1%)     1644.34      
(4.8%)   -1.6% ( -10% -    7%) 0.245
                           HighTerm     1210.15      (5.5%)     1191.05      
(7.5%)   -1.6% ( -13% -   12%) 0.448
          BrowseDayOfYearSSDVFacets        4.19      (6.9%)        4.12      
(5.5%)   -1.5% ( -13% -   11%) 0.436
                    LowSloppyPhrase       11.70      (3.4%)       11.52      
(1.7%)   -1.5% (  -6% -    3%) 0.069
                          MedPhrase       17.15      (2.5%)       16.90      
(3.6%)   -1.5% (  -7% -    4%) 0.140
                   HighSloppyPhrase        8.52      (5.1%)        8.40      
(3.2%)   -1.4% (  -9% -    7%) 0.287
            AndHighMedDayTaxoFacets       45.58      (2.8%)       44.97      
(2.2%)   -1.3% (  -6% -    3%) 0.093
                       OrNotHighLow      999.69      (3.1%)      987.25      
(3.6%)   -1.2% (  -7% -    5%) 0.242
           AndHighHighDayTaxoFacets       11.16      (3.2%)       11.02      
(3.8%)   -1.2% (  -8% -    6%) 0.271
                         AndHighLow      662.82      (2.5%)      655.64      
(3.0%)   -1.1% (  -6% -    4%) 0.212
                            Respell       41.99      (2.7%)       41.58      
(3.2%)   -1.0% (  -6% -    5%) 0.290
                       OrHighNotMed     1074.76      (4.8%)     1065.13      
(6.7%)   -0.9% ( -11% -   11%) 0.626
                    MedSloppyPhrase        6.08      (3.4%)        6.02      
(2.0%)   -0.9% (  -6% -    4%) 0.317
                        AndHighHigh       30.48      (4.4%)       30.26      
(4.2%)   -0.7% (  -8% -    8%) 0.588
                          OrHighLow      451.34      (2.9%)      448.36      
(4.2%)   -0.7% (  -7% -    6%) 0.562
                      OrNotHighHigh      807.68      (4.5%)      803.96      
(5.2%)   -0.5% (  -9% -    9%) 0.764
                LowIntervalsOrdered       21.59      (5.9%)       21.49      
(5.6%)   -0.5% ( -11% -   11%) 0.804
                         AndHighMed       82.97      (3.9%)       82.62      
(4.1%)   -0.4% (  -8% -    7%) 0.737
                           PKLookup      196.30      (4.4%)      195.53      
(4.0%)   -0.4% (  -8% -    8%) 0.769
                      OrHighNotHigh      815.15      (4.4%)      812.52      
(5.8%)   -0.3% ( -10% -   10%) 0.843
                       OrNotHighMed     1132.97      (3.6%)     1130.84      
(6.7%)   -0.2% ( -10% -   10%) 0.912
                MedIntervalsOrdered       18.77      (9.9%)       18.74      
(9.2%)   -0.2% ( -17% -   21%) 0.958
                         OrHighHigh       15.86      (5.1%)       15.88      
(3.5%)    0.1% (  -8% -    9%) 0.916
                             Fuzzy1       63.19      (2.6%)       63.33      
(2.8%)    0.2% (  -5% -    5%) 0.799
               HighIntervalsOrdered        4.97     (10.2%)        4.98      
(9.5%)    0.2% ( -17% -   22%) 0.938
                          OrHighMed       43.11      (4.8%)       43.25      
(3.8%)    0.3% (  -7% -    9%) 0.819
                             IntNRQ       47.52     (27.6%)       47.67     
(29.2%)    0.3% ( -44% -   79%) 0.971
                             Fuzzy2       74.23      (2.7%)       74.48      
(3.3%)    0.3% (  -5% -    6%) 0.722
                        LowSpanNear       85.95      (3.7%)       86.34      
(4.5%)    0.5% (  -7% -    9%) 0.731
             OrHighMedDayTaxoFacets        1.91      (6.9%)        1.93      
(8.0%)    1.1% ( -12% -   17%) 0.655
                       OrHighNotLow     1181.50      (3.9%)     1194.56      
(6.5%)    1.1% (  -8% -   11%) 0.511
                            MedTerm     1734.50      (4.6%)     1754.47      
(5.9%)    1.2% (  -8% -   12%) 0.492
               MedTermDayTaxoFacets       11.24      (5.6%)       11.37      
(7.7%)    1.2% ( -11% -   15%) 0.576
                  HighTermMonthSort        9.60     (23.2%)        9.72     
(18.3%)    1.2% ( -32% -   55%) 0.852
                       HighSpanNear       10.32      (3.9%)       10.46      
(4.3%)    1.4% (  -6% -    9%) 0.289
                        MedSpanNear        2.74      (4.1%)        2.79      
(4.6%)    2.1% (  -6% -   11%) 0.130
              HighTermDayOfYearSort       43.60     (14.0%)       44.66     
(16.9%)    2.4% ( -24% -   38%) 0.619
                            Prefix3       19.72     (21.3%)       20.62     
(17.8%)    4.6% ( -28% -   55%) 0.462
                           Wildcard       36.11      (9.6%)       37.81      
(9.0%)    4.7% ( -12% -   25%) 0.109
                         TermDTSort       82.04     (13.2%)       87.57     
(18.3%)    6.7% ( -21% -   44%) 0.182
               HighTermTitleBDVSort       54.11     (13.7%)       58.78     
(18.9%)    8.6% ( -21% -   47%) 0.099
              BrowseMonthTaxoFacets        5.10     (15.2%)        7.21     
(79.7%)   41.4% ( -46% -  160%) 0.023
        BrowseRandomLabelTaxoFacets        4.04      (9.8%)        5.99     
(87.3%)   48.3% ( -44% -  161%) 0.014
               BrowseDateTaxoFacets        4.62     (12.9%)        6.96     
(90.6%)   50.6% ( -46% -  177%) 0.014
          BrowseDayOfYearTaxoFacets        4.68     (13.4%)        7.10     
(92.6%)   51.8% ( -47% -  182%) 0.013
   ```
   
   Java version:
   ```
   java version "16.0.2" 2021-07-20
   Java(TM) SE Runtime Environment (build 16.0.2+7-67)
   Java HotSpot(TM) 64-Bit Server VM (build 16.0.2+7-67, mixed mode, sharing)
   ```


-- 
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