jtibshirani commented on PR #749: URL: https://github.com/apache/lucene/pull/749#issuecomment-1087829315
@nknize I'm sorry about that, I missed that you wanted to discuss this before merging. I see this fix as a self-contained improvement -- previously we silently accepted but then failed on integer, float, or double sort fields, whereas now we handle and document them. The query is in sandbox and it doesn't seem like a big problem it doesn't support float/ double fields for now. Could you explain the open question you'd like to resolve? -- 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
