dungba88 commented on PR #12624:
URL: https://github.com/apache/lucene/pull/12624#issuecomment-1828839806

   Ah I think since we removed the finish(), getting the reverse bytes reader 
is expectedly slower. We have to copy the bytes to a readonly buffer every 
time. If this is a problem maybe let try to add the finish() again? WDYT
   
   The ram bytes used is an easy fix. I'll change it in next rev.
   
   The difference in number of nodes is interesting. Lwt me look deeper. But 
did the 9.x you are testing has incorporated the value-based node hash?


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