Solr
  1. Solr
  2. SOLR-343

Constraining date facets by facet.mincount

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.2
    • Fix Version/s: 1.5, 3.1, 4.0-ALPHA
    • Component/s: search
    • Labels:
      None
    • Environment:

      Solr 1.2+

      Description

      It would be helpful to allow the facet.mincount parameter to work with date facets, i.e. constraining the results so that it would be possible to filter out date ranges in the results where no documents occur from the server-side.

      1. DateFacetsMincountPatch.patch
        1 kB
        Raiko Eckstein
      2. SOLR-343.patch
        4 kB
        Uri Boness

        Activity

        Hide
        Raiko Eckstein added a comment -

        This patch allows constraining date facets similar to facet.mincount. The constraint is checked for each facet field.
        The facet.date.other parameters are not affected as they have to be requested explicitly anyway.

        Show
        Raiko Eckstein added a comment - This patch allows constraining date facets similar to facet.mincount. The constraint is checked for each facet field. The facet.date.other parameters are not affected as they have to be requested explicitly anyway.
        Hide
        Grant Ingersoll added a comment - - edited

        Can you add tests for this please? They are in the BasicFunctionalityTest

        Show
        Grant Ingersoll added a comment - - edited Can you add tests for this please? They are in the BasicFunctionalityTest
        Hide
        Raiko Eckstein added a comment -

        Yes, i try to have the tests ready soon.

        Show
        Raiko Eckstein added a comment - Yes, i try to have the tests ready soon.
        Hide
        Uri Boness added a comment -

        Updated this patch to work with the current trunk and added tests

        Show
        Uri Boness added a comment - Updated this patch to work with the current trunk and added tests
        Hide
        Hoss Man added a comment -

        Patch looks good, test looks good.

        thanks guys!

        Show
        Hoss Man added a comment - Patch looks good, test looks good. thanks guys!
        Hide
        Hoss Man added a comment -

        Correcting Fix Version based on CHANGES.txt, see this thread for more details...

        http://mail-archives.apache.org/mod_mbox/lucene-dev/201005.mbox/%3Calpine.DEB.1.10.1005251052040.24672@radix.cryptio.net%3E

        Show
        Hoss Man added a comment - Correcting Fix Version based on CHANGES.txt, see this thread for more details... http://mail-archives.apache.org/mod_mbox/lucene-dev/201005.mbox/%3Calpine.DEB.1.10.1005251052040.24672@radix.cryptio.net%3E
        Hide
        Grant Ingersoll added a comment -

        Bulk close for 3.1.0 release

        Show
        Grant Ingersoll added a comment - Bulk close for 3.1.0 release

          People

          • Assignee:
            Hoss Man
            Reporter:
            Raiko Eckstein
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development