rmuir commented on code in PR #16816:
URL: https://github.com/apache/lucene/pull/16816#discussion_r4254089183
##########
lucene/core/src/java/org/apache/lucene/search/FieldExistsQuery.java:
##########
@@ -152,10 +165,19 @@ public Query rewrite(IndexSearcher indexSearcher) throws
IOException {
allReadersRewritable = false;
break;
}
+ } else if (fieldInfo.getIndexOptions() != IndexOptions.NONE) { // the
field only indexes terms
Review Comment:
maybe if we moved this line comment to be the first comment inside the "else
if" we'd get better formatting throughout here. It works in this case, but in
the other cases below it is causing confusing wrapping
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
