You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by GitBox <gi...@apache.org> on 2021/10/11 15:57:18 UTC

[GitHub] [lucene] jpountz opened a new pull request #374: LUCENE-10165: Implement Lucene90DocValuesProducer#getMergeInstance.

jpountz opened a new pull request #374:
URL: https://github.com/apache/lucene/pull/374


   This speeds up merging by returning doc values that perform faster when all doc
   IDs and values are consumed.


-- 
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-unsubscribe@lucene.apache.org

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



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


[GitHub] [lucene] rmuir commented on pull request #374: LUCENE-10165: Implement Lucene90DocValuesProducer#getMergeInstance.

Posted by GitBox <gi...@apache.org>.
rmuir commented on pull request #374:
URL: https://github.com/apache/lucene/pull/374#issuecomment-947898940


   looks fine to me, as long as the bulk-decoding really gives speedup and doesn't regress anything (e.g. confuse hotspot). 


-- 
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-unsubscribe@lucene.apache.org

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



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


[GitHub] [lucene] jpountz merged pull request #374: LUCENE-10165: Implement Lucene90DocValuesProducer#getMergeInstance.

Posted by GitBox <gi...@apache.org>.
jpountz merged pull request #374:
URL: https://github.com/apache/lucene/pull/374


   


-- 
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-unsubscribe@lucene.apache.org

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



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


[GitHub] [lucene] jpountz commented on pull request #374: LUCENE-10165: Implement Lucene90DocValuesProducer#getMergeInstance.

Posted by GitBox <gi...@apache.org>.
jpountz commented on pull request #374:
URL: https://github.com/apache/lucene/pull/374#issuecomment-948349796


   I didn't observe hotspot confusion when benchmarking locally but I'll be watching nigthlies to check if they see something.


-- 
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-unsubscribe@lucene.apache.org

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



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