kkewwei commented on PR #16669:
URL: https://github.com/apache/lucene/pull/16669#issuecomment-5712494248

   I remove the `sparse` path and use the `dense` path universally, test twice, 
the final results are:
   test1:
   ```
                               TaskQPS baseline      StdDevQPS 
my_modified_version      StdDev                Pct diff p-value
                 ReqExclSparse1TopN      143.18      (2.9%)      138.73      
(2.4%)   -3.1% (  -8% -    2%) 0.000
               ReqExclBoundary2TopN      134.58      (3.0%)      131.97      
(2.4%)   -1.9% (  -7% -    3%) 0.028
                           PKLookup      192.52      (8.6%)      197.90      
(5.3%)    2.8% ( -10% -   18%) 0.227
                  ReqExclDense3TopN        2.83      (7.0%)       13.19     
(15.1%)  366.4% ( 321% -  417%) 0.000
                  ReqExclDense7TopN        1.29      (8.4%)        8.70     
(19.1%)  572.7% ( 502% -  655%) 0.000
                 ReqExclDense11TopN        0.77      (8.4%)        6.74     
(26.5%)  777.0% ( 684% -  886%) 0.000
                 ReqExclDense16TopN        0.55      (8.5%)        5.32     
(31.1%)  871.2% ( 766% -  995%) 0.000
   ```
   
   test2:
   ```
                               TaskQPS baseline      StdDevQPS 
my_modified_version      StdDev                Pct diff p-value
                 ReqExclSparse1TopN      138.79      (2.6%)      134.85      
(2.6%)   -2.8% (  -7% -    2%) 0.001
               ReqExclBoundary2TopN      130.56      (2.6%)      128.51      
(2.8%)   -1.6% (  -6% -    3%) 0.069
                           PKLookup      182.40      (9.8%)      181.24      
(9.1%)   -0.6% ( -17% -   20%) 0.830
                  ReqExclDense3TopN        2.80      (6.2%)       12.83     
(17.6%)  357.6% ( 314% -  406%) 0.000
                  ReqExclDense7TopN        1.32      (8.0%)        8.41     
(21.9%)  536.6% ( 469% -  615%) 0.000
                 ReqExclDense11TopN        0.78      (7.9%)        6.47     
(28.8%)  728.9% ( 641% -  831%) 0.000
                 ReqExclDense16TopN        0.56      (7.9%)        5.07     
(34.5%)  807.4% ( 708% -  923%) 0.000
   ```
   
   It looks like the situation hasn't worsened; perhaps the `spare` path can be 
deleted.
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to