You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucenenet.apache.org by GitBox <gi...@apache.org> on 2021/11/18 17:43:12 UTC

[GitHub] [lucenenet] NightOwl888 opened a new pull request #541: BUG: Lucene.Net.Codecs.SimpleText.SimpleTextFieldsReader::NextDoc(): Fixed assert that was throwing on BytesRef.Utf8ToString()

NightOwl888 opened a new pull request #541:
URL: https://github.com/apache/lucenenet/pull/541


   Assert statement was using `BytesRef.Utf8ToString()`, which was causing exceptions when the `BytesRef` didn't contain valid UTF8. Changed to use [`BytesRefFormatter`](Lucene.Net.Codecs.SimpleText.SimpleTextFieldsReader::NextDoc()), which defers the call to convert the `BytesRef` until after the assert fails.
   
   This was causing several tests to randomly fail that use `SimpleText` to compare other codecs with.


-- 
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: dev-unsubscribe@lucenenet.apache.org

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



[GitHub] [lucenenet] NightOwl888 merged pull request #541: BUG: Lucene.Net.Codecs.SimpleText.SimpleTextFieldsReader::NextDoc(): Fixed assert that was throwing on BytesRef.Utf8ToString()

Posted by GitBox <gi...@apache.org>.
NightOwl888 merged pull request #541:
URL: https://github.com/apache/lucenenet/pull/541


   


-- 
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: dev-unsubscribe@lucenenet.apache.org

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