madrob commented on a change in pull request #1371: URL: https://github.com/apache/lucene-solr/pull/1371#discussion_r414654670
########## File path: solr/core/src/java/org/apache/solr/search/CollapsingQParserPlugin.java ########## @@ -218,7 +245,7 @@ public static GroupHeadSelector build(final SolrParams localParams) { public String hint; private boolean needsScores = true; private boolean needsScores4Collapsing = false; - private int nullPolicy; + private NullPolicy nullPolicy; private Set<BytesRef> boosted; // ordered by "priority" public static final int NULL_POLICY_IGNORE = 0; Review comment: These constants can be removed and everything routed through the NullPolicy enum. Same for the `NULL_COLLAPSE` string above and others. ---------------------------------------------------------------- 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. 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