fix ATOM sort and searchAfter - Githubissues

Yelp / nrtsearch

A high performance gRPC server on top of Apache Lucene

Apache License 2.0

266 stars 40 forks source link

fix ATOM sort and searchAfter #732

Closed taoyyu closed 1 month ago

taoyyu commented 1 month ago

Sort with ATOM was broken. There was a recent fix to use SORTED doc value by default. However, ATOM is still broken due to conversion between BytesRef and String here https://github.com/taoyyu/nrtsearch/blob/3b366075766d80033c05dd5c66c872def56f831a/src/main/java/com/yelp/nrtsearch/server/luceneserver/search/sort/SortParser.java#L156

And we have to convert the String to BytesRef for searchAfter.

Added some tests for it.