[ https://issues.apache.org/jira/browse/SOLR-5616?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Dennis Gove closed SOLR-5616. ----------------------------- Resolution: Fixed > Make grouping code use response builder needDocList > --------------------------------------------------- > > Key: SOLR-5616 > URL: https://issues.apache.org/jira/browse/SOLR-5616 > Project: Solr > Issue Type: Bug > Reporter: Steven Bower > Assignee: Dennis Gove > Fix For: 6.4 > > Attachments: SOLR-5616.patch, SOLR-5616.patch, SOLR-5616.patch > > > Right now the grouping code does this to check if it needs to generate a > docList for grouped results: > {code} > if (rb.doHighlights || rb.isDebug() || params.getBool(MoreLikeThisParams.MLT, > false) ){ > ... > } > {code} > this is ugly because any new component that needs a doclist, from grouped > results, will need to modify QueryComponent to add a check to this if. > Ideally this should just use the rb.isNeedDocList() flag... > Coincidentally this boolean is really never used at for non-grouped results > it always gets generated.. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org