jpountz commented on code in PR #13692:
URL: https://github.com/apache/lucene/pull/13692#discussion_r1733487895


##########
lucene/core/src/java/org/apache/lucene/codecs/lucene912/Lucene912PostingsReader.java:
##########
@@ -509,15 +547,14 @@ private void refillRemainder() throws IOException {
       if (docFreq == 1) {
         docBuffer[0] = singletonDocID;
         freqBuffer[0] = totalTermFreq;
-        docBuffer[1] = NO_MORE_DOCS;
         docCountUpto++;
       } else {
         // Read vInts:
         PostingsUtil.readVIntBlock(docIn, docBuffer, freqBuffer, left, 
indexHasFreq, needsFreq);
         prefixSum(docBuffer, left, prevDocID);
-        docBuffer[left] = NO_MORE_DOCS;
         docCountUpto += left;
       }
+      Arrays.fill(docBuffer, left, docBuffer.length, NO_MORE_DOCS);

Review Comment:
   Here are the results:
   
   ```
                               TaskQPS baseline      StdDevQPS 
my_modified_version      StdDev                Pct diff p-value
                     CountOrHighMed      167.66      (7.0%)      161.29      
(7.4%)   -3.8% ( -17% -   11%) 0.096
                            Prefix3      231.74      (5.5%)      224.18      
(6.1%)   -3.3% ( -14% -    8%) 0.074
                    CountOrHighHigh       49.49     (16.9%)       48.04     
(17.9%)   -2.9% ( -32% -   38%) 0.595
               HighTermTitleBDVSort       17.79      (6.3%)       17.39      
(5.0%)   -2.2% ( -12% -    9%) 0.214
                       OrNotHighMed      306.15      (6.2%)      301.76      
(5.8%)   -1.4% ( -12% -   11%) 0.447
                            LowTerm      823.11      (6.0%)      814.89      
(5.8%)   -1.0% ( -12% -   11%) 0.591
                           Wildcard       81.54      (4.7%)       81.16      
(4.0%)   -0.5% (  -8% -    8%) 0.735
                            MedTerm      547.52      (7.0%)      545.34      
(8.0%)   -0.4% ( -14% -   15%) 0.868
                    MedSloppyPhrase        7.83      (5.2%)        7.82      
(5.8%)   -0.1% ( -10% -   11%) 0.942
                           PKLookup      286.70      (2.5%)      286.39      
(3.1%)   -0.1% (  -5% -    5%) 0.903
                        CountPhrase        6.59      (3.3%)        6.59      
(4.1%)   -0.0% (  -7% -    7%) 0.980
                          MedPhrase       46.01      (9.6%)       46.03      
(3.6%)    0.0% ( -12% -   14%) 0.986
                        OrStopWords       33.71      (5.6%)       33.78      
(5.8%)    0.2% ( -10% -   12%) 0.909
                           HighTerm      473.92      (8.0%)      474.99      
(8.2%)    0.2% ( -14% -   17%) 0.929
                           Or3Terms      252.82      (3.6%)      253.40      
(3.2%)    0.2% (  -6% -    7%) 0.829
                          OrHighLow      779.20      (2.8%)      781.17      
(2.8%)    0.3% (  -5% -    6%) 0.777
                            Respell       49.87      (2.7%)       50.01      
(2.5%)    0.3% (  -4% -    5%) 0.736
                    LowSloppyPhrase       18.60      (3.5%)       18.66      
(3.9%)    0.3% (  -6% -    8%) 0.787
                         AndHighMed      132.84      (2.4%)      133.32      
(1.7%)    0.4% (  -3% -    4%) 0.583
                   HighSloppyPhrase       15.25      (4.0%)       15.31      
(4.1%)    0.4% (  -7% -    8%) 0.742
                  HighTermTitleSort      138.90      (2.4%)      139.48      
(2.2%)    0.4% (  -4% -    5%) 0.567
                             Fuzzy1      103.74      (2.6%)      104.22      
(1.8%)    0.5% (  -3% -    5%) 0.516
                MedIntervalsOrdered        3.21      (7.0%)        3.23      
(7.2%)    0.5% ( -12% -   15%) 0.817
                         OrHighHigh       74.58      (3.9%)       74.97      
(4.7%)    0.5% (  -7% -    9%) 0.693
                LowIntervalsOrdered       23.64      (5.2%)       23.78      
(4.7%)    0.6% (  -8% -   11%) 0.709
                       OrHighNotLow      376.98      (7.8%)      379.21      
(8.1%)    0.6% ( -14% -   17%) 0.814
                        LowSpanNear        7.82      (1.8%)        7.88      
(1.7%)    0.7% (  -2% -    4%) 0.209
                  HighTermMonthSort     2902.96      (5.7%)     2926.30      
(6.0%)    0.8% ( -10% -   13%) 0.665
                        AndHighHigh       86.36      (1.9%)       87.14      
(2.3%)    0.9% (  -3% -    5%) 0.176
                And2Terms2StopWords      132.56      (3.5%)      133.76      
(2.6%)    0.9% (  -4% -    7%) 0.347
                          And3Terms      124.51      (3.2%)      125.64      
(2.2%)    0.9% (  -4% -    6%) 0.296
                          OrHighMed      249.41      (2.4%)      251.97      
(3.8%)    1.0% (  -5% -    7%) 0.310
                       HighSpanNear        4.84      (2.2%)        4.89      
(2.0%)    1.0% (  -3% -    5%) 0.128
                       OrHighNotMed      361.52      (7.3%)      365.54      
(7.6%)    1.1% ( -12% -   17%) 0.637
                             Fuzzy2       67.15      (3.8%)       68.02      
(1.5%)    1.3% (  -3% -    6%) 0.164
                 Or2Terms2StopWords      264.64      (2.9%)      268.12      
(2.6%)    1.3% (  -4% -    7%) 0.131
               HighIntervalsOrdered        4.01      (6.8%)        4.07      
(5.6%)    1.4% ( -10% -   14%) 0.491
                         HighPhrase       38.37      (5.9%)       38.93      
(2.6%)    1.4% (  -6% -   10%) 0.315
                      OrHighNotHigh      192.14      (7.1%)      195.02      
(6.4%)    1.5% ( -11% -   16%) 0.483
                        MedSpanNear        9.03      (2.2%)        9.17      
(2.2%)    1.6% (  -2% -    6%) 0.022
              HighTermDayOfYearSort      776.53      (3.5%)      789.12      
(3.6%)    1.6% (  -5% -    8%) 0.146
                       AndStopWords       30.04      (3.9%)       30.60      
(2.8%)    1.9% (  -4% -    8%) 0.082
                      OrNotHighHigh      223.21      (7.0%)      227.77      
(6.5%)    2.0% ( -10% -   16%) 0.337
                         TermDTSort      333.94      (7.0%)      341.36      
(5.5%)    2.2% (  -9% -   15%) 0.263
                         OrHighRare      244.42     (10.1%)      250.18     
(10.2%)    2.4% ( -16% -   25%) 0.463
                       OrNotHighLow      999.03      (3.6%)     1025.44      
(4.4%)    2.6% (  -5% -   10%) 0.036
                          CountTerm     7229.95      (8.6%)     7423.29     
(11.6%)    2.7% ( -16% -   25%) 0.408
                         AndHighLow      965.21      (4.0%)      994.65      
(3.5%)    3.1% (  -4% -   10%) 0.010
                          LowPhrase       30.82      (8.4%)       31.77      
(3.1%)    3.1% (  -7% -   15%) 0.128
                    CountAndHighMed      133.07      (1.3%)      138.93      
(2.0%)    4.4% (   1% -    7%) 0.000
                             IntNRQ      111.40     (25.8%)      117.21     
(25.7%)    5.2% ( -36% -   76%) 0.522
                   CountAndHighHigh       48.04      (1.2%)       53.06      
(2.3%)   10.5% (   6% -   14%) 0.000
   ```



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