[ 
https://issues.apache.org/jira/browse/SOLR-2976?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13536641#comment-13536641
 ] 

Robert Muir commented on SOLR-2976:
-----------------------------------

Yonik: Ill port over my test. But i think this is the piece of the code that 
causes it in SimpleFacets:
{code}
boolean multiToken = sf.multiValued() || ft.multiValuedFieldCache();

if (TrieField.getMainValuePrefix(ft) != null) {
  // A TrieField with multiple parts indexed per value... currently only
  // UnInvertedField can handle this case, so force it's use.
  enumMethod = false;
  multiToken = true;
}
{code}

This second "if" I think causes the problem? It only returns null if 
precisionStep is 0:
{code}
if (trie.precisionStep  == Integer.MAX_VALUE)
  return null;
{code}
                
> stats.facet no longer works on single valued trie fields that don't use 
> precision step
> --------------------------------------------------------------------------------------
>
>                 Key: SOLR-2976
>                 URL: https://issues.apache.org/jira/browse/SOLR-2976
>             Project: Solr
>          Issue Type: Bug
>    Affects Versions: 3.5
>            Reporter: Hoss Man
>            Assignee: Robert Muir
>         Attachments: SOLR-2976_3.4_test.patch, SOLR-2976.patch
>
>
> As reported on the mailing list, 3.5 introduced a regression that prevents 
> single valued Trie fields that don't use precision steps (to add course 
> grained terms) from being used in stats.facet.
> two immediately obvious problems...
> 1) in 3.5 the stats component is checking if isTokenzed() is true for the 
> field type (which is probably wise) but regardless of the precisionStep used, 
> TrieField.isTokenized is hardcoded to return true
> 2) the 3.5 stats faceting will fail if the FieldType is multivalued - it 
> doesn't check if the SchemaField is configured to be single valued 
> (overriding the FieldType)
> so even if a user has something like this in their schema...
> {code}
> <fieldType name="long" class="solr.TrieLongField" precisionStep="0" 
> omitNorms="true" />
> <field name="ts" type="long" indexed="true" stored="true" required="true" 
> multiValued="false" />
> {code}
> ...stats.facet will not work.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to