Currently only single valued fields are possible to do range queries on due to limitations of Solr and the fact that these were not indexed using the sortable flag.

See : http://stackoverflow.com/questions/16416396/querying-solr-multi-valued-f...

When we index Drupal Commerce product that have different variations we do need to filter on price for each of the variations of this product. The documentation tells us that we should only do range queries for multiValued field if we are using the sortable fields. In code this means sint, sdouble and sfloat.

Attached patch fixes this and also adds some flexibility to the handles so that if user input is defined it does not go back to the machine values of the solr stats but it keeps the handles at whatever value the user has selected.

CommentFileSizeAuthor
#5 interdiff-2144877-2-5.txt8.2 KBBR0kEN
#5 apachesolr-range_query_multivalued-2144877-5.patch7.27 KBBR0kEN
#2 apachesolr-range_query_multivalued-2144877-2.patch5.29 KBdanielmrichards
#1 2144877-1.patch5.31 KBNick_vh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Nick_vh’s picture

Nick_vh
he/him
Location Ghent
CreditAttribution: Nick_vh commented
FileSize
2144877-1.patch5.31 KB
danielmrichards’s picture

danielmrichards CreditAttribution: danielmrichards commented
Status: Active » Reviewed & tested by the community
FileSize
apachesolr-range_query_multivalued-2144877-2.patch5.29 KB

Patch in #1 works nicely for me. Just made a small change to reinstate the caching of the statistics result.

BR0kEN’s picture

BR0kEN
Primary language Ukrainian
CreditAttribution: BR0kEN as a volunteer and for Propeople (now part of FFW), FFW commented
Status: Reviewed & tested by the community » Needs work

Unfortunately patch breaks the search on Solr 4.x. Not whole logic in it, but exact this place:

    if (in_array($field_info['index_type'], array('sint', 'sdouble', 'sfloat'))) {
      $field_info['multiple'] = TRUE;
BR0kEN’s picture

BR0kEN
Primary language Ukrainian
CreditAttribution: BR0kEN as a volunteer and for Propeople (now part of FFW), FFW commented

Solr 3.x - patch works well.
Solr 4.x - can not use FieldCache on multivalued field: <FIELD>
Solr 5.x - Stats can only facet on single-valued fields

BR0kEN’s picture

BR0kEN
Primary language Ukrainian
CreditAttribution: BR0kEN as a volunteer and for Propeople (now part of FFW), FFW commented
Status: Needs work » Needs review
Related issues: +#2144885: Support product variations better so that you don't need to index the same product multiple times
FileSize
apachesolr-range_query_multivalued-2144877-5.patch7.27 KB
interdiff-2144877-2-5.txt8.2 KB

This patch alongside with #2144885: Support product variations better so that you don't need to index the same product multiple times allows to use Stats and range queries on every Solr.

BR0kEN’s picture

BR0kEN
Primary language Ukrainian
CreditAttribution: BR0kEN as a volunteer and for Propeople (now part of FFW), FFW commented
Issue tags: +DevDaysSeville